Pixyco-XYZ/openmesh
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'Fix_calc_sector_angle' into 'master'

Browse Source 
calc_sector_angle: Check for real division by zero not with epsilon that was way to large

See merge request OpenMesh/OpenMesh!148
This commit is contained in:
Jan Möbius
2018-01-05 11:47:55 +01:00
parent 62c22a0100 548a73bfad
commit 7856a4371b
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
Doc/changelog.docu
View File

@@ -24,6 +24,7 @@
<li>Added 1-4 triangle split funtion(splits all edges at Midpoints)</li> <li>Added 1-4 triangle split funtion(splits all edges at Midpoints)</li>
<li>Boost range support (Thanks to Bastian Pranzas for the patch)</li> <li>Boost range support (Thanks to Bastian Pranzas for the patch)</li>
<li>Made the face and edge split operations that copy properties also copy builtin properties</li> <li>Made the face and edge split operations that copy properties also copy builtin properties</li>
<li>calc_sector_angle: Check for real division by zero not with epsilon that was way to large</li>
</ul> </ul>
<b>Utils</b> <b>Utils</b>

2
src/OpenMesh/Core/Mesh/PolyMeshT.hh
View File

@@ -445,7 +445,7 @@ public:
Normal v0, v1; Normal v0, v1;
calc_sector_vectors(_in_heh, v0, v1); calc_sector_vectors(_in_heh, v0, v1);
Scalar denom = v0.norm()*v1.norm(); Scalar denom = v0.norm()*v1.norm();
if (is_zero(denom)) if ( denom == Scalar(0))
{ {
return 0; return 0;
} }