Browse Source

Remove nodelist_recompute_all_hsdir_indices() as unused.

Closes 25108.
Nick Mathewson 6 years ago
parent
commit
c2757c3774
3 changed files with 3 additions and 23 deletions
  1. 3 0
      changes/ticket25108
  2. 0 21
      src/or/nodelist.c
  3. 0 2
      src/or/nodelist.h

+ 3 - 0
changes/ticket25108

@@ -0,0 +1,3 @@
+  o Code simplification and refactoring:
+    - Remove the unused nodelist_recompute_all_hsdir_indices(). Closes ticket
+      25108.

+ 0 - 21
src/or/nodelist.c

@@ -373,27 +373,6 @@ node_set_hsdir_index(node_t *node, const networkstatus_t *ns)
   return;
 }
 
-/** Recompute all node hsdir indices. */
-void
-nodelist_recompute_all_hsdir_indices(void)
-{
-  networkstatus_t *consensus;
-  if (!the_nodelist) {
-    return;
-  }
-
-  /* Get a live consensus. Abort if not found */
-  consensus = networkstatus_get_live_consensus(approx_time());
-  if (!consensus) {
-    return;
-  }
-
-  /* Recompute all hsdir indices */
-  SMARTLIST_FOREACH_BEGIN(the_nodelist->nodes, node_t *, node) {
-    node_set_hsdir_index(node, consensus);
-  } SMARTLIST_FOREACH_END(node);
-}
-
 /** Called when a node's address changes. */
 static void
 node_addrs_changed(node_t *node)

+ 0 - 2
src/or/nodelist.h

@@ -35,8 +35,6 @@ void nodelist_remove_routerinfo(routerinfo_t *ri);
 void nodelist_purge(void);
 smartlist_t *nodelist_find_nodes_with_microdesc(const microdesc_t *md);
 
-void nodelist_recompute_all_hsdir_indices(void);
-
 void nodelist_free_all(void);
 void nodelist_assert_ok(void);