From 76f61da37ba6443712b212a747ce8ee56218dcec Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 8 Feb 2007 22:08:04 +0000 Subject: [PATCH] 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 --- src/or/directory.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/or/directory.c b/src/or/directory.c index e181fa8fbd..c0eaf48d56 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)); */ }