From 0f93752f14dc26d7918f3f83e22abc278f73d950 Mon Sep 17 00:00:00 2001 From: jean-pierre charras Date: Thu, 24 Aug 2023 18:47:11 +0200 Subject: [PATCH] Revert "Record hole knockouts in zone filler earlier" This reverts commit 1ccaf45740f620e7a59519160f5f727003c72f66. Breaks sone thermal shapes. Not needed after Clipper2 update. --- pcbnew/zone_filler.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/pcbnew/zone_filler.cpp b/pcbnew/zone_filler.cpp index ff3a67e038..ee54dcc56a 100644 --- a/pcbnew/zone_filler.cpp +++ b/pcbnew/zone_filler.cpp @@ -1449,10 +1449,6 @@ bool ZONE_FILLER::fillCopperZone( const ZONE* aZone, PCB_LAYER_ID aLayer, PCB_LA */ buildCopperItemClearances( aZone, aLayer, noConnectionPads, clearanceHoles ); - - for( PAD* pad : thermalConnectionPads ) - addHoleKnockout( pad, 0, clearanceHoles ); - DUMP_POLYS_TO_COPPER_LAYER( clearanceHoles, In3_Cu, wxT( "clearance-holes" ) ); if( m_progressReporter && m_progressReporter->IsCancelled() ) @@ -1594,6 +1590,9 @@ bool ZONE_FILLER::fillCopperZone( const ZONE* aZone, PCB_LAYER_ID aLayer, PCB_LA * islands */ + for( PAD* pad : thermalConnectionPads ) + addHoleKnockout( pad, 0, clearanceHoles ); + aFillPolys.BooleanIntersection( aMaxExtents, SHAPE_POLY_SET::PM_FAST ); DUMP_POLYS_TO_COPPER_LAYER( aFillPolys, In16_Cu, wxT( "after-trim-to-outline" ) ); aFillPolys.BooleanSubtract( clearanceHoles, SHAPE_POLY_SET::PM_FAST );