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

Handle closed shapes being processed after other shapes. 

Fixes https://gitlab.com/kicad/code/kicad/issues/6845
This commit is contained in:
Jeff Young 2020-12-26 10:59:06 +00:00
parent fd77982496
commit ab97035ecb
1 changed files with 118 additions and 97 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
pcbnew/convert_drawsegment_list_to_polygon.cpp
View File

@ -197,10 +197,10 @@ bool ConvertOutlineToPolygon( std::vector<PCB_SHAPE*>& aSegList, SHAPE_POLY_SET&
return true; return true;
bool polygonComplete = false; bool polygonComplete = false;
bool selfIntersecting = false;
wxString msg; wxString msg;
PCB_SHAPE* graphic; PCB_SHAPE* graphic = nullptr;
wxPoint prevPt;
std::set<PCB_SHAPE*> startCandidates( aSegList.begin(), aSegList.end() ); std::set<PCB_SHAPE*> startCandidates( aSegList.begin(), aSegList.end() );
@ -337,8 +337,10 @@ bool ConvertOutlineToPolygon( std::vector<PCB_SHAPE*>& aSegList, SHAPE_POLY_SET&
// Grab the left most point, assume its on the board's perimeter, and see if we can put // Grab the left most point, assume its on the board's perimeter, and see if we can put
// enough graphics together by matching endpoints to formulate a cohesive polygon. // enough graphics together by matching endpoints to formulate a cohesive polygon.
graphic = (PCB_SHAPE*) aSegList[xmini]; PCB_SHAPE* prevGraphic = nullptr;
wxPoint prevPt;
graphic = (PCB_SHAPE*) aSegList[xmini];
graphic->SetFlags( SKIP_STRUCT ); graphic->SetFlags( SKIP_STRUCT );
startCandidates.erase( graphic ); startCandidates.erase( graphic );
@ -413,6 +415,23 @@ bool ConvertOutlineToPolygon( std::vector<PCB_SHAPE*>& aSegList, SHAPE_POLY_SET&
{ {
switch( graphic->GetShape() ) switch( graphic->GetShape() )
{ {
case S_RECT:
case S_CIRCLE:
{
// As a non-first item, closed shapes can't be anything but self-intersecting
if( aErrorHandler )
{
wxASSERT( prevGraphic );
(*aErrorHandler)( _( "(self-intersecting)" ), prevGraphic, graphic, prevPt );
}
selfIntersecting = true;
// A closed shape will finish where it started, so no point in updating prevPt
}
break;
case S_SEGMENT: case S_SEGMENT:
{ {
wxPoint nextPt; wxPoint nextPt;
@ -432,9 +451,10 @@ bool ConvertOutlineToPolygon( std::vector<PCB_SHAPE*>& aSegList, SHAPE_POLY_SET&
break; break;
case S_ARC: case S_ARC:
{
// We do not support arcs in polygons, so approximate an arc with a series of // We do not support arcs in polygons, so approximate an arc with a series of
// short lines and put those line segments into the !same! PATH. // short lines and put those line segments into the !same! PATH.
{
wxPoint pstart = graphic->GetArcStart(); wxPoint pstart = graphic->GetArcStart();
wxPoint pend = graphic->GetArcEnd(); wxPoint pend = graphic->GetArcEnd();
wxPoint pcenter = graphic->GetCenter(); wxPoint pcenter = graphic->GetCenter();
@ -470,9 +490,10 @@ bool ConvertOutlineToPolygon( std::vector<PCB_SHAPE*>& aSegList, SHAPE_POLY_SET&
break; break;
case S_CURVE: case S_CURVE:
{
// We do not support Bezier curves in polygons, so approximate with a series // We do not support Bezier curves in polygons, so approximate with a series
// of short lines and put those line segments into the !same! PATH. // of short lines and put those line segments into the !same! PATH.
{
wxPoint nextPt; wxPoint nextPt;
bool first = true; bool first = true;
bool reverse = false; bool reverse = false;
@ -537,6 +558,7 @@ bool ConvertOutlineToPolygon( std::vector<PCB_SHAPE*>& aSegList, SHAPE_POLY_SET&
if( nextGraphic && !( nextGraphic->GetFlags() & SKIP_STRUCT ) ) if( nextGraphic && !( nextGraphic->GetFlags() & SKIP_STRUCT ) )
{ {
prevGraphic = graphic;
graphic = nextGraphic; graphic = nextGraphic;
graphic->SetFlags( SKIP_STRUCT ); graphic->SetFlags( SKIP_STRUCT );
startCandidates.erase( graphic ); startCandidates.erase( graphic );
@ -815,7 +837,6 @@ bool ConvertOutlineToPolygon( std::vector<PCB_SHAPE*>& aSegList, SHAPE_POLY_SET&
// All of the silliness that follows is to work around the segment iterator while checking // All of the silliness that follows is to work around the segment iterator while checking
// for collisions. // for collisions.
// TODO: Implement proper segment and point iterators that follow std // TODO: Implement proper segment and point iterators that follow std
bool selfIntersecting = false;
for( auto seg1 = aPolygons.IterateSegmentsWithHoles(); seg1; seg1++ ) for( auto seg1 = aPolygons.IterateSegmentsWithHoles(); seg1; seg1++ )
{ {