Thanks Robert, But won’t just simply removing it mean that adding a render leaf will be changing what the current frame is? For occlusion nodes I’ve realized that it’s not currently possible to use them within instanced hierarchy without tracking which traversal generated a given query, the cull visitor traversal index (along with other changes) can help disambiguate which draw query it was. Gedalia
On Sun, May 13, 2018 at 6:55 AM Robert Osfield <robert.osfi...@gmail.com> wrote: > Hi Gedalia, > > Well spotted, this issue has been in the code for almost two decades > without anyone noticing. I've removed the duplicate from CullVisitor > and checked this into OSG master and the 3.6 branch. > > Robert. > > On 12 May 2018 at 21:34, Gedalia Pasternak <gpaster...@mak.com> wrote: > > Both CullVisitor and it's base class NodeVisitor have members named > > _traversalNumber. Yielding duplicate member variables with the same name, > > and inconsistent behavior depending on how you access the class. > > Cull visitor's should be renamed, maybe to _cullTraversalNumber or > something > > that wouldn't conflict with the base class. An accessor would be nice as > > well. In the case of NodeVisitor it's what frame it's up to, for cull > > visitor it's what accepted object it's up to ( > > // Otherwise need to create new renderleaf. > > RenderLeaf* renderleaf = new > > RenderLeaf(drawable,projection,matrix,depth,_traversalNumber++); > > ) > > > > -Gedalia > > > > _______________________________________________ > > osg-users mailing list > > osg-users@lists.openscenegraph.org > > > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > > > _______________________________________________ > osg-users mailing list > osg-users@lists.openscenegraph.org > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > -- DI-Guy Engineering Lead, VT MÄK 150 Cambridge Park Drive, 3rd Floor, Cambridge, MA 02140
_______________________________________________ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org