<div dir="ltr"><div><div>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.</div><div>

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 (  <br></div><div>   // Otherwise need to create new renderleaf.</div>    RenderLeaf* renderleaf = new RenderLeaf(drawable,projection,matrix,depth,_traversalNumber++);<br>)<br></div><br><div><div>-Gedalia<br></div></div></div>