[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[or-cvs] r9534: Explain that update_router_desciptor_downloads happens every (in tor/trunk: . src/or)
- To: or-cvs@xxxxxxxxxxxxx
- Subject: [or-cvs] r9534: Explain that update_router_desciptor_downloads happens every (in tor/trunk: . src/or)
- From: nickm@xxxxxxxx
- Date: Thu,  8 Feb 2007 17:08:07 -0500 (EST)
- Delivered-to: archiver@seul.org
- Delivered-to: or-cvs-outgoing@seul.org
- Delivered-to: or-cvs@seul.org
- Delivery-date: Thu, 08 Feb 2007 17:08:39 -0500
- Reply-to: or-talk@xxxxxxxxxxxxx
- Sender: owner-or-cvs@xxxxxxxxxxxxx
Author: nickm
Date: 2007-02-08 17:08:04 -0500 (Thu, 08 Feb 2007)
New Revision: 9534
Modified:
   tor/trunk/
   tor/trunk/src/or/directory.c
Log:
 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
Property changes on: tor/trunk
___________________________________________________________________
 svk:merge ticket from /tor/trunk [r11727] on 8246c3cf-6607-4228-993b-4d95d33730f1
Modified: tor/trunk/src/or/directory.c
===================================================================
--- tor/trunk/src/or/directory.c	2007-02-08 22:07:56 UTC (rev 9533)
+++ tor/trunk/src/or/directory.c	2007-02-08 22:08:04 UTC (rev 9534)
@@ -349,11 +349,13 @@
 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 @@
                 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)); */
 }