]> granicus.if.org Git - graphviz/commitdiff
remove unnecessary do_swapping parameter to country_graph_coloring_internal
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 23 Jun 2021 00:44:47 +0000 (17:44 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 25 Jun 2021 14:39:33 +0000 (07:39 -0700)
This function is only ever called with do_swapping == DO_SWAPPING.

cmd/gvmap/country_graph_coloring.c
cmd/gvmap/country_graph_coloring.h

index b6b798fcc08f0405993d49792a48389a97bd1baa..e7e6e895bcde404cd62bb361349ca5c5e96ab6f5 100644 (file)
@@ -271,7 +271,7 @@ void improve_antibandwidth_by_swapping(SparseMatrix A, int *p){
   }
 }
   
-static void country_graph_coloring_internal(int seed, SparseMatrix A, int **p, real *norm_1, int do_swapping){
+static void country_graph_coloring_internal(int seed, SparseMatrix A, int **p, real *norm_1){
   int n = A->m, i, j, jj;
   SparseMatrix L, A2;
   int *ia = A->ia, *ja = A->ja;
@@ -317,15 +317,7 @@ static void country_graph_coloring_internal(int seed, SparseMatrix A, int **p, r
 
   start2 = clock();
   /* swapping */
-  if (do_swapping) {
-    if (do_swapping == DO_SWAPPING){
-      improve_antibandwidth_by_swapping(A2, *p);
-    } else if (do_swapping == DO_SWAPPING_CHEAP) {
-      improve_antibandwidth_by_swapping_cheap(A2, *p);
-    } else {
-      assert(0);
-    }
-  }
+  improve_antibandwidth_by_swapping(A2, *p);
   if (Verbose) {
     fprintf(stderr, "cpu time for greedy refinement = %f\n",
             ((double)(clock() - start2)) / CLOCKS_PER_SEC);
@@ -341,5 +333,5 @@ static void country_graph_coloring_internal(int seed, SparseMatrix A, int **p, r
   SparseMatrix_delete(L);
 }
 void country_graph_coloring(int seed, SparseMatrix A, int **p, real *norm_1){
-  country_graph_coloring_internal(seed, A, p, norm_1, DO_SWAPPING);
+  country_graph_coloring_internal(seed, A, p, norm_1);
 }
index 578b43955f93e54c3fbac8e62064d09957683716..9d8a0d476098ab4f0a8bc3dae1dc8dbb773438b7 100644 (file)
@@ -13,8 +13,6 @@
 
 #include <sparse/SparseMatrix.h>
 
-enum {DO_SWAPPING = 1, DO_SWAPPING_CHEAP};
-
 void country_graph_coloring(int seed, SparseMatrix A, int **p, real *norm_1);
 
 void improve_antibandwidth_by_swapping(SparseMatrix A, int *p);