Use router's connectivity algorithm, not CONNECTIVITY_DATA's.

Fixes https://gitlab.com/kicad/code/kicad/issues/10745
This commit is contained in:
Jeff Young 2022-09-16 22:48:33 +01:00
parent f10fdfc7d7
commit 107f409106
1 changed files with 21 additions and 35 deletions

View File

@ -319,49 +319,35 @@ int EDIT_TOOL::Drag( const TOOL_EVENT& aEvent )
for( EDA_ITEM* item : aCollector ) for( EDA_ITEM* item : aCollector )
{ {
switch( item->Type() ) if( PCB_TRACK* track = dynamic_cast<PCB_TRACK*>( item ) )
{ {
case PCB_TRACE_T: if( track->Type() == PCB_VIA_T )
case PCB_ARC_T: vias.push_back( track );
tracks.push_back( static_cast<PCB_TRACK*>( item ) ); else
break; tracks.push_back( track );
case PCB_VIA_T:
vias.push_back( static_cast<PCB_TRACK*>( item ) );
break;
default:
break;
} }
} }
if( tracks.size() == 2 ) auto connected = []( PCB_TRACK* track, const VECTOR2I& pt )
{
return track->GetStart() == pt || track->GetEnd() == pt;
};
if( tracks.size() == 2 && vias.size() == 0 )
{ {
const BOARD* board = tracks[0]->GetBoard(); if( connected( tracks[0], tracks[1]->GetStart() )
std::shared_ptr<CONNECTIVITY_DATA> c = board->GetConnectivity(); || connected( tracks[0], tracks[1]->GetEnd() ) )
std::vector<BOARD_CONNECTED_ITEM*> cItems;
int accuracy = KiROUND( 5 * aCollector.GetGuide()->OnePixelInIU() );
if( vias.size() == 1 )
{ {
cItems = c->GetConnectedItemsAtAnchor( vias[0], aPt, aCollector.Remove( tracks[1] );
{ PCB_TRACE_T, PCB_ARC_T },
vias[0]->GetWidth() / 2 + accuracy );
if( alg::contains( cItems, tracks[0] )
&& alg::contains( cItems, tracks[1] ) )
{
aCollector.Remove( tracks[0] );
aCollector.Remove( tracks[1] );
}
} }
else if( vias.size() == 0 ) }
else if( tracks.size() == 2 && vias.size() == 1 )
{
if( connected( tracks[0], vias[0]->GetPosition() )
&& connected( tracks[1], vias[0]->GetPosition() ) )
{ {
cItems = c->GetConnectedItemsAtAnchor( tracks[0], aPt, aCollector.Remove( tracks[0] );
{ PCB_TRACE_T, PCB_ARC_T }, aCollector.Remove( tracks[1] );
tracks[0]->GetWidth() / 2 + accuracy );
if( alg::contains( cItems, tracks[1] ) )
aCollector.Remove( tracks[1] );
} }
} }
}, },