[tor-commits] [tor/master] Remove two extrneous semicolons in dirserv.c

nickm at torproject.org nickm at torproject.org
Wed Jan 16 21:52:00 UTC 2013


commit e4821fa14de6d76219d4e5c1a320ba263bd5e46d
Author: Nick Mathewson <nickm at torproject.org>
Date:   Wed Jan 16 16:49:39 2013 -0500

    Remove two extrneous semicolons in dirserv.c
    
    In 6fbdf635 we added a couple of statements like:
        if (test) {
           ...
        };
    
    The extraneous semicolons there get flagged as worrisome empty
    statements by the cparser library, so let's fix them.
    
    Patch by Christian Grothoff; fixes bug 7115.
---
 changes/bug7115  |    4 ++++
 src/or/dirserv.c |    4 ++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/changes/bug7115 b/changes/bug7115
new file mode 100644
index 0000000..976b9a9
--- /dev/null
+++ b/changes/bug7115
@@ -0,0 +1,4 @@
+  o Minor bugfixes (portability)
+    - Remove a couple of extraneous semicolos that were upsetting the
+      cparser library. Patch by Christian Grothoff. Fixes bug 7115;
+      bugfix on 0.2.2.1-alpha.
diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index 2351386..8af848c 100644
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -2181,7 +2181,7 @@ routerstatus_format_entry(char *buf, size_t buf_len,
             "(wanted descriptor %s).",
             id, dd);
         return -1;
-      };
+      }
 
       /* This assert can fire for the control port, because
        * it can request NS documents before all descriptors
@@ -2205,7 +2205,7 @@ routerstatus_format_entry(char *buf, size_t buf_len,
         tor_assert(tor_memeq(desc->cache_info.signed_descriptor_digest,
               rs->descriptor_digest,
               DIGEST_LEN));
-      };
+      }
     }
 
     if (format == NS_CONTROL_PORT && rs->has_bandwidth) {



More information about the tor-commits mailing list