From 72794f1416da4fb92c556faf0a87e0b1ae536916 Mon Sep 17 00:00:00 2001 From: Seth Hillbrand Date: Mon, 20 May 2019 06:00:58 -0700 Subject: [PATCH] eagle: Use the "first" tag instead of pin num Eagle parsing allows pin 1 to be different from the other pins in the package but this is set by a tag marking the pin as "first" rather than the actual number of the pin. Fixes: lp:1829251 * https://bugs.launchpad.net/kicad/+bug/1829251 (cherry picked from commit dbc40096db46c32ffe8f73377e831c6ca08ceab7) --- pcbnew/eagle_plugin.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pcbnew/eagle_plugin.cpp b/pcbnew/eagle_plugin.cpp index 56ff347f4d..2f509daa99 100644 --- a/pcbnew/eagle_plugin.cpp +++ b/pcbnew/eagle_plugin.cpp @@ -1471,7 +1471,7 @@ void EAGLE_PLUGIN::packagePad( MODULE* aModule, wxXmlNode* aTree ) const aModule->PadsList().PushBack( pad ); transferPad( e, pad ); - if( pad->GetName() == wxT( "1" ) && m_rules->psFirst != EPAD::UNDEF ) + if( e.first && *e.first && m_rules->psFirst != EPAD::UNDEF ) shape = m_rules->psFirst; else if( aModule->GetLayer() == F_Cu && m_rules->psTop != EPAD::UNDEF ) shape = m_rules->psTop;