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

Avoid case where whole line is new head 

If the whole line shifts, we may have no points of the old tail in the
new line.  In this case, we may not only not find the split point but we
may also have fewer points in the new line than were in the old head.
Clamping to the maximum potential point count prevents creating invalid
lines

Fixes https://gitlab.com/kicad/code/kicad/-/issues/16591
This commit is contained in:
Seth Hillbrand 2024-04-19 17:51:47 -07:00
parent 3435c6ebee
commit 586fddeec1
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pcbnew/router/pns_line_placer.cpp
View File

@ -877,6 +877,10 @@ bool LINE_PLACER::splitHeadTail( const LINE& aNewLine, const LINE& aOldTail, LIN
if( !found )
i--;
// If the old tail doesn't have any points of the new line, we can't split it.
if( i >= l2.PointCount() )
i = l2.PointCount() - 1;
newHead.Clear();
if( i == 0 )