]> granicus.if.org Git - graphviz/commitdiff
sparse QuadTree_get_nearest: remove 'flag' parameter
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 30 Jul 2022 00:52:12 +0000 (17:52 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 4 Aug 2022 01:10:28 +0000 (18:10 -0700)
This is always set to 0 and conveys no information to the caller.

cmd/gvmap/make_map.c
lib/edgepaint/node_distinct_coloring.c
lib/sparse/QuadTree.c
lib/sparse/QuadTree.h

index dec68d7e58f08301625690cdf0b73ca6b3037811..20d5de43b01457a283aad57a462107b3afe64173 100644 (file)
@@ -1181,8 +1181,7 @@ static int make_map_internal(int exclude_random, int include_OK_points,
        point[j] = xmin[j] + (xmax[j] - xmin[j])*drand();
       }
       
-      QuadTree_get_nearest(qt, point, ymin, &imin, &min, flag);
-      assert(!(*flag));
+      QuadTree_get_nearest(qt, point, ymin, &imin, &min);
 
       if (min > shore_depth_tol){/* point not too close, accepted */
        for (j = 0; j < dim2; j++){
index 64e9d1ca05dd9bb944b273022206de3d82763ea9..a4da5510a415642efbf713a40bf501b33ca79563 100644 (file)
@@ -37,7 +37,7 @@ static void node_distinct_coloring_internal2(int scheme, QuadTree qt,
   static const int iter_max = 100;
   double cspace_size = 0.7;
   double red[3], black[3], min;
-  int flag = 0, imin;
+  int imin;
   double *wgt = NULL;
 
   assert(accuracy > 0);
@@ -54,8 +54,7 @@ static void node_distinct_coloring_internal2(int scheme, QuadTree qt,
   if (n == 1){
     if (scheme == COLOR_LAB){
       assert(qt);
-      QuadTree_get_nearest(qt, black, colors, &imin, &min, &flag);
-      assert(!flag);
+      QuadTree_get_nearest(qt, black, colors, &imin, &min);
       LAB2RGB_real_01(colors);
       *color_diff0 = 1000; *color_diff_sum0 = 1000;
     } else {
@@ -66,12 +65,10 @@ static void node_distinct_coloring_internal2(int scheme, QuadTree qt,
   } else if (n == 2){
     if (scheme == COLOR_LAB){
       assert(qt);
-      QuadTree_get_nearest(qt, black, colors, &imin, &min, &flag);
-      assert(!flag);
+      QuadTree_get_nearest(qt, black, colors, &imin, &min);
       LAB2RGB_real_01(colors);
 
-      QuadTree_get_nearest(qt, red, colors+cdim, &imin, &min, &flag);
-      assert(!flag);
+      QuadTree_get_nearest(qt, red, colors+cdim, &imin, &min);
       LAB2RGB_real_01(colors+cdim);
       *color_diff0 = 1000; *color_diff_sum0 = 1000;
 
index c2d6f1c211598aad8e9a7a20aa7d1190dbfe15d6..1e5fabb3931234deb8e5590bd32cecf5ef2664bd 100644 (file)
@@ -713,9 +713,8 @@ static void QuadTree_get_nearest_internal(QuadTree qt, double *x, double *y, dou
 }
 
 
-void QuadTree_get_nearest(QuadTree qt, double *x, double *ymin, int *imin, double *min, int *flag){
+void QuadTree_get_nearest(QuadTree qt, double *x, double *ymin, int *imin, double *min){
 
-  *flag = 0;
   *min = -1;
 
   QuadTree_get_nearest_internal(qt, x, ymin, min, imin, TRUE);
index 2c2f684ec3201650f64a906d685da1700b8edf4e..6e0c7d8fc7f3b1bfe1c35f95e3682546f21b0619 100644 (file)
@@ -56,7 +56,7 @@ void QuadTree_get_supernodes(QuadTree qt, double bh, double *point, int nodeid,
 void QuadTree_get_repulsive_force(QuadTree qt, double *force, double *x, double bh, double p, double KP, double *counts, int *flag);
 
 /* find the nearest point and put in ymin, index in imin and distance in min */
-void QuadTree_get_nearest(QuadTree qt, double *x, double *ymin, int *imin, double *min, int *flag);
+void QuadTree_get_nearest(QuadTree qt, double *x, double *ymin, int *imin, double *min);
 
 QuadTree QuadTree_new_in_quadrant(int dim, double *center, double width, int max_level, int i);