haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add multi-level fallback in tesselation 

Some clipper cases do not get resolved with the `FAST` mode, so in those
cases, do a second pass in the `SIMPLE` mode.  If both fail, return a
broken polygon instead of an infinite loop

Fixes https://gitlab.com/kicad/code/kicad/issues/12761
This commit is contained in:
Seth Hillbrand 2022-10-28 10:05:04 -07:00
parent fb21eb7457
commit 546e16a002
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
libs/kimath/src/geometry/shape_poly_set.cpp
View File

@ -2695,6 +2695,7 @@ void SHAPE_POLY_SET::CacheTriangulation( bool aPartition )
std::vector<std::unique_ptr<TRIANGULATED_POLYGON>>& dest )
{
bool triangulationValid = false;
int pass = 0;
while( polySet.OutlineCount() > 0 )
{
@ -2709,7 +2710,15 @@ void SHAPE_POLY_SET::CacheTriangulation( bool aPartition )
// This may result in multiple, disjoint polygons.
if( !tess.TesselatePolygon( polySet.Polygon( 0 ).front() ) )
{
polySet.Fracture( PM_FAST );
++pass;
if( pass == 1 )
polySet.Fracture( PM_FAST );
else if( pass == 2 )
polySet.Fracture( PM_STRICTLY_SIMPLE );
else
break;
triangulationValid = false;
continue;
}