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

pcbnew: Correct layer naming 

Layer bit sets get chosen for the layer name, starting with the copper
layers and then the technicals.  Additionally, multi-layer pads are
appended with an indicator that additional layer bits are set.

Fixes: lp:
* https://bugs.launchpad.net/kicad/+bug/
This commit is contained in:
Seth Hillbrand 2018-12-19 20:57:14 -08:00
parent 8f7c073514
commit 135ff314ca
1 changed files with 17 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
pcbnew/class_pad.cpp
View File

@ -1369,26 +1369,29 @@ wxString LayerMaskDescribe( const BOARD *aBoard, LSET aMask )
{
// Try to be smart and useful. Check all copper first.
if( aMask[F_Cu] && aMask[B_Cu] )
return wxT( "all copper layers" );
return _( "All copper layers" );
// Check for single copper.
LSEQ cu = aBoard->GetEnabledLayers().CuStack();
// Check for copper.
auto layer = aBoard->GetEnabledLayers().AllCuMask() & aMask;
if( cu )
return aBoard->GetLayerName( *cu );
// No copper; check for single techincal.
LSEQ tech = aBoard->GetEnabledLayers().Technicals();
if( tech )
for( int i = 0; i < 2; i++ )
{
wxString layerInfo = aBoard->GetLayerName( *tech );
for( int bit = PCBNEW_LAYER_ID_START; bit < PCB_LAYER_ID_COUNT; ++bit )
{
if( layer[ bit ] )
{
wxString layerInfo = aBoard->GetLayerName( static_cast<PCB_LAYER_ID>( bit ) );
if( tech.size() > 1 )
layerInfo << _( " & others" );
if( aMask.count() > 1 )
layerInfo << _( " and others" );
return layerInfo;
}
}
// No copper; check for technicals.
layer = aBoard->GetEnabledLayers().AllTechMask() & aMask;
}
// No copper, no technicals: no layer
return _( "no layers" );