From 63923bcbe82dfc966abd057ae05eaba14e951404 Mon Sep 17 00:00:00 2001 From: Matthew Fernandez Date: Fri, 13 May 2022 06:49:46 -0700 Subject: [PATCH] edgepaint: remove unused 'nnodes' in 'node_distinct_coloring' MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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 | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/lib/edgepaint/node_distinct_coloring.c b/lib/edgepaint/node_distinct_coloring.c index 498fc5ecf..1ebda2fe2 100644 --- a/lib/edgepaint/node_distinct_coloring.c +++ b/lib/edgepaint/node_distinct_coloring.c @@ -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]; -- 2.40.0