ForkMaster/geom
1
0
Fork 0
mirror of https://git.salome-platform.org/gitpub/modules/geom.git synced 2025-04-24 07:52:03 +05:00
Code Issues Packages Projects Releases Wiki Activity

Fixed BUG with omniORB refCount. see file comment

Browse Source
This commit is contained in:
abd 2006-04-21 12:45:44 +00:00
parent 0483cfa3b7
commit 4c5ba18e8d
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/GEOM_I/GEOM_IOperations_i.cc
View File

@ -38,7 +38,16 @@
GEOM_IOperations_i::GEOM_IOperations_i(PortableServer::POA_ptr thePOA, GEOM::GEOM_Gen_ptr theEngine, ::GEOM_IOperations* theImpl) GEOM_IOperations_i::GEOM_IOperations_i(PortableServer::POA_ptr thePOA, GEOM::GEOM_Gen_ptr theEngine, ::GEOM_IOperations* theImpl)
:SALOME::GenericObj_i( thePOA ), _impl(theImpl), _engine(theEngine) :SALOME::GenericObj_i( thePOA ), _impl(theImpl), _engine(theEngine)
{ {
thePOA->activate_object(this); // Win32 porting: the next line is dangerous - GEOM_IOperations_i is an intermediate
// base class, therefore <this> is not completely constructed here ->
// passing it to activate_object() leads to unpredictable behavior
// resulted from memory corruption ( Rational Purify reports ABR errors ).
// Moreover, all GEOM_IxxxOperation_i servant classes are activated implicitly
// by GEOM_Gen_i::GetxxxOperations() methods.
// Therefore, this line is commented.
// In case if <thePOA> does not have ImplicitActivation policy, then
// activate_object() calls will be necessary in all GEOM_IxxxOperation_i constructors!
//thePOA->activate_object(this);
} }
//============================================================================= //=============================================================================