In general, this seems a good move to me. Replacing global parameters for function local arguments makes code much clearer. However, this change may break things (I think this is what Peter was saying)
We should revise every use of IsLineInPolygon (and respective Arc/Polygon functions). Maybe one of the affected places would be DRCFind(), when it does:
Chad,
In general, this seems a good move to me. Replacing global parameters for function local arguments makes code much clearer. However, this change may break things (I think this is what Peter was saying)
@ IsLineInPolygon (LineType *Line, PolygonType *Polygon)
- if (!(lp = LinePoly (Line, Line->Thickness + Bloat)))
+ if (!(lp = LinePoly (Line, Line->Thickness + 2*bloat)))
We should revise every use of IsLineInPolygon (and respective Arc/Polygon functions). Maybe one of the affected places would be DRCFind(), when it does:
Bloat = -PCB->Shrink;