Hi Tugkan,

On 2/23/07, Tugkan Calapoglu <[EMAIL PROTECTED]> wrote:
Hi Robert,

In include/osg/DeleteHandler there is following method:

inline void doDelete(const Referenced* object) { delete object; }

Shouldn't it be object->unref() instead?

No, the DeleteHandler does the actual delete itself.  Objects
shouldn't get into the DeleteHandler unless their ref count has gone
to zero and they are ready for deletion.

Perhaps one could add a check into the DeleteHandler so it double
checks the ref count on objects being added and when they are about to
be deleted.   This might catch this problem case from causing a crash,
but there is clearly a problem further upstream.

Robert.
_______________________________________________
osg-users mailing list
osg-users@openscenegraph.net
http://openscenegraph.net/mailman/listinfo/osg-users
http://www.openscenegraph.org/

Reply via email to