]> granicus.if.org Git - graphviz/commitdiff
topfish: remove unmutated 'cur_level'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 6 Mar 2022 03:26:00 +0000 (19:26 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 9 Mar 2022 06:04:37 +0000 (22:04 -0800)
This file has a fairly strange structure where almost every use of `cur_level`
is actually referencing a local that shadows the global `cur_level`. The global
`cur_level` is never written to and remains 0 throughout execution, making it
somewhat useless.

lib/topfish/hierarchy.c

index 896b2ae0c6ab31759eda07adec72815559831831..771ffb34dc5af24cd69c99771900af5c1f2b0ba5 100644 (file)
@@ -26,8 +26,6 @@
 #include <common/arith.h>
 #include <topfish/hierarchy.h>
 
-static int cur_level = 0;
-
 /////////////////////////
 // Some utilities for  //
 // 'maxmatch(..)'      //
@@ -913,7 +911,7 @@ set_active_levels(Hierarchy * hierarchy, int *foci_nodes, int num_foci,
     ex_vtx_data *cgraph;
     int *cv2v;
     int v, u;
-    int min_level = cur_level;
+    int min_level = 0;
 
     graph = hierarchy->geom_graphs[min_level]; // finest graph
     n = hierarchy->nvtxs[min_level];
@@ -1086,7 +1084,7 @@ find_closest_active_node(Hierarchy * hierarchy, double x, double y,
        {
                min_dist = findClosestActiveNode(hierarchy, i, top_level, x, y,min_dist, &closest_node, &closest_node_level);
     }
-    *closest_fine_node =find_leftmost_descendant(hierarchy, closest_node,closest_node_level, cur_level);
+    *closest_fine_node =find_leftmost_descendant(hierarchy, closest_node,closest_node_level, 0);
 
     return min_dist;
 }