Changeset 2243

Timestamp:

10/01/21 10:40:31 (3 years ago)

Author:

jderouillat

Message:

Move context cleaning in xios_finalize (for clients) and when servers have finished their work

Location:

XIOS/dev/dev_ym/XIOS_COUPLING/src

Files:

• cxios.cpp (modified) (2 diffs)
• interface/c/icdata.cpp (modified) (1 diff)
• object_template.hpp (modified) (1 diff)
• object_template_impl.hpp (modified) (1 diff)
• server.cpp (modified) (1 diff)

XIOS/dev/dev_ym/XIOS_COUPLING/src/cxios.cpp

@@ -175 +175 @@
 
 #endif
-     
-     // Delete CContext
-     xios_map<StdString, xios_map<StdString, std::shared_ptr<CContext> > >* contextMap = CObjectTemplate<CContext>::GetAllMapobject();
-     for(auto it = contextMap->begin(); it != contextMap->end(); ++it)
-     {
-       for(auto it2 = it->second.begin(); it2 != it->second.end(); ++it2)
-       {
-           std::shared_ptr<CContext> todel = it2->second;
-           todel.reset();
-       }
-       it->second.clear();
-
-       std::vector<std::shared_ptr<CContext> >*  contextVect = CObjectTemplate<CContext>::GetAllVectobjectPtr(it->first);
-       for(auto it = contextVect->begin(); it != contextVect->end(); ++it)
-       {
-           it->reset();
-       }
-       contextVect->clear();
-       
-     }
-     contextMap->clear();
-     
   }
 
@@ -247 +225 @@
     report(0)<<CMemChecker::getAllCumulatedMem() ;
     CServer::closeInfoStream();
-    
-    // Delete CContext
-    xios_map<StdString, xios_map<StdString, std::shared_ptr<CContext> > >* contextMap = CObjectTemplate<CContext>::GetAllMapobject();
-    for(auto it = contextMap->begin(); it != contextMap->end(); ++it)
-    {
-      for(auto it2 = it->second.begin(); it2 != it->second.end(); ++it2)
-      {
-          std::shared_ptr<CContext> todel = it2->second;
-          todel.reset();
-      }
-      it->second.clear();
-
-      std::vector<std::shared_ptr<CContext> >*  contextVect = CObjectTemplate<CContext>::GetAllVectobjectPtr(it->first);
-      for(auto it = contextVect->begin(); it != contextVect->end(); ++it)
-      {
-          it->reset();
-      }
-      contextVect->clear();
-      
-    }
-    contextMap->clear();
   }
 

XIOS/dev/dev_ym/XIOS_COUPLING/src/interface/c/icdata.cpp

@@ -146 +146 @@
      CTimer::get("XIOS finalize").resume();
      CXios::clientFinalize();
+     // Delete CContext
+     CObjectTemplate<CContext>::cleanStaticDataStructure();
    }
    CATCH_DUMP_STACK

XIOS/dev/dev_ym/XIOS_COUPLING/src/object_template.hpp

@@ -69 +69 @@
          static std::vector<std::shared_ptr<DerivedType> > &
             GetAllVectobject(const StdString & contextId);
-         static std::vector<std::shared_ptr<DerivedType> > *
-            GetAllVectobjectPtr(const StdString & contextId);
-
-         static xios_map<StdString, xios_map<StdString, std::shared_ptr<DerivedType> > >*
-            GetAllMapobject();
-
 
          /// Destructeur ///
          virtual ~CObjectTemplate(void);
+         
+         static void cleanStaticDataStructure(void);     
 
          static bool has(const string& id) ;

XIOS/dev/dev_ym/XIOS_COUPLING/src/object_template_impl.hpp

@@ -68 +68 @@
       return (CObjectTemplate<T>::AllVectObj[contextId]);
    }
-   template <class T>
-      std::vector<std::shared_ptr<T> > *
-         CObjectTemplate<T>::GetAllVectobjectPtr(const StdString & contextId)
-   {
-      return &(CObjectTemplate<T>::AllVectObj[contextId]);
-   }
-
-  template <class T>
-      xios_map<StdString, xios_map<StdString, std::shared_ptr<T> > >*
-         CObjectTemplate<T>::GetAllMapobject()
-   {
-      return &(CObjectTemplate<T>::AllMapObj);
+  
+   template <class T>
+      void CObjectTemplate<T>::cleanStaticDataStructure(void)
+   {
+     xios_map<StdString, xios_map<StdString, std::shared_ptr<T> > >* allMap = &(CObjectTemplate<T>::AllMapObj);
+      for(auto it = allMap->begin(); it != allMap->end(); ++it)
+      {
+        for(auto it2 = it->second.begin(); it2 != it->second.end(); ++it2)
+        {
+          std::shared_ptr<T> todel = it2->second;
+          todel.reset();
+        }
+        it->second.clear();
+
+        std::vector<std::shared_ptr<T> >* allVect = &(CObjectTemplate<T>::AllVectObj[it->first]);
+        for(auto it = allVect->begin(); it != allVect->end(); ++it)
+        {
+          it->reset();
+        }
+        allVect->clear();
+      }
+      allMap->clear();
    }
 

XIOS/dev/dev_ym/XIOS_COUPLING/src/server.cpp

@@ -205 +205 @@
       }
       CTimer::get("XIOS event loop").suspend() ;
+
+      // Delete CContext
+      CObjectTemplate<CContext>::cleanStaticDataStructure();
     }