r11727@catbus: nickm | 2007-02-08 17:07:50 -0500

 Explain that update_router_desciptor_downloads happens every 10 seconds in main.c, and we don't need to call it from directory.c


svn:r9534

1 files changed, 6 insertions, 5 deletions

diff --git a/src/or/directory.c b/src/or/directory.c
index e181fa8fb..c0eaf48d5 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -349,11 +349,13 @@ connection_dir_download_networkstatus_failed(dir_connection_t *conn,
 static void
 connection_dir_download_routerdesc_failed(dir_connection_t *conn)
 {
-  /* Try again. No need to increment the failure count for routerdescs, since
+  /* No need to increment the failure count for routerdescs, since
    * it's not their fault. */
+
+  /* There's no need to call this here: we already call it every 10 seconds *
+   *  (DESCRIPTOR_RETRY_INTERVAL) in main.c -NM */
   /* update_router_descriptor_downloads(time(NULL)); */
   (void) conn;
-  /* XXXX012 Why did the above get commented out? -NM */
 }
 
 /** Helper for directory_initiate_command_(router|trusted_dir): send the
@@ -2055,9 +2057,8 @@ dir_routerdesc_download_failed(smartlist_t *failed, int status_code)
                 cp, (int)rs->n_download_failures);
   });
 
-  /* XXX012 why did this get commented out too? -RD */
-  /*        Because we already call update_router_descriptor_downloads()
-   *        every 10 seconds (DESCRIPTOR_RETRY_INTERVAL) in main.c -NM */
+  /* There's no need to call this here: we already call it every 10 seconds *
+   *  (DESCRIPTOR_RETRY_INTERVAL) in main.c -NM */
   /* update_router_descriptor_downloads(time(NULL)); */
 }