From: Roger Dingledine <arma@torproject.org>
Date: Tue, 20 Jul 2010 12:07:44 +0000 (-0400)
Subject: naked constants are bad
X-Git-Tag: tor-0.2.2.15-alpha~72
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=66d5ce266e5d9a85048255c3aee8373d0c2cf651;p=thirdparty%2Ftor.git

naked constants are bad
---

diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c
index 33f208a7c7..ef6751858a 100644
--- a/src/or/circuitbuild.c
+++ b/src/or/circuitbuild.c
@@ -2323,6 +2323,9 @@ onionskin_answer(or_circuit_t *circ, uint8_t cell_type, const char *payload,
   return 0;
 }
 
+/** How many hops does a general-purpose circuit have by default? */
+#define DEFAULT_ROUTE_LEN 3
+
 /** Choose a length for a circuit of purpose <b>purpose</b>.
  * Default length is 3 + the number of endpoints that would give something
  * away. If the routerlist <b>routers</b> doesn't have enough routers
@@ -2338,7 +2341,7 @@ new_route_len(uint8_t purpose, extend_info_t *exit,
 
   tor_assert(routers);
 
-  routelen = 3;
+  routelen = DEFAULT_ROUTE_LEN;
   if (exit &&
       purpose != CIRCUIT_PURPOSE_TESTING &&
       purpose != CIRCUIT_PURPOSE_S_ESTABLISH_INTRO)