]> granicus.if.org Git - graphviz/commitdiff
edgepaint: remove unused 'nnodes' in 'node_distinct_coloring'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 13 May 2022 13:49:46 +0000 (06:49 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 13 May 2022 14:04:18 +0000 (07:04 -0700)
The CMake macOS CI job has begun erroring out:

  …/lib/edgepaint/node_distinct_coloring.c:204:17: error: variable 'nnodes'
    set but not used [-Werror,-Wunused-but-set-variable]
  int i, j, jj, nnodes = 0;
                ^

While the error is accurate, nothing has changed on our side that would have
caused this. I can only assume Gitlab have upgraded the macOS runners to pull in
a new version of Clang.

lib/edgepaint/node_distinct_coloring.c

index 498fc5ecf2972332d4d566937374aaa48c293a54..1ebda2fe281734bf25bcbd3453be926881ff848b 100644 (file)
@@ -201,7 +201,7 @@ int node_distinct_coloring(char *color_scheme, char *lightness, bool weightedQ,
   int ncomps, *comps = NULL, *comps_ptr = NULL;
   int nn, n;
   double *ctmp;
-  int i, j, jj, nnodes = 0;
+  int i, j, jj;
   QuadTree qt = NULL;
   int cdim;
   int scheme = COLOR_LAB;
@@ -261,9 +261,6 @@ int node_distinct_coloring(char *color_scheme, char *lightness, bool weightedQ,
     nn = comps_ptr[i+1] - comps_ptr[i];
     B = SparseMatrix_get_submatrix(A, nn, nn, &(comps[comps_ptr[i]]), &(comps[comps_ptr[i]]));
     node_distinct_coloring_internal(scheme, qt, weightedQ, B, cdim, accuracy, iter_max, seed, ctmp);
-    if (B->m > 2) {
-      nnodes += B->m;
-    }
 
     for (j = comps_ptr[i]; j < comps_ptr[i+1]; j++){
       jj = j - comps_ptr[i];