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

Sigh. But addCorner() *does* need mapCoords(). 

And it's got an off-by-one error in it too.  I don't think this
ever worked at all.

Fixes https://gitlab.com/kicad/code/kicad/issues/9926
This commit is contained in:
Jeff Young 2021-12-11 00:38:50 +00:00
parent 1f126c5619
commit 15d1abeeef
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
eeschema/tools/ee_point_editor.cpp
View File

@ -854,13 +854,14 @@ int EE_POINT_EDITOR::addCorner( const TOOL_EVENT& aEvent )
SHAPE_LINE_CHAIN& poly = shape->GetPolyShape().Outline( 0 );
VECTOR2I cursor = getViewControls()->GetCursorPosition( !aEvent.DisableGridSnapping() );
wxPoint pos = mapCoords( cursor );
int currentMinDistance = INT_MAX;
int closestLineStart = 0;
for( unsigned i = 0; i < poly.GetPointCount() - 1; ++i )
{
int distance = (int) DistanceLinePoint( (wxPoint) poly.CPoint( i ),
(wxPoint) poly.CPoint( i + 1 ), (wxPoint) cursor );
(wxPoint) poly.CPoint( i + 1 ), pos );
if( distance < currentMinDistance )
{
@ -870,7 +871,7 @@ int EE_POINT_EDITOR::addCorner( const TOOL_EVENT& aEvent )
}
saveItemsToUndo();
poly.Insert( closestLineStart, (wxPoint) cursor );
poly.Insert( closestLineStart + 1, pos );
updateItem( shape, true );
updatePoints();