Just to clarify... reading the code in question (not just the patch) makes me want to tear my eyes out.
I appreciate that the patch brings the via case in line with the pin case, but two wrongs do not make a right here...
The "magic" behaviour of setting "0" to mean "oh - put it at the width of the pin", is evil, and should be removed from both cases.
Just to clarify... reading the code in question (not just the patch) makes me want to tear my eyes out.
I appreciate that the patch brings the via case in line with the pin case, but two wrongs do not make a right here...
The "magic" behaviour of setting "0" to mean "oh - put it at the width of the pin", is evil, and should be removed from both cases.