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

pad.cpp: fix incorrect layer mask in PAD::SetAttribute() for aperture pad 

A aperture pad is not on copper layer, so do not force a copper layer for this pad
This commit is contained in:
jean-pierre charras 2024-04-21 11:48:01 +02:00
parent 6540c3ec9f
commit 03e2b3c42e
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
pcbnew/pad.cpp
View File

@ -824,7 +824,11 @@ void PAD::SetAttribute( PAD_ATTRIB aAttribute )
else
{
m_layerMask &= ~LSET::AllCuMask();
m_layerMask.set( B_Cu );
// A SMD can be a "aperture pad", with no copper layer.
// So ensure we can set the bottom copper layer
if( !IsAperturePad() )
m_layerMask.set( B_Cu );
}
m_drill = VECTOR2I( 0, 0 );
@ -1679,8 +1683,10 @@ const BOX2I PAD::ViewBBox() const
void PAD::ImportSettingsFrom( const PAD& aMasterPad )
{
SetShape( aMasterPad.GetShape() );
SetLayerSet( aMasterPad.GetLayerSet() );
SetAttribute( aMasterPad.GetAttribute() );
// SetAttribute() can change m_layerMask. be sure we keep the original mask
// by calling SetLayerSet() after SetAttribute()
SetLayerSet( aMasterPad.GetLayerSet() );
SetProperty( aMasterPad.GetProperty() );
// Must be after setting attribute and layerSet