minmaskgap action doesn't work on vias with zero clearance

[Peter Clifton]

See summary...

In order for the "mismaskgap" action to operate, the via you're working on must already have non-zero solder mask clearance.

[Andrew Poelstra (asp11)]

This is a deliberate check that is not present for pads or pins:

action.c: 4328: if (via->Mask && via->Mask < via->Thickness + value)

The fix would be to simply remove the "via->Mask &&", but why was that there in the first place?

[DJ Delorie (djdelorie)]

Because tented vias are very common, and to untent them all would be the less expected action.

The purpose of MinMaskGap() is to ensure that *existing* gaps meet the minimum, to pass DRC at the fab.
If you want to *create* a gap, there are other commands for doing that on vias.

[Traumflug (mah-jump-ing)]

Marking this as 'won't fix', because it matches expected behaviour.