[or-cvs] [tor/release-0.2.2 113/162] Clean up my 1776 fix a bit

arma at torproject.org arma at torproject.org
Mon Dec 20 03:06:40 UTC 2010


Author: Nick Mathewson <nickm at torproject.org>
Date: Mon, 15 Nov 2010 19:38:19 -0500
Subject: Clean up my 1776 fix a bit
Commit: 45b500d5a60ca6c43e5a545b9e15e203be1feeac

Sebastian notes (and I think correctly) that one of our ||s should
have been an &&, which simplifies a boolean expression to decide
whether to replace bridges.  I'm also refactoring out the negation at
the start of the expression, to make it more readable.
---
 src/or/routerlist.c |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 0e9265f..670574a 100644
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -3238,8 +3238,12 @@ router_add_to_routerlist(routerinfo_t *router, const char **msg,
     const int was_bridge = old_router &&
       old_router->purpose == ROUTER_PURPOSE_BRIDGE;
 
-    if (! (routerinfo_is_a_configured_bridge(router) &&
-           (router->purpose == ROUTER_PURPOSE_BRIDGE || !was_bridge))) {
+    if (routerinfo_is_a_configured_bridge(router) &&
+        router->purpose == ROUTER_PURPOSE_BRIDGE &&
+        !was_bridge) {
+      log_info(LD_DIR, "Replacing non-bridge descriptor with bridge "
+               "descriptor for router '%s'", router->nickname);
+    } else {
       log_info(LD_DIR,
                "Dropping descriptor that we already have for router '%s'",
                router->nickname);
-- 
1.7.1




More information about the tor-commits mailing list