Hi Robert, I'm definitely in favor of this change. It will simplify a lot of code.
Regarding the change to Node::ParentList, I think this will cause some breakage. I know I have a lot of code that assumes the elements of ParentList are group nodes and performs Group operations on them (addChild, removeChild, etc..). Perhaps Geode could be modified to inherit from Group instead? The add/remove drawable methods could simply be wrappers for add/remove child. Just throwing the idea out there, I'm sure this would cause a whole host of other problems though. In general, I think breaking code is going to be unavoidable. The bounding sphere/box wrapper class is an interesting solution. I'm worried about the potential for segfaults though. It might be better to simply rename the method and force people to update their code at compile time, rather than dealing with possible segfaults at runtime. Cheers, Farshid
_______________________________________________ osg-users mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
