[or-cvs] r15308: Backport: More bug 707 fixes: no #if/#endif pairs inside mac (in tor/branches/tor-0_2_0-patches: . src/or)

[nickm at seul.org]

Author: nickm
Date: 2008-06-16 14:35:34 -0400 (Mon, 16 Jun 2008)
New Revision: 15308

Modified:
   tor/branches/tor-0_2_0-patches/ChangeLog
   tor/branches/tor-0_2_0-patches/src/or/routerlist.c
Log:
Backport: More bug 707 fixes: no #if/#endif pairs inside macro arguments.

Modified: tor/branches/tor-0_2_0-patches/ChangeLog
===================================================================
--- tor/branches/tor-0_2_0-patches/ChangeLog	2008-06-16 18:35:21 UTC (rev 15307)
+++ tor/branches/tor-0_2_0-patches/ChangeLog	2008-06-16 18:35:34 UTC (rev 15308)
@@ -1,7 +1,8 @@
 Changes in version 0.2.0.29 - 2008-06-13
   o Minor bugfixes:
-    - Fix a macro/CPP interactions that was confusing some compilers.
-      
+    - Fix a macro/CPP interactions that was confusing some compilers:
+      some GCCs don't like #if/#endif pairs inside macro arguments.
+      Fix for bug 707.
 
 
 Changes in version 0.2.0.28-rc - 2008-06-13

Modified: tor/branches/tor-0_2_0-patches/src/or/routerlist.c
===================================================================
--- tor/branches/tor-0_2_0-patches/src/or/routerlist.c	2008-06-16 18:35:21 UTC (rev 15307)
+++ tor/branches/tor-0_2_0-patches/src/or/routerlist.c	2008-06-16 18:35:34 UTC (rev 15308)
@@ -4391,7 +4391,6 @@
                     r->cache_info.signed_descriptor_digest);
     tor_assert(&(r->cache_info) == sd2);
     tor_assert(r->cache_info.routerlist_index == r_sl_idx);
-#if 0
     /* XXXX021.
      *
      *   Hoo boy.  We need to fix this one, and the fix is a bit tricky, so
@@ -4409,13 +4408,13 @@
      * reset our retry count for an extrainfo, but that's not the end
      * of the world.  Changing the representation in 0.2.0.x would just
      * destabilize the codebase.
-     */
+
     if (!tor_digest_is_zero(r->cache_info.extra_info_digest)) {
       signed_descriptor_t *sd3 =
         sdmap_get(rl->desc_by_eid_map, r->cache_info.extra_info_digest);
       tor_assert(sd3 == &(r->cache_info));
     }
-#endif
+    */
   });
   SMARTLIST_FOREACH(rl->old_routers, signed_descriptor_t *, sd,
   {
@@ -4424,14 +4423,13 @@
     sd2 = sdmap_get(rl->desc_digest_map, sd->signed_descriptor_digest);
     tor_assert(sd == sd2);
     tor_assert(sd->routerlist_index == sd_sl_idx);
-#if 0
-    /* XXXX021 see above. */
+    /* XXXX021 see above.
     if (!tor_digest_is_zero(sd->extra_info_digest)) {
       signed_descriptor_t *sd3 =
         sdmap_get(rl->desc_by_eid_map, sd->extra_info_digest);
       tor_assert(sd3 == sd);
     }
-#endif
+    */
   });
 
   RIMAP_FOREACH(rl->identity_map, d, r) {