diff options
| author | Roger Dingledine <arma@torproject.org> | 2008-12-22 06:25:49 +0000 |
|---|---|---|
| committer | Roger Dingledine <arma@torproject.org> | 2008-12-22 06:25:49 +0000 |
| commit | 0c6ff7d990e45a6699cad5f7f140182ed0c5c991 (patch) | |
| tree | 62b6e54619be90e4cadfa0e7afc1278d41a84750 /src | |
| parent | 7262bf5eab97105ac833d0700a30052e053537dc (diff) | |
| download | tor-0c6ff7d990e45a6699cad5f7f140182ed0c5c991.tar tor-0c6ff7d990e45a6699cad5f7f140182ed0c5c991.tar.gz | |
the default should not be the notify the poster, unless something
more extreme happens. the default should be to be quiet unless
something more extreme happens.
at least, this doesn't generate complaints anymore. perhaps that
means it is working better? :)
svn:r17724
Diffstat (limited to 'src')
| -rw-r--r-- | src/or/dirserv.c | 10 | ||||
| -rw-r--r-- | src/or/or.h | 4 |
2 files changed, 4 insertions, 10 deletions
diff --git a/src/or/dirserv.c b/src/or/dirserv.c index 399e63cc8..598316fcd 100644 --- a/src/or/dirserv.c +++ b/src/or/dirserv.c @@ -568,13 +568,7 @@ authdir_wants_to_reject_router(routerinfo_t *ri, const char **msg, static int WRA_MORE_SEVERE(was_router_added_t a, was_router_added_t b) { - if (b == ROUTER_ADDED_SUCCESSFULLY) { - return 1; - } else if (b == ROUTER_ADDED_NOTIFY_GENERATOR) { - return !WRA_WAS_ADDED(a); - } else { - return a < b; - } + return a < b; } /** As for dirserv_add_descriptor(), but accepts multiple documents, and @@ -595,7 +589,7 @@ dirserv_add_multiple_descriptors(const char *desc, uint8_t purpose, int general = purpose == ROUTER_PURPOSE_GENERAL; tor_assert(msg); - r=ROUTER_ADDED_NOTIFY_GENERATOR; /*Least severe return value. */ + r=ROUTER_ADDED_SUCCESSFULLY; /*Least severe return value. */ format_iso_time(time_buf, now); if (tor_snprintf(annotation_buf, sizeof(annotation_buf), diff --git a/src/or/or.h b/src/or/or.h index 06d1b8935..ffb989443 100644 --- a/src/or/or.h +++ b/src/or/or.h @@ -4373,8 +4373,8 @@ void router_set_status(const char *digest, int up); /** Return value for router_add_to_routerlist() and dirserv_add_descriptor() */ typedef enum was_router_added_t { - ROUTER_ADDED_SUCCESSFULLY = 0, - ROUTER_ADDED_NOTIFY_GENERATOR = 1, + ROUTER_ADDED_SUCCESSFULLY = 1, + ROUTER_ADDED_NOTIFY_GENERATOR = 0, ROUTER_BAD_EI = -1, ROUTER_WAS_NOT_NEW = -2, ROUTER_NOT_IN_CONSENSUS = -3, |