gEDA-cvs: pcb.git: branch: master updated (207ae38342197df946880fcbf9fdd35b9a89fa4e)

[git@xxxxxxxxxx]

The branch, master has been updated
       via  207ae38342197df946880fcbf9fdd35b9a89fa4e (commit)
      from  2deda48c6b9fad996877d708a3f165f2eedd53d3 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


=========
 Summary
=========

 src/polygon.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)


=================
 Commit Messages
=================

commit 207ae38342197df946880fcbf9fdd35b9a89fa4e
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Commit: Peter Clifton <pcjc2@xxxxxxxxx>

    Allow PolygonHoles() to be passed a NULL region
    
    Passing a NULL region implies we wish to be called back for all
    holes in the polygon, as we don't have a region to test against.
    
    Fixes crash in the DRC window when the design has polygons, and
    is viewed with either thindraw, or thindraw polygons.

:100644 100644 bc7b555... d5df073... M	src/polygon.c

=========
 Changes
=========

commit 207ae38342197df946880fcbf9fdd35b9a89fa4e
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Commit: Peter Clifton <pcjc2@xxxxxxxxx>

    Allow PolygonHoles() to be passed a NULL region
    
    Passing a NULL region implies we wish to be called back for all
    holes in the polygon, as we don't have a region to test against.
    
    Fixes crash in the DRC window when the design has polygons, and
    is viewed with either thindraw, or thindraw polygons.

diff --git a/src/polygon.c b/src/polygon.c
index bc7b555..d5df073 100644
--- a/src/polygon.c
+++ b/src/polygon.c
@@ -1256,8 +1256,9 @@ PolygonHoles (PolygonType *polygon, const BoxType *range,
     return 0;
   for (pl = pa->contours->next; pl; pl = pl->next)
     {
-      if (pl->xmin > range->X2 || pl->xmax < range->X1 ||
-          pl->ymin > range->Y2 || pl->ymax < range->Y1)
+      if (range != NULL &&
+          (pl->xmin > range->X2 || pl->xmax < range->X1 ||
+           pl->ymin > range->Y2 || pl->ymax < range->Y1))
         continue;
       if (callback (pl, user_data))
         {




_______________________________________________
geda-cvs mailing list
geda-cvs@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-cvs