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

Merge pull request #148 from LaurensVoerman/SmoothingVisitor

Browse Source 
fix bug in SmoothingVisitor tripped by bunny.ply
after duplicating the vertices to allow for multiple normals the indices of the new mesh (with duplicated vertices) were used with the vertices of the old mesh, causing a vector subscript out of range error.
This commit is contained in:
OpenSceneGraph git repository 2016-10-06 14:14:02 +01:00 committed by GitHub
commit 0a3b2402c8
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/osgUtil/SmoothingVisitor.cpp
View File

@ -659,14 +659,16 @@ static void smooth_new(osg::Geometry& geom, double creaseAngle)
// fsef.listProblemVertices();
fsef.updateGeometry();
vertices = dynamic_cast<osg::Vec3Array*>(geom.getVertexArray());
normals = dynamic_cast<osg::Vec3Array*>(geom.getNormalArray());
osg::TriangleIndexFunctor<SmoothTriangleIndexFunctor> stif2;
if (stif2.set(vertices, normals))
{
// accumulate all the normals
geom.accept(stif);
geom.accept(stif2);
// normalize the normals
stif.normalize();
stif2.normalize();
}
}