[tor-commits] [tor/master] Make the router_get_*_orport interface consistent with node_*.

[nickm@xxxxxxxxxxxxxx]

commit 39ec781b8fa164dd2b68cec349f2b38d81fa3cbb
Author: Linus Nordberg <linus@xxxxxxxxxxx>
Date:   Mon Nov 28 13:51:56 2011 +0100

    Make the router_get_*_orport interface consistent with node_*.
---
 src/or/nodelist.c |    3 +--
 src/or/router.c   |   22 +++++++++++-----------
 src/or/router.h   |    4 ++--
 3 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/src/or/nodelist.c b/src/or/nodelist.c
index 921ce1a..7d634a1 100644
--- a/src/or/nodelist.c
+++ b/src/or/nodelist.c
@@ -740,8 +740,7 @@ void
 node_get_pref_ipv6_addr(const node_t *node, tor_addr_port_t *ap_out)
 {
   if (node->ri) {
-    tor_addr_copy(&ap_out->addr, &node->ri->ipv6_addr);
-    ap_out->port = node->ri->ipv6_orport;
+    router_get_pref_ipv6_orport(node->ri, ap_out);
   } else if (node->rs) {
     /* No IPv6 in routerstatus_t yet.  XXXprop186 ok for private
        bridges but needs fixing */
diff --git a/src/or/router.c b/src/or/router.c
index a0ae79a..6904e85 100644
--- a/src/or/router.c
+++ b/src/or/router.c
@@ -2132,16 +2132,6 @@ router_get_prim_orport(const routerinfo_t *router, tor_addr_port_t *ap_out)
   ap_out->port = router->or_port;
 }
 
-/** Copy the alternative, presumably IPv6, OR port (IP address and TCP
- * port) for <b>router</b> into *<b>ap_out</b>. */
-void
-router_get_alt_orport(const routerinfo_t *router, tor_addr_port_t *ap_out)
-{
-  tor_assert(ap_out != NULL);
-  tor_addr_copy(&ap_out->addr, &router->ipv6_addr);
-  ap_out->port = router->ipv6_orport;
-}
-
 /** Return 1 if we prefer the IPv6 address and OR TCP port of
  * <b>router</b>, else 0.
  *
@@ -2162,11 +2152,21 @@ void
 router_get_pref_orport(const routerinfo_t *router, tor_addr_port_t *ap_out)
 {
   if (router_ipv6_preferred(router))
-    router_get_alt_orport(router, ap_out);
+    router_get_pref_ipv6_orport(router, ap_out);
   else
     router_get_prim_orport(router, ap_out);
 }
 
+/** Copy the preferred IPv6 OR port (IP address and TCP port) for
+ * <b>router</b> into *<b>ap_out</b>. */
+void
+router_get_pref_ipv6_orport(const routerinfo_t *router, tor_addr_port_t *ap_out)
+{
+  tor_assert(ap_out != NULL);
+  tor_addr_copy(&ap_out->addr, &router->ipv6_addr);
+  ap_out->port = router->ipv6_orport;
+}
+
 /** Load the contents of <b>filename</b>, find the last line starting with
  * <b>end_line</b>, ensure that its timestamp is not more than 25 hours in
  * the past or more than 1 hour in the future with respect to <b>now</b>,
diff --git a/src/or/router.h b/src/or/router.h
index 6095d02..d426b25 100644
--- a/src/or/router.h
+++ b/src/or/router.h
@@ -87,10 +87,10 @@ int router_dump_router_to_string(char *s, size_t maxlen, routerinfo_t *router,
                                  crypto_pk_env_t *ident_key);
 void router_get_prim_orport(const routerinfo_t *router,
                             tor_addr_port_t *addr_port_out);
-void router_get_alt_orport(const routerinfo_t *router,
-                           tor_addr_port_t *addr_port_out);
 void router_get_pref_orport(const routerinfo_t *router,
                             tor_addr_port_t *addr_port_out);
+void router_get_pref_ipv6_orport(const routerinfo_t *router,
+                                 tor_addr_port_t *addr_port_out);
 int router_ipv6_preferred(const routerinfo_t *router);
 int extrainfo_dump_to_string(char **s, extrainfo_t *extrainfo,
                              crypto_pk_env_t *ident_key);



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits