On Mon, 2008-12-29 at 01:39 -0500, DJ Delorie wrote: > > First, please don't say "dicer" when you don't mean "dicer". The > > polygon dicer is the code that cuts a polygon with holes (such as > > any typical plane) up into sub-parts which have no holes. These are > > used to render to certain devices. > > The dicer turns a polygon-with-holes into N polygons-without-holes, > right? If so, that's what I meant. After dicing, none of the > remnants actually intersect the pads any more. The existing code > checks to see if the pad intersects the remnants, not if the pad > intersects the undiced polygon-with-holes.
Not quite correct. The original "polygons" actually have holes in them, so don't technically intersect with the pads. The Polygon consists of a series of contours. The one you're thinking of is the outer contour. There are also inner contours, which define the holes cut from the polygon by the pads. I believe I may have corrected this bug more formally in the work I did on flooded fills with island removal. (I certainly recall having to fix the connectivity check between a pad and a polygon. Let me look and see if I can find the right patch.). -- Peter Clifton Electrical Engineering Division, Engineering Department, University of Cambridge, 9, JJ Thomson Avenue, Cambridge CB3 0FA Tel: +44 (0)7729 980173 - (No signal in the lab!) _______________________________________________ geda-dev mailing list [email protected] http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev
