Hi Anna, I've looked into the NodeTrackerManipulator::validateNodePath() implementation and can confirm that the "itr != _trackNodePath.begin()" is an error.
When reviewing the implementation I decided that the code would be better off using the new osg::ObserverNodePath rather than it's own local equivalent container. I have changed NodeTrackerManipulator across to using osg::ObserverNodePath and in doing so the validateNodePath() is no longer required, so it along with the erroneous code is now gone ;-) Updating to svn/trunk will get your this improvement. Robert. On Tue, Feb 1, 2011 at 10:48 AM, Anna Palazzolo <[email protected]> wrote: > Hi, > > I was investigating about some problems I'm having using the > NodeTrackerManipulator, and while debugging the code I reached the call to > the NodeTrackerManipulator validateNodePath method > > bool NodeTrackerManipulator::validateNodePath() const > { > for(ObserverNodePath::const_iterator itr = _trackNodePath.begin(); > itr != _trackNodePath.begin(); > ++itr) > { ..... > > Maybe I'm missing something, but it seems to me that the for condition is > never satisfied: itr=.begin; itr!=.begin, ++itr , so the inner code is never > executed. > > Am I right? > > > Thank you! > > Cheers, > Anna > > ------------------ > Read this topic online here: > http://forum.openscenegraph.org/viewtopic.php?p=36220#36220 > > > > > > _______________________________________________ > osg-users mailing list > [email protected] > http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org > _______________________________________________ osg-users mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
