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

Fix issue with DRC for through hole being applied to SMD pads 

Commit d7272b7f5e fixed the use of
the eagle DRC to change the library pad shapes. This was also
applied incorrectly to SMD stacks.

Remove the references to m_rules in packageSMD() and the code
that deals with them. eagle 7.7 does not have any SMD pad options
other than rectangle with rounding.

(cherry picked from commit 070c6b8ce0)
This commit is contained in:
Ben Dooks 2020-01-24 17:28:09 +00:00 committed by Seth Hillbrand
parent 83c3a78e6d
commit fc1f713106
1 changed files with 2 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
pcbnew/eagle_plugin.cpp
View File

@ -1909,36 +1909,11 @@ void EAGLE_PLUGIN::packageSMD( MODULE* aModule, wxXmlNode* aTree ) const
if( !IsCopperLayer( layer ) )
return;
bool shape_set = false;
int shape = EPAD::UNDEF;
D_PAD* pad = new D_PAD( aModule );
aModule->PadsList().PushBack( pad );
transferPad( e, pad );
if( pad->GetName() == wxT( "1" ) && m_rules->psFirst != EPAD::UNDEF )
shape = m_rules->psFirst;
else if( layer == F_Cu && m_rules->psTop != EPAD::UNDEF )
shape = m_rules->psTop;
else if( layer == B_Cu && m_rules->psBottom != EPAD::UNDEF )
shape = m_rules->psBottom;
switch( shape )
{
case EPAD::ROUND:
case EPAD::OCTAGON:
shape_set = true;
pad->SetShape( PAD_SHAPE_CIRCLE );
break;
case EPAD::SQUARE:
shape_set = true;
pad->SetShape( PAD_SHAPE_RECT );
break;
default:
pad->SetShape( PAD_SHAPE_RECT );
}
pad->SetAttribute( PAD_ATTRIB_SMD );
wxSize padSize( e.dx.ToPcbUnits(), e.dy.ToPcbUnits() );
@ -1959,7 +1934,7 @@ void EAGLE_PLUGIN::packageSMD( MODULE* aModule, wxXmlNode* aTree ) const
int roundRadius = eagleClamp( m_rules->srMinRoundness * 2,
(int)( minPadSize * m_rules->srRoundness ), m_rules->srMaxRoundness * 2 );
if( !shape_set && ( e.roundness || roundRadius > 0 ) )
if( e.roundness || roundRadius > 0 )
{
double roundRatio = (double) roundRadius / minPadSize / 2.0;