From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Sun, 12 Oct 2014 19:34:42 +0000 (+0100)
Subject: src/types.c: Clarify addr_rrtypeflag loop (style)
X-Git-Tag: wip.ipv6.2014-10-13.tip~3
X-Git-Url: https://git.distorted.org.uk/~mdw/adns/commitdiff_plain/de3f4632a456b96947ca51613694121465bcc15a

src/types.c: Clarify addr_rrtypeflag loop (style)

This structure avoids repeating the loop termination condition.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---

diff --git a/src/types.c b/src/types.c
index 55a7709..5062c6a 100644
--- a/src/types.c
+++ b/src/types.c
@@ -373,8 +373,10 @@ static unsigned addr_rrtypeflag(adns_rrtype type) {
   int i;
 
   type &= adns_rrt_typemask;
-  for (i=0; i<addr_nrrtypes && type!=addr_all_rrtypes[i]; i++);
-  return i < addr_nrrtypes ? 1 << i : 0;
+  for (i=0; i<addr_nrrtypes; i++)
+     if (type==addr_all_rrtypes[i])
+       return 1 << i;
+  return 0;
 }
 
 /* About CNAME handling in addr queries.