OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-05 18:16:23 +03:00
Code Packages Releases Activity

0023965: Making unnecessary copies of TopoDS_Face in Voxel_FastConverter when checking triangulation

Browse Source 
Using reference of TopoDS_Face instead of a copy.
Checking if the obtained triangulation is not Null before continuing.
This commit is contained in:
Pawel 2013-05-15 17:24:43 +02:00
parent eb57b012e8
commit 034b4775dc
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/Voxel/Voxel_FastConverter.cxx
View File

@ -121,7 +121,7 @@ void Voxel_FastConverter::Init()
TopExp_Explorer expl(myShape, TopAbs_FACE);
for (; expl.More(); expl.Next())
{
TopoDS_Face F = TopoDS::Face(expl.Current());
const TopoDS_Face & F = TopoDS::Face(expl.Current());
Handle(Poly_Triangulation) T = BRep_Tool::Triangulation(F, L);
if (T.IsNull() || (T->Deflection() > myDeflection))
{
@ -141,9 +141,10 @@ void Voxel_FastConverter::Init()
expl.Init(myShape, TopAbs_FACE);
for (; expl.More(); expl.Next())
{
TopoDS_Face F = TopoDS::Face(expl.Current());
const TopoDS_Face & F = TopoDS::Face(expl.Current());
Handle(Poly_Triangulation) T = BRep_Tool::Triangulation(F, L);
myNbTriangles += T->NbTriangles();
if (T.IsNull() == Standard_False)
myNbTriangles += T->NbTriangles();
}
}
@ -188,8 +189,10 @@ Standard_Boolean Voxel_FastConverter::Convert(Standard_Integer& progress,
TopExp_Explorer expl(myShape, TopAbs_FACE);
for (; expl.More(); expl.Next())
{
TopoDS_Face F = TopoDS::Face(expl.Current());
const TopoDS_Face & F = TopoDS::Face(expl.Current());
Handle(Poly_Triangulation) T = BRep_Tool::Triangulation(F, L);
if (T.IsNull())
continue;
gp_Trsf trsf;
Standard_Boolean transform = !L.IsIdentity();