From: Nick Mathewson <nickm@torproject.org>
Date: Wed, 13 Mar 2013 13:11:31 +0000 (-0400)
Subject: Simplify the logic of circuit_build_times_get_initial_timeout.
X-Git-Tag: tor-0.2.4.12-alpha~5^2~27^2~1
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=7543fb61e34e9235c6c330270d0c370c64c3322a;p=thirdparty%2Ftor.git

Simplify the logic of circuit_build_times_get_initial_timeout.
---

diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c
index 24362460cd..fb8240b940 100644
--- a/src/or/circuitbuild.c
+++ b/src/or/circuitbuild.c
@@ -540,23 +540,16 @@ circuit_build_times_get_initial_timeout(void)
    * Check if we have LearnCircuitBuildTimeout, and if we don't,
    * always use CircuitBuildTimeout, no questions asked.
    */
-  /*XXXX025 this logic could be cleaned up a lot. */
-  if (get_options()->LearnCircuitBuildTimeout) {
-    if (!unit_tests && get_options()->CircuitBuildTimeout) {
-      timeout = get_options()->CircuitBuildTimeout*1000;
-      if (timeout < circuit_build_times_min_timeout()) {
-        log_warn(LD_CIRC, "Config CircuitBuildTimeout too low. Setting to %ds",
-                 circuit_build_times_min_timeout()/1000);
-        timeout = circuit_build_times_min_timeout();
-      }
-    } else {
-      timeout = circuit_build_times_initial_timeout();
+  if (!unit_tests && get_options()->CircuitBuildTimeout) {
+    timeout = get_options()->CircuitBuildTimeout*1000;
+    if (get_options()->LearnCircuitBuildTimeout &&
+        timeout < circuit_build_times_min_timeout()) {
+      log_warn(LD_CIRC, "Config CircuitBuildTimeout too low. Setting to %ds",
+               circuit_build_times_min_timeout()/1000);
+      timeout = circuit_build_times_min_timeout();
     }
   } else {
-    if (get_options()->CircuitBuildTimeout > 0)
-      timeout = get_options()->CircuitBuildTimeout*1000;
-    else
-      timeout = circuit_build_times_initial_timeout();
+    timeout = circuit_build_times_initial_timeout();
   }
 
   return timeout;