PNS: Line placer returns false when nothing committed
The true/false return should be triggered on whether any elements have been commited to the line Fixes #3969 | https://gitlab.com/kicad/code/kicad/issues/3969
This commit is contained in:
parent
20ca5a6973
commit
18db471967
|
@ -1101,16 +1101,16 @@ bool LINE_PLACER::FixRoute( const VECTOR2I& aP, ITEM* aEndItem, bool aForceFinis
|
|||
|
||||
if( !l.SegmentCount() )
|
||||
{
|
||||
if( pl.EndsWithVia() )
|
||||
{
|
||||
// Nothing to commit if we have an empty line
|
||||
if( !pl.EndsWithVia() )
|
||||
return false;
|
||||
|
||||
m_lastNode->Add( Clone( pl.Via() ) );
|
||||
m_lastNode = NULL;
|
||||
m_currentNode = NULL;
|
||||
|
||||
m_idle = true;
|
||||
m_placementCorrect = true;
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
|
|
|
@ -834,8 +834,10 @@ void ROUTER_TOOL::performRouting()
|
|||
else if( evt->IsAction( &ACT_EndTrack ) )
|
||||
{
|
||||
bool still_routing = true;
|
||||
|
||||
while( still_routing )
|
||||
still_routing = m_router->FixRoute( m_endSnapPoint, m_endItem );
|
||||
|
||||
break;
|
||||
}
|
||||
else if( evt->IsCancelInteractive() || evt->IsActivate()
|
||||
|
|
Loading…
Reference in New Issue