realAscot/kicad-source
1
0
Fork 0
mirror of https://gitlab.com/kicad/code/kicad.git synced 2025-09-14 02:03:12 +02:00
Code Issues Packages Projects Releases Wiki Activity

Eeschema: order pins by number, not position

Browse Source 
Also use StrNumCmp, then the pin numbers are ordered exactly the
same way as pads in footprints.
This commit is contained in:
John Beard 2025-09-01 20:46:59 +08:00
parent 262f1fdabb
commit 5d967289ee
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
eeschema/sch_pin.cpp
View File

@ -1456,8 +1456,10 @@ wxString SCH_PIN::getItemDescription( ALT* aAlt ) const
int SCH_PIN::compare( const SCH_ITEM& aOther, int aCompareFlags ) const
{
// Ignore the UUID here.
int retv = SCH_ITEM::compare( aOther, aCompareFlags | SCH_ITEM::COMPARE_FLAGS::EQUALITY );
// Ignore the UUID here
// And the position, which we'll do after the number.
int retv = SCH_ITEM::compare( aOther, aCompareFlags | SCH_ITEM::COMPARE_FLAGS::EQUALITY
| SCH_ITEM::COMPARE_FLAGS::SKIP_TST_POS );
if( retv )
return retv;
@ -1467,7 +1469,10 @@ int SCH_PIN::compare( const SCH_ITEM& aOther, int aCompareFlags ) const
wxCHECK( tmp, -1 );
if( m_number != tmp->m_number )
return m_number.Cmp( tmp->m_number );
{
// StrNumCmp: sort the same as the pads in the footprint file
return StrNumCmp( m_number, tmp->m_number ) < 0;
}
if( m_position.x != tmp->m_position.x )
return m_position.x - tmp->m_position.x;