[tor-commits] [tor/master] Use SMARTLIST_FOREACH_BEGIN and _END, not a for loop

[nickm at torproject.org]

commit ae9a831294bd9bc6129b447ea3e7e24ab210794c
Author: Robert Ransom <rransom.8774 at gmail.com>
Date:   Sat Oct 15 14:22:56 2011 -0700

    Use SMARTLIST_FOREACH_BEGIN and _END, not a for loop
---
 src/or/rendservice.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/or/rendservice.c b/src/or/rendservice.c
index e89d689..30f5df7 100644
--- a/src/or/rendservice.c
+++ b/src/or/rendservice.c
@@ -1940,8 +1940,7 @@ rend_services_introduce(void)
 
     /* Find out which introduction points we have in progress for this
        service. */
-    for (j=0; j < smartlist_len(service->intro_nodes); ++j) {
-      intro = smartlist_get(service->intro_nodes, j);
+    SMARTLIST_FOREACH_BEGIN(service->intro_nodes, rend_intro_point_t *, intro){
       router = router_get_by_digest(intro->extend_info->identity_digest);
       if (!router || !find_intro_circuit(intro, service->pk_digest)) {
         log_info(LD_REND,"Giving up on %s as intro point for %s.",
@@ -1954,12 +1953,12 @@ rend_services_introduce(void)
           service->desc_is_dirty = now;
         }
         rend_intro_point_free(intro);
-        smartlist_del(service->intro_nodes,j--);
+        SMARTLIST_DEL_CURRENT(service->intro_nodes, intro);
         intro_point_set_changed = 1;
       }
       if (router)
         smartlist_add(intro_routers, router);
-    }
+    } SMARTLIST_FOREACH_END(intro);
 
     if (!intro_point_set_changed &&
         (smartlist_len(service->intro_nodes) >=