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

eeschema: Re-load the RTree when we re-cache components 

The components are reloaded when libraries change and when loading a
schematic.  The initial load does not contain pins and so bbox hits may
miss elements.

Fixes #3800 | https://gitlab.com/kicad/code/kicad/issues/3800
This commit is contained in:
Seth Hillbrand 2020-01-23 06:05:12 -08:00
parent 995bef8736
commit bcc68b2dd9
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
eeschema/sch_screen.cpp
View File

@ -4,7 +4,7 @@
* Copyright (C) 2013 Jean-Pierre Charras, jp.charras at wanadoo.fr
* Copyright (C) 2012 SoftPLC Corporation, Dick Hollenbeck <dick@softplc.com>
* Copyright (C) 2008 Wayne Stambaugh <stambaughw@gmail.com>
* Copyright (C) 1992-2019 KiCad Developers, see AUTHORS.txt for contributors.
* Copyright (C) 1992-2020 KiCad Developers, see AUTHORS.txt for contributors.
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
@ -498,6 +498,8 @@ void SCH_SCREEN::UpdateSymbolLinks( bool aForce )
for( auto aItem : Items().OfType( SCH_COMPONENT_T ) )
cmps.push_back( static_cast<SCH_COMPONENT*>( aItem ) );
for( auto cmp : cmps )
Remove( cmp );
// Must we resolve?
if( (m_modification_sync != mod_hash) || aForce )
@ -513,6 +515,11 @@ void SCH_SCREEN::UpdateSymbolLinks( bool aForce )
for( auto cmp : cmps )
cmp->UpdatePins();
}
// Changing the symbol may adjust the bbox of the symbol. This re-inserts the
// item with the new bbox
for( auto cmp : cmps )
Append( cmp );
}
}