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

Fixed bug in FlipLayerMask 

- Offset for internal copper layers was incorrect
- Incorrect mask was used
This commit is contained in:
Oliver Walters 2017-09-23 15:20:20 +10:00 committed by jean-pierre charras
parent 7e0fa329d0
commit 363fc598c0
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
common/lset.cpp
View File

@ -554,23 +554,28 @@ LSET FlipLayerMask( LSET aMask, int aCopperLayersCount )
if( aCopperLayersCount >= 4 ) // Internal layers exist if( aCopperLayersCount >= 4 ) // Internal layers exist
{ {
LSET internalMask = aMask & ~LSET::InternalCuMask(); LSET internalMask = aMask & LSET::InternalCuMask();
if( internalMask != LSET::InternalCuMask() ) if( internalMask != LSET::InternalCuMask() )
{ // the mask does not include all internal layers. Therefore {
// the mask does not include all internal layers. Therefore
// the flipped mask for internal copper layers must be built // the flipped mask for internal copper layers must be built
int innerLayerCnt = aCopperLayersCount -2; int innerLayerCnt = aCopperLayersCount -2;
for( int ii = 0; ii < innerLayerCnt; ii++ ) for( int ii = 0; ii < innerLayerCnt; ii++ )
{ {
//TODO there is a problem with this code if( internalMask[innerLayerCnt - ii] )
if( internalMask[innerLayerCnt - ii + In1_Cu] ) {
newMask.set( ii + In1_Cu ); newMask.set( ii + In1_Cu );
}
else else
{
newMask.reset( ii + In1_Cu ); newMask.reset( ii + In1_Cu );
} }
} }
} }
}
return newMask; return newMask;
} }