(UniSetManager): Переделан интерфейс, оставлены только две функции

TODO

@@ -40,6 +40,8 @@ Version 2.0
 - ТЕСТЫ! ТЕСТЫ! ТЕСТЫ!
 
 
+- Перевести взаимодействие PassiveObject <--> ProxyManager на shared_ptr (или weak_ptr).
+
 
 Version 2.1
 ============
@@ -48,6 +50,10 @@ Version 2.1
 
 - IOBase: возможность задавать свою формулу преобразования входного сигнала (помимо калибровки)..
 
+- переписать систему инициализации и работы между UniSetActivator,UniSetManager и UniSetObject-ами
+(уменьшить количество функций (связность) и определится с кто отвечает за удаление объекта и т.п.)
+
+
 
 version 3
 =========

Utilities/NullController/main.cc

@@ -73,7 +73,7 @@ int main(int argc, char** argv)
 
         NullController nc(ID,askfile,s_field,s_fvalue,c_field,c_fvalue,dbDumping);
         UniSetActivatorPtr act = UniSetActivator::Instance();
-        act->addObject( nc.get_ptr() );
+        act->add( nc.get_ptr() );
         act->run(false);
         return 0;
     }

Utilities/SMonit/smonitor.cc

@@ -36,7 +36,7 @@ int main( int argc, const char **argv )
 
         UniSetActivatorPtr act = UniSetActivator::Instance();
         SMonitor tp(ID);
-        act->addObject(tp.get_ptr());
+        act->add(tp.get_ptr());
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

Utilities/codegen/ctl-cpp-main-alone.xsl

@@ -73,7 +73,7 @@ int main( int argc,char* argv[] )
 		obj.mylog.logFile( logname.c_str() );
 
 		auto act = UniSetActivator::Instance();
-		act->addObject(static_cast<class UniSetObject*>(&obj));
+		act->add(obj.get_ptr());
 
 		SystemMessage sm(SystemMessage::StartUp);
 		act->broadcast( sm.transport_msg() );

Utilities/codegen/ctl-cpp-main.xsl

@@ -81,7 +81,7 @@ int main( int argc, const char** argv )
 		</xsl:if>
 
 		auto act = UniSetActivator::Instance();
-		act-&gt;addObject(static_cast&lt;class UniSetObject*&gt;(&amp;obj));
+		act-&gt;add(obj.get_ptr());
 
 		SystemMessage sm(SystemMessage::StartUp); 
 		act-&gt;broadcast( sm.transport_msg() );

Utilities/codegen/skel-main.cc

@@ -26,7 +26,7 @@ int main( int argc, const char **argv )
         }
 
         Skel o("Skel",cnode);
-        act.addObject( static_cast<UniSetObject*>(&o) );
+        act.add(o.get_ptr());
 
         SystemMessage sm(SystemMessage::StartUp);
         act.broadcast( sm.transport_msg() );

extensions/DBServer-MySQL/main.cc

@@ -54,7 +54,7 @@ int main(int argc, char** argv)
 
         DBServer_MySQL dbs(ID);
         auto act = UniSetActivator::Instance();
-        act->addObject(dbs.get_ptr());
+        act->add(dbs.get_ptr());
         act->run(false);
     }
     catch(Exception& ex)

extensions/DBServer-SQLite/main.cc

@@ -55,7 +55,7 @@ int main(int argc, char** argv)
         DBServer_SQLite dbs(ID);
 
         auto act = UniSetActivator::Instance();
-        act->addObject(dbs.get_ptr());
+        act->add(dbs.get_ptr());
         act->run(false);
     }
     catch(Exception& ex)

extensions/IOControl/iocontrol.cc

@@ -53,7 +53,7 @@ int main(int argc, const char **argv)
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(ic);
+        act->add(ic);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/LogicProcessor/plogicproc.cc

@@ -71,7 +71,7 @@ int main(int argc, const char **argv)
         PassiveLProcessor plc(schema,ID,shmID);
 
         auto act = UniSetActivator::Instance();
-        act->addObject(plc.get_ptr());
+        act->add(plc.get_ptr());
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/ModbusMaster/main.cc

@@ -59,7 +59,7 @@ int main( int argc, const char** argv )
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(mb);
+        act->add(mb);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/ModbusMaster/multi-main.cc

@@ -58,7 +58,7 @@ int main( int argc, const char** argv )
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(mb);
+        act->add(mb);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/ModbusMaster/rtuexchange.cc

@@ -57,7 +57,7 @@ int main( int argc, char** argv )
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(rs);
+        act->add(rs);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/ModbusMaster/tests/run_test_mbtcpmaster.cc

@@ -47,8 +47,8 @@ int main(int argc, char* argv[] )
 
         auto act = UniSetActivator::Instance();
 
-        act->addObject(shm);
-        act->addObject(mb);
+        act->add(shm);
+        act->add(mb);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/ModbusMaster/tests/run_test_mbtcpmultimaster.cc

@@ -47,8 +47,8 @@ int main(int argc, char* argv[] )
 
         auto act = UniSetActivator::Instance();
 
-        act->addObject(shm);
-        act->addObject(mb);
+        act->add(shm);
+        act->add(mb);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/ModbusSlave/mbslave.cc

@@ -63,7 +63,7 @@ int main(int argc, const char **argv)
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(s);
+        act->add(s);
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );
 

extensions/ModbusSlave/mbtcp-multislave.cc

@@ -63,7 +63,7 @@ int main(int argc, const char **argv)
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(s);
+        act->add(s);
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );
 

extensions/ModbusSlave/tests/tests_with_sm.cc

@@ -46,8 +46,8 @@ int main(int argc, char* argv[] )
 
         auto act = UniSetActivator::Instance();
 
-        act->addObject(shm);
-        act->addObject(mbs);
+        act->add(shm);
+        act->add(mbs);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/RRDServer/main.cc

@@ -59,7 +59,7 @@ int main( int argc, const char** argv )
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(db);
+        act->add(db);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/SMDBServer/main.cc

@@ -23,8 +23,7 @@ int main( int argc, const char** argv )
 
     try
     {
-        string confile=UniSetTypes::getArgParam("--confile",argc, argv, "configure.xml");
-        conf = new Configuration( argc, argv, confile );
+        auto conf = uniset_init( argc, argv, confile );
 
         string logfilename(conf->getArgParam("--smdbserver-logfile"));
         if( logfilename.empty() )
@@ -51,24 +50,24 @@ int main( int argc, const char** argv )
             return 1;
         }
 
-        SMDBServer* db = SMDBServer::init_smdbserver(argc,argv,shmID);
+        auto db = SMDBServer::init_smdbserver(argc,argv,shmID);
         if( !db )
         {
             dcrit << "(smdbserver): init не прошёл..." << endl;
             return 1;
         }
 
-        UniSetActivator act;
-        act.addObject(static_cast<class UniSetObject*>(db));
+        auto act = UniSetActivator::Instance();
+        act->add(db);
 
         SystemMessage sm(SystemMessage::StartUp); 
-        act.broadcast( sm.transport_msg() );
+        act->broadcast( sm.transport_msg() );
 
         ulog << "\n\n\n";
         ulog << "(main): -------------- SMDBServer START -------------------------\n\n";
         dlog << "\n\n\n";
         dlog << "(main): -------------- SMDBServer START -------------------------\n\n";
-        act.run(false);
+        act->run(false);
         return 0;
     }
     catch( Exception& ex )

extensions/SharedMemory/smemory.cc

@@ -33,9 +33,9 @@ int main(int argc, const char **argv)
         if( !shm )
             return 1;
 
-        UniSetActivatorPtr act = UniSetActivator::Instance();
+        auto act = UniSetActivator::Instance();
 
-        act->addObject( shm ); // ->get_ptr() );
+        act->add(shm);
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );
         act->run(false);

extensions/SharedMemoryPlus/smemory-plus.cc

@@ -83,7 +83,7 @@ int main( int argc, const char **argv )
         if( !shm )
             return 1;
 
-        act->addManager(shm);
+        act->add(shm);
 
 #ifdef UNISET_ENABLE_IO
         // ------------ IOControl ----------------
@@ -113,7 +113,7 @@ int main( int argc, const char **argv )
                 if( io_thr == NULL )
                     return 1;
 
-                act->addObject(ic);
+                act->add(ic);
                 lst_iothr.push_back( io_thr );
             }
         }
@@ -139,7 +139,7 @@ int main( int argc, const char **argv )
                 if( !rtu )
                     return 1;
 
-                act->addObject(rtu);
+                act->add(rtu);
             }
         }
         // ------------- MBSlave --------------
@@ -163,7 +163,7 @@ int main( int argc, const char **argv )
                 if( !mbs )
                     return 1;
 
-                act->addObject(mbs);
+                act->add(mbs);
             }
         }
 
@@ -189,7 +189,7 @@ int main( int argc, const char **argv )
                 if( !mbm1 )
                     return 1;
 
-                act->addObject(mbm1);
+                act->add(mbm1);
             }
         }
         // ------------- UNetUDP --------------
@@ -203,7 +203,7 @@ int main( int argc, const char **argv )
             if( dlog.is_info() )
                 dlog.info() << "(smemory-plus): add UNetExchnage.." << endl;
 
-            act->addObject(unet);
+            act->add(unet);
         }
         // ---------------------------------------
            // попытка решить вопрос с "зомби" процессами

extensions/UNetUDP/tests/tests_with_sm.cc

@@ -46,8 +46,8 @@ int main(int argc, char* argv[] )
 
         auto act = UniSetActivator::Instance();
 
-        act->addObject(shm);
-        act->addObject(unet);
+        act->add(shm);
+        act->add(unet);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/UNetUDP/unetexchange.cc

@@ -57,7 +57,7 @@ int main( int argc, const char** argv )
         }
 
         auto act = UniSetActivator::Instance();
-        act->addObject(unet);
+        act->add(unet);
 
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );

extensions/UniNetwork/uninet.cc

@@ -46,7 +46,7 @@ int main(int argc, const char **argv)
 
         auto act = UniSetActivator::Instance();
 
-        act->addObject(uex);
+        act->add(uex);
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );
         act->run(true);

extensions/include/UObject_SK.h

@@ -8,7 +8,7 @@
  ВСЕ ВАШИ ИЗМЕНЕНИЯ БУДУТ ПОТЕРЯНЫ.
 */ 
 // --------------------------------------------------------------------------
-// generate timestamp: 2014-11-30+03:00
+// generate timestamp: 2015-01-12+03:00
 // -----------------------------------------------------------------------------
 #ifndef UObject_SK_H_
 #define UObject_SK_H_
@@ -125,6 +125,7 @@ class UObject_SK:
         int askPause; /*!< пауза между неудачными попытками заказать датчики */
         
         IOController_i::SensorInfo si;
+        bool forceOut; /*!< флаг принудительного обноления "выходов" */
 
 
     private:

extensions/lib/UObject_SK.cc

@@ -11,7 +11,7 @@
  ВСЕ ВАШИ ИЗМЕНЕНИЯ БУДУТ ПОТЕРЯНЫ.
 */ 
 // --------------------------------------------------------------------------
-// generate timestamp: 2014-11-30+03:00
+// generate timestamp: 2015-01-12+03:00
 // -----------------------------------------------------------------------------
 #include "Configuration.h"
 #include "Exceptions.h"
@@ -40,6 +40,7 @@ confnode(0),
 smReadyTimeout(0),
 activated(false),
 askPause(2000),
+forceOut(false),
 
 end_private(false)
 {
@@ -75,6 +76,7 @@ confnode(cnode),
 smReadyTimeout(0),
 activated(false),
 askPause(uniset_conf()->getPIntProp(cnode,"askPause",2000)),
+forceOut(false),
 
 end_private(false)
 {
@@ -97,8 +99,14 @@ end_private(false)
 
 
 
+
+
+
     UniXML::iterator it(cnode);
-    string heart = conf->getArgParam("--heartbeat-id",it.getProp("heartbeat_id"));
+
+    forceOut = conf->getArgPInt("--" + argprefix + "force-out",it.getProp("forceOut"),false);
+
+    string heart = conf->getArgParam("--" + argprefix + "heartbeat-id",it.getProp("heartbeat_id"));
     if( !heart.empty() )
     {
         idHeartBeat = conf->getSensorID(heart);
@@ -109,24 +117,24 @@ end_private(false)
             throw SystemError(err.str());
         }
 
-        int heartbeatTime = conf->getArgPInt("--heartbeat-time",it.getProp("heartbeatTime"),conf->getHeartBeatTime());
+        int heartbeatTime = conf->getArgPInt("--" + argprefix + "heartbeat-time",it.getProp("heartbeatTime"),conf->getHeartBeatTime());
         if( heartbeatTime>0 )
             ptHeartBeat.setTiming(heartbeatTime);
         else
             ptHeartBeat.setTiming(UniSetTimer::WaitUpTime);
 
-        maxHeartBeat = conf->getArgPInt("--heartbeat-max",it.getProp("heartbeat_max"), 10);
+        maxHeartBeat = conf->getArgPInt("--" + argprefix + "heartbeat-max",it.getProp("heartbeat_max"), 10);
     }
 
     // Инициализация значений
     
     
-    sleep_msec = conf->getArgPInt("--sleep-msec","150", 150);
+    sleep_msec = conf->getArgPInt("--" + argprefix + "sleep-msec","150", 150);
 
     resetMsgTime = conf->getPIntProp(cnode,"resetMsgTime", 2000);
     ptResetMsg.setTiming(resetMsgTime);
 
-    smReadyTimeout = conf->getArgInt("--sm-ready-timeout","");
+    smReadyTimeout = conf->getArgInt("--" + argprefix + "sm-ready-timeout","");
     if( smReadyTimeout == 0 )
         smReadyTimeout = 60000;
     else if( smReadyTimeout < 0 )
@@ -135,9 +143,9 @@ end_private(false)
     smTestID = conf->getSensorID(init3_str(conf->getArgParam("--" + argprefix + "sm-test-id"),conf->getProp(cnode,"smTestID"),""));
     
 
-    activateTimeout    = conf->getArgPInt("--activate-timeout", 20000);
+    activateTimeout    = conf->getArgPInt("--" + argprefix + "activate-timeout", 20000);
 
-    int msec = conf->getArgPInt("--startup-timeout", 10000);
+    int msec = conf->getArgPInt("--" + argprefix + "startup-timeout", 10000);
     ptStartUpTimeout.setTiming(msec);
 
     // ===================== <variables> =====================
@@ -367,7 +375,7 @@ void UObject_SK::callback()
             if( !receiveMessage(msg) )
                 break;
             processingMessage(&msg);
-            updateOutputs(false);
+            updateOutputs(forceOut);
 //            updatePreviousValues();
         }
 
@@ -382,7 +390,7 @@ void UObject_SK::callback()
         }
 
         // обновление выходов
-        updateOutputs(false);
+        updateOutputs(forceOut);
         updatePreviousValues();
     }
     catch( Exception& ex )

extensions/tests/SMemoryTest/smemory-test.cc

@@ -29,13 +29,13 @@ int main(int argc, const char **argv)
         ulog.logFile( logname );
         dlog.logFile( logname );
 
-        SharedMemory* shm = SharedMemory::init_smemory(argc, argv);
+        auto shm = SharedMemory::init_smemory(argc, argv);
         if( !shm )
             return 1;
 
-        UniSetActivatorPtr act = UniSetActivator::Instance();
+        auto act = UniSetActivator::Instance();
 
-        act->addObject(static_cast<class UniSetObject*>(shm));
+        act->add(shm);
 
         int num = conf->getArgPInt("--numproc",1);
 
@@ -47,7 +47,7 @@ int main(int argc, const char **argv)
             cout << "..create " << s.str() << endl;
             TestProc* tp = new TestProc(conf->getObjectID(s.str()));
             tp->init_dlog(dlog);
-            act->addObject(static_cast<class UniSetObject*>(tp));
+            act->add(tp->get_ptr());
         }
 
         SystemMessage sm(SystemMessage::StartUp);

extensions/tests/tests_with_sm.cc

@@ -70,7 +70,7 @@ int main(int argc, char* argv[] )
         auto act = UniSetActivator::Instance();
 
 
-        act->addObject(shm);
+        act->add(shm);
         SystemMessage sm(SystemMessage::StartUp);
         act->broadcast( sm.transport_msg() );
         act->run(true);

include/UniSetActivator.h

@@ -69,7 +69,7 @@ class UniSetActivator:
         virtual void oaDestroy(int signo=0);
         void waitDestroy();
 
-        inline void oakill(int signo){ raise(signo);}
+//        inline void oakill(int signo){ raise(signo); }
 
         virtual UniSetTypes::ObjectType getType() override { return UniSetTypes::ObjectType("UniSetActivator"); }
 
@@ -114,7 +114,6 @@ class UniSetActivator:
 
     private:
 
-//        static void processingSignal(int signo);
         static void terminated(int signo);
         static void finishterm(int signo);
         static void normalexit();
@@ -123,8 +122,7 @@ class UniSetActivator:
         void term( int signo );
         void init();
 
-        friend class ThreadCreator<UniSetActivator>;
-        ThreadCreator<UniSetActivator> *orbthr;
+        std::shared_ptr< ThreadCreator<UniSetActivator> > orbthr;
 
         CORBA::ORB_var orb;
         TerminateEvent_Signal s_term;
@@ -132,31 +130,6 @@ class UniSetActivator:
         bool omDestroy;
         bool sig;
         pid_t thpid; // pid orb потока
-
-        struct Info
-        {
-            Info( pid_t p ):msgpid(p){}
-            
-            pid_t msgpid;    // pid порожденого потока обработки сообщений
-        };
-
-        struct OInfo:
-            public Info
-        {
-            OInfo( std::shared_ptr<UniSetObject> o, pid_t p ):Info(p),obj(o){}
-            std::shared_ptr<UniSetObject> obj;
-        };
-
-        struct MInfo:
-            public Info
-        {
-            MInfo( std::shared_ptr<UniSetManager> m, pid_t p ):Info(p),mnr(m){}
-            std::shared_ptr<UniSetManager> mnr;
-        };
-
-        std::deque<OInfo> lstOInfo;
-        std::deque<MInfo> lstMInfo;
-        void getinfo();
 };
 
 /*

include/UniSetManager.h

@@ -72,26 +72,19 @@ class UniSetManager:
         virtual UniSetTypes::SimpleInfoSeq* getObjectsInfo( CORBA::Long MaxLength=300 ) override ;
 
         // --------------------------
-        void initPOA( const std::weak_ptr<UniSetManager>& rmngr );
-
-        virtual bool addObject( const std::shared_ptr<UniSetObject>& obj );
-        virtual bool removeObject( const std::shared_ptr<UniSetObject>& obj );
-
-        virtual bool addManager( const std::shared_ptr<UniSetManager>& mngr );
-        virtual bool removeManager( const std::shared_ptr<UniSetManager>& mngr );
-
+        virtual bool add( const std::shared_ptr<UniSetObject>& obj );
+        virtual bool remove( const std::shared_ptr<UniSetObject>& obj );
+        // --------------------------
         /*! Получение доступа к подчиненному менеджеру по идентификатору
-         * \return объект ненайден будет возвращен 0.
+         * \return shared_ptr<>, если объект не найден будет возвращен shared_ptr<> = nullptr
         */ 
         const std::shared_ptr<UniSetManager> itemM(const UniSetTypes::ObjectId id);
 
-
         /*! Получение доступа к подчиненному объекту по идентификатору
-         * \return объект ненайден будет возвращен 0.
+         * \return shared_ptr<>, если объект не найден будет возвращен shared_ptr<> = nullptr
         */ 
         const std::shared_ptr<UniSetObject> itemO( const UniSetTypes::ObjectId id );
 
-
         // Функции для аботы со списками подчиненных объектов
         inline UniSetManagerList::const_iterator beginMList()
         {
@@ -122,6 +115,11 @@ class UniSetManager:
 
         UniSetManager();
 
+        virtual bool addManager( const std::shared_ptr<UniSetManager>& mngr );
+        virtual bool removeManager( const std::shared_ptr<UniSetManager>& mngr );
+        virtual bool addObject( const std::shared_ptr<UniSetObject>& obj );
+        virtual bool removeObject( const std::shared_ptr<UniSetObject>& obj );
+
         enum OManagerCommand{ deactiv, activ, initial, term };
         friend std::ostream& operator<<(std::ostream& os, OManagerCommand& cmd );
 
@@ -132,6 +130,8 @@ class UniSetManager:
 
         virtual void sigterm( int signo ) override;
 
+        void initPOA( const std::weak_ptr<UniSetManager>& rmngr );
+
         //! \note Переопределяя не забывайте вызвать базовую 
         virtual bool activateObject() override;
         //! \note Переопределяя не забывайте вызвать базовую 
@@ -147,8 +147,6 @@ class UniSetManager:
 
     private:
 
-        friend class UniSetActivator;
-
         int sig;
         UniSetManagerList mlist;
         ObjectsList olist;

src/ObjectRepository/UniSetActivator.cc

@@ -98,7 +98,6 @@ void UniSetActivator::Destroy()
 // ---------------------------------------------------------------------------
 UniSetActivator::UniSetActivator( const ObjectId id ):
 UniSetManager(id),
-orbthr(0),
 omDestroy(false),
 sig(false)
 {
@@ -107,7 +106,6 @@ sig(false)
 // ------------------------------------------------------------------------------------------
 UniSetActivator::UniSetActivator():
 UniSetManager(UniSetTypes::DefaultObjectId),
-orbthr(0),
 omDestroy(false),
 sig(false)
 {
@@ -146,8 +144,7 @@ UniSetActivator::~UniSetActivator()
         if( orbthr->isRunning() )
             orbthr->join();
 
-        delete orbthr;
-        orbthr = 0;
+        orbthr.reset();
     }
 
     if( !procterm )
@@ -205,7 +202,6 @@ void UniSetActivator::oaDestroy(int signo)
             pman->deactivate(false,true);
             ulogsys << myname << "(oaDestroy): pman deactivate ok. " << endl;
 
-            ulogsys << myname << "(oaDestroy): orbthr=" << orbthr << endl;
             if( orbthr )
             {
                 try
@@ -263,7 +259,6 @@ void UniSetActivator::run( bool thread )
 
     UniSetManager::activate(); // а там вызывается активация всех подчиненных объектов и менеджеров
 
-    getinfo();        // заполнение информации об объектах
     active=true;
 
     ulogsys << myname << "(run): активизируем менеджер"<<endl;
@@ -274,7 +269,7 @@ void UniSetActivator::run( bool thread )
     if( thread )
     {
         uinfo << myname << "(run): запускаемся с созданием отдельного потока...  "<< endl;
-        orbthr = new ThreadCreator<UniSetActivator>(this, &UniSetActivator::work);
+        orbthr = make_shared< ThreadCreator<UniSetActivator> >(this, &UniSetActivator::work);
         int ret = orbthr->start();
         if( ret !=0 )
         {
@@ -327,7 +322,7 @@ void UniSetActivator::stop()
 
 void UniSetActivator::work()
 {
-    ulogsys << myname << "(work): запускаем orb на обработку запросов...(orbthr=" << orbthr << ")" << endl;
+    ulogsys << myname << "(work): запускаем orb на обработку запросов..." << endl;
     try
     {
         if( orbthr )
@@ -371,15 +366,6 @@ void UniSetActivator::work()
     ulogsys << myname << "(oaDestroy): orb destroy ok."<< endl;
 }
 // ------------------------------------------------------------------------------------------
-void UniSetActivator::getinfo()
-{
-    for( auto it=beginMList(); it!=endMList(); ++it )
-        lstMInfo.emplace_back( (*it), (*it)->getMsgPID() );
-
-    for( auto it=beginOList(); it!= endOList(); ++it )
-        lstOInfo.emplace_back( (*it), (*it)->getMsgPID() );
-}
-// ------------------------------------------------------------------------------------------
 void UniSetActivator::sysCommand( const UniSetTypes::SystemMessage *sm )
 {
     switch(sm->command)
@@ -541,8 +527,7 @@ void UniSetActivator::term( int signo )
             if( orbthr->isRunning() )
                 orbthr->join();
 
-            delete orbthr;
-            orbthr = 0;
+            orbthr.reset();
         }
 
         sigterm(signo);

src/ObjectRepository/UniSetManager.cc

@@ -88,8 +88,8 @@ class OPush: public unary_function< const std::shared_ptr<UniSetObject>& , bool>
 UniSetManager::UniSetManager():
 UniSetObject(UniSetTypes::DefaultObjectId),
 sig(0),
-olistMutex("olistMutex"),
-mlistMutex("mlistMutex")
+olistMutex("UniSetManager_olistMutex"),
+mlistMutex("UniSetManager_mlistMutex")
 {
 }
 // ------------------------------------------------------------------------------------------
@@ -179,8 +179,27 @@ void UniSetManager::initPOA( const std::weak_ptr<UniSetManager>& rmngr )
     managers(initial);
 }
 // ------------------------------------------------------------------------------------------
+bool UniSetManager::add( const std::shared_ptr<UniSetObject>& obj )
+{
+    auto m = std::dynamic_pointer_cast<UniSetManager>(obj);
+    if( m )
+        return addManager(m);
+
+    return addObject(obj);
+}
+// ------------------------------------------------------------------------------------------
+bool UniSetManager::remove( const std::shared_ptr<UniSetObject>& obj )
+{
+    auto m = std::dynamic_pointer_cast<UniSetManager>(obj);
+    if( m )
+        return removeManager(m);
+
+    return removeObject(obj);
+}
+// ------------------------------------------------------------------------------------------
 bool UniSetManager::addObject( const std::shared_ptr<UniSetObject>& obj )
 {
+
     {    //lock
         uniset_rwmutex_wrlock lock(olistMutex);
         auto li=find(olist.begin(),olist.end(), obj);
@@ -493,11 +512,8 @@ int UniSetManager::objectsCount()
 {
     int res( olist.size()+mlist.size() );
 
-    for( UniSetManagerList::const_iterator it= beginMList();
-            it!= endMList(); ++it )
-    {
-        res+= (*it)->objectsCount();
-    }
+    for( auto i: mlist )
+        res += i->objectsCount();
 
     return res;
 }
@@ -541,9 +557,9 @@ int UniSetManager::getObjectsInfo( const std::shared_ptr<UniSetManager>& mngr, S
         return ind;
 
     // а далее у его менеджеров (рекурсивно)
-    for( auto it=mngr->beginMList(); it!=mngr->endMList(); ++it )
+    for( auto &i: mlist )
     {
-        ind = getObjectsInfo((*it),seq,ind,uplimit);
+        ind = getObjectsInfo(i,seq,ind,uplimit);
         if( ind > uplimit )
             break;
     }