From 31bb5a2b68c0e51fdd4d3e4238392d81f54249f9 Mon Sep 17 00:00:00 2001 From: Robert Drehmel Date: Mon, 4 Feb 2013 07:27:04 +0100 Subject: [PATCH 1/1] The special case ChangeClearSize(Selected, 0mm) is broken when used to change a via's solder mask clearance. It does work as expected for pins. A patch is found attached which makes the idiomatic sequence to easily provide the minimum solder mask clearance work again: - selecting all pins and vias - ChangeClearSize(Selected, 0mm) - ChangeClearSize(Selected, +0.15mm) Signed-off-by: Bert Timmerman --- src/change.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/src/change.c b/src/change.c index 7e95672..db8999c 100644 --- a/src/change.c +++ b/src/change.c @@ -2470,6 +2470,12 @@ ChangeViaMaskSize (PinType *Via) value = (Absolute) ? Absolute : Via->Mask + Delta; value = MAX (value, 0); + if (value == Via->Mask && Absolute == 0) { + if (TEST_FLAG(HOLEFLAG, Via)) + value = Via->DrillingHole; + else + value = Via->Thickness; + } if (value != Via->Mask) { AddObjectToMaskSizeUndoList (VIA_TYPE, Via, Via, Via); -- 1.7.3.4