]> granicus.if.org Git - graphviz/commitdiff
mingle: remove unused 'total_wgt' in 'agglomerative_ink_bundling_internal'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 13 May 2022 15:04:07 +0000 (08:04 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 13 May 2022 15:04:07 +0000 (08:04 -0700)
The CMake macOS CI job has begun erroring out:

  …/lib/mingle/agglomerative_bundling.cpp:400:27: error: variable 'total_wgt'
    set but not used [-Werror,-Wunused-but-set-variable]
  double eps = 0., wgt, total_wgt = 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/mingle/agglomerative_bundling.cpp

index 0d285dfaa88b7cf99bc27e0b2b93a0f5360d3a25..213bb118ae10d0d1fae6d44540654f8a599dfc8a 100644 (file)
@@ -397,7 +397,7 @@ static pedge* agglomerative_ink_bundling_internal(int dim, SparseMatrix A, pedge
     pedge *mid_edges, midedge;/* middle section of edges that will be bundled again */
     int ne, npp, l;
     SparseMatrix A_mid;
-    double eps = 0., wgt, total_wgt = 0;
+    double eps = 0., wgt;
 
    /* make new edges using meet1 and meet2.
        
@@ -417,7 +417,6 @@ static pedge* agglomerative_ink_bundling_internal(int dim, SparseMatrix A, pedge
       pick = &(ja[ia[i]]);
       wgt = 0.;
       for (j = ia[i]; j < ia[i+1]; j++) wgt += edges[j]->wgt;
-      total_wgt += wgt;
       if (DEBUG) if (Verbose) fprintf(stderr,"calling ink3...\n");
       ink1 = ink(edges, ia[i+1]-ia[i], pick, &ink0, &meet1, &meet2, angle_param, angle);
       if (DEBUG) if (Verbose) fprintf(stderr,"done calling ink3...\n");