FGFS-4.1/OpenSceneGraph
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

From Pjotr Svetachov, "small bug which I have found in the

Browse Source 
FlattenStaticTransformsDuplicatingSharedSubgraphsVisitor: the visitor
will replace part of the tree when visiting the nodes but it does not
update the current node path to reflect this. If you then have
multiple nested transform nodes it will usually crash and sometimes it
won't crash but the resulting mesh will miss some nodes."
This commit is contained in:
Robert Osfield
2013-01-23 17:37:26 +00:00
parent cd34312ea3
commit d3e9b61f2b
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/osgUtil/Optimizer.cpp
View File

@@ -4631,6 +4631,8 @@ void Optimizer::FlattenStaticTransformsDuplicatingSharedSubgraphsVisitor::apply(
if(parent_group)
{
parent_group->replaceChild(&group, new_group);
// also replace the node in the nodepath
_nodePath[nodepathsize-1] = new_group;
// traverse the new Group
traverse(*(new_group));
}
@@ -4672,6 +4674,8 @@ void Optimizer::FlattenStaticTransformsDuplicatingSharedSubgraphsVisitor::apply(
if(parent_group)
{
parent_group->replaceChild(&transform, group.get());
// also replace the node in the nodepath
_nodePath[nodepathsize-1] = group;
// traverse the new Group
traverse(*(group.get()));
}
@@ -4706,7 +4710,8 @@ void Optimizer::FlattenStaticTransformsDuplicatingSharedSubgraphsVisitor::apply(
if(parent_group)
{
parent_group->replaceChild(&lod, new_lod.get());
// also replace the node in the nodepath
_nodePath[nodepathsize-1] = new_lod;
// move center point
if(!_matrixStack.empty())
new_lod->setCenter(new_lod->getCenter() * _matrixStack.back());