[tor-commits] [tor/master] Remove unused variable in directory_command_should_use_begindir()

nickm at torproject.org nickm at torproject.org
Mon Feb 27 15:58:47 UTC 2017


commit 89334a040d6496191e3cee1d69ecd3c0e566583a
Author: Alexander Færøy <ahf at torproject.org>
Date:   Fri Feb 10 23:01:52 2017 +0000

    Remove unused variable in directory_command_should_use_begindir()
    
    This patch removes the unused router_purpose variable in
    directory_command_should_use_begindir().
---
 src/or/directory.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/or/directory.c b/src/or/directory.c
index 36714bc..4ef39af 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -1078,11 +1078,10 @@ directory_must_use_begindir(const or_options_t *options)
 static int
 directory_command_should_use_begindir(const or_options_t *options,
                                       const tor_addr_t *addr,
-                                      int or_port, uint8_t router_purpose,
+                                      int or_port,
                                       dir_indirection_t indirection,
                                       const char **reason)
 {
-  (void) router_purpose;
   tor_assert(reason);
   *reason = NULL;
 
@@ -1190,8 +1189,7 @@ directory_initiate_command_rend(const tor_addr_port_t *or_addr_port,
    * send our directory request)? */
   const int use_begindir = directory_command_should_use_begindir(options,
                                      &or_addr_port->addr, or_addr_port->port,
-                                     router_purpose, indirection,
-                                     &begindir_reason);
+                                     indirection, &begindir_reason);
   /* Will the connection go via a three-hop Tor circuit? Note that this
    * is separate from whether it will use_begindir. */
   const int anonymized_connection = dirind_is_anon(indirection);





More information about the tor-commits mailing list