Don't sync the world twice in inline router tools.

Activate() already clears and syncs the world via PNS::TOOL_BASE::Reset,
so there's no need to sync in InlineDrag and InlineBreakTrack.
This commit is contained in:
Alex 2022-09-09 05:06:03 +03:00 committed by Seth Hillbrand
parent 64335d94d7
commit cc4e34ef49
1 changed files with 4 additions and 4 deletions

View File

@ -1838,10 +1838,10 @@ int ROUTER_TOOL::InlineDrag( const TOOL_EVENT& aEvent )
item->SetLocked( false );
}
m_toolMgr->RunAction( PCB_ACTIONS::selectionClear, true );
Activate();
m_toolMgr->RunAction( PCB_ACTIONS::selectionClear, true );
m_router->SyncWorld();
m_startItem = nullptr;
PNS::ITEM* startItem = nullptr;
@ -2120,10 +2120,10 @@ int ROUTER_TOOL::InlineBreakTrack( const TOOL_EVENT& aEvent )
if( item->Type() != PCB_TRACE_T )
return 0;
m_toolMgr->RunAction( PCB_ACTIONS::selectionClear, true );
Activate();
m_toolMgr->RunAction( PCB_ACTIONS::selectionClear, true );
m_router->SyncWorld();
m_startItem = m_router->GetWorld()->FindItemByParent( item );
TOOL_MANAGER* toolManager = frame()->GetToolManager();