Gerrit Voss wrote:
> On Tue, 2006-10-03 at 17:14 -0500, [EMAIL PROTECTED] wrote:
>> Author: allenb
>> Date: Tue Oct  3 17:14:39 2006
>> New Revision: 226
>> Trac changeset: http://opensg.vrsource.org/trac/changeset/226
>> ViewVC: http://realityforge.vrsource.org/viewvc/opensg?view=rev&revision=226
>>
>> Modified:
>>    trunk/Source/System/FieldContainer/Base/OSGReflexiveContainer.h
>> Log:
>> Make Reflexive container destructor public so other classes can destroy it 
>> when they compose objects or derive from this class.  (boost internals 
>> needed it).
> 
> this looks dangerous, nobody should ever derive from this class
> (except for FieldContainer and FieldBundle). 

Agreed.

> If you manage
> to call this destructor on anything a OpenSG factory created
> you will end up with an inconsistent system. Could you
> give a little background why this one needs to be public ??

I don't have much information on it.  The code calling is deep inside 
boost and is inside some metaprogramming code created by the 
preprocessor library.  In other words it is not clear why it is being 
called.

I would like to investigate it but right now (the next couple weeks) I 
don't have much time to track it down.  For now what if we made a ticket 
to say this should be removed/fixed/understood before the first 2.0 
release?

-Allen


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Opensg-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensg-users

Reply via email to