On Fri, Oct 10, 2008 at 01:36:59AM +0100, Peter Clifton wrote: > On Thu, 2008-10-09 at 17:14 -0700, Ben Jackson wrote: > > > > > Is there any reason why the existing code picked out and worked with > > > rectangular polygons only? > > > > If you look up in GatherSubnets you'll see the initial connection lists > > only add rectangular polygons. > > > If you are getting via-rats over arbitrary polygons with your changes > > I'd like to know how it's working! > > See "IsPointInPolyIgnoreHoles()". It looks at the first piece of a > clipped poly (the visible part, as used for connectivity), and just test > for a hit within its outer contour, ignoring any inner ones.
Yes, but that test is equivalent to the IsPointInBox except in the limited case of rectangles which have parts cut off. So the enhancement avoids via-rats over areas that are missing from the outer contour, but not holes. It does not ADD via rats for arbitrary polygons. If you want it to work for arbitrary polygons you'll have to include them in the GatherSubnets code. -- Ben Jackson AD7GD <[EMAIL PROTECTED]> http://www.ben.com/ _______________________________________________ geda-dev mailing list [email protected] http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev
