ifconfig(8): Use nitems() consistently
authorAaron LI <aly@aaronly.me>
Tue, 20 Mar 2018 07:45:43 +0000 (15:45 +0800)
committerAaron LI <aly@aaronly.me>
Sun, 5 Aug 2018 01:47:42 +0000 (09:47 +0800)
sbin/ifconfig/af_inet6.c
sbin/ifconfig/ifbridge.c
sbin/ifconfig/ifcarp.c
sbin/ifconfig/ifclone.c
sbin/ifconfig/ifconfig.c
sbin/ifconfig/ifieee80211.c
sbin/ifconfig/iflagg.c
sbin/ifconfig/ifmedia.c
sbin/ifconfig/ifvlan.c
sbin/ifconfig/regdomain.c

index 1f23434..91bb76b 100644 (file)
@@ -529,12 +529,10 @@ static struct option in6_Lopt = { "L", "[-L]", in6_Lopt_cb };
 static __constructor(101) void
 inet6_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(inet6_cmds);  i++)
+       for (i = 0; i < nitems(inet6_cmds);  i++)
                cmd_register(&inet6_cmds[i]);
        af_register(&af_inet6);
        opt_register(&in6_Lopt);
-#undef N
 }
index f508fd1..48c1881 100644 (file)
@@ -156,7 +156,7 @@ bridge_interfaces(int s, const char *prefix)
                        printf("port %u priority %u",
                            req->ifbr_portno, req->ifbr_priority);
                        printf(" pathcost %u", req->ifbr_path_cost);
-                       if (req->ifbr_state < NELEM(stpstates))
+                       if (req->ifbr_state < nitems(stpstates))
                                printf(" %s", stpstates[req->ifbr_state]);
                        else
                                printf(" <unknown state %d>",
@@ -592,11 +592,9 @@ static struct afswtch af_bridge = {
 static __constructor(101) void
 bridge_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(bridge_cmds);  i++)
+       for (i = 0; i < nitems(bridge_cmds);  i++)
                cmd_register(&bridge_cmds[i]);
        af_register(&af_bridge);
-#undef N
 }
index abb2ae1..b59c4dc 100644 (file)
@@ -250,11 +250,9 @@ static struct afswtch af_carp = {
 static __constructor(101) void
 carp_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(carp_cmds);  i++)
+       for (i = 0; i < nitems(carp_cmds);  i++)
                cmd_register(&carp_cmds[i]);
        af_register(&af_carp);
-#undef N
 }
index c5a3031..145345b 100644 (file)
@@ -181,11 +181,9 @@ static struct option clone_Copt = { .opt = "C", .opt_usage = "[-C]", .cb = clone
 static __constructor(101) void
 clone_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        size_t i;
 
-       for (i = 0; i < N(clone_cmds);  i++)
+       for (i = 0; i < nitems(clone_cmds);  i++)
                cmd_register(&clone_cmds[i]);
        opt_register(&clone_Copt);
-#undef N
 }
index 926b10b..77eb0c0 100644 (file)
@@ -440,7 +440,6 @@ cmd_register(struct cmd *p)
 static const struct cmd *
 cmd_lookup(const char *name, int iscreate)
 {
-#define        N(a)    (sizeof(a)/sizeof(a[0]))
        const struct cmd *p;
 
        for (p = cmds; p != NULL; p = p->c_next)
@@ -454,7 +453,6 @@ cmd_lookup(const char *name, int iscreate)
                        }
                }
        return NULL;
-#undef N
 }
 
 struct callback {
@@ -1127,10 +1125,8 @@ static struct cmd basic_cmds[] = {
 static __constructor(101) void
 ifconfig_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(basic_cmds);  i++)
+       for (i = 0; i < nitems(basic_cmds);  i++)
                cmd_register(&basic_cmds[i]);
-#undef N
 }
index 5e1bbfa..7a6d66f 100644 (file)
@@ -2831,7 +2831,6 @@ printrsnie(const char *tag, const u_int8_t *ie, size_t ielen, int maxlen)
 static void
 printwpsie(const char *tag, const u_int8_t *ie, size_t ielen, int maxlen)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        u_int8_t len = ie[1];
 
        printf("%s", tag);
@@ -2872,7 +2871,7 @@ printwpsie(const char *tag, const u_int8_t *ie, size_t ielen, int maxlen)
                                break;
                        case IEEE80211_WPS_DEV_PASS_ID:
                                n = LE_READ_2(ie);
-                               if (n < N(dev_pass_id))
+                               if (n < nitems(dev_pass_id))
                                        printf(" dpi:%s", dev_pass_id[n]);
                                break;
                        case IEEE80211_WPS_UUID_E:
@@ -2886,7 +2885,6 @@ printwpsie(const char *tag, const u_int8_t *ie, size_t ielen, int maxlen)
                }
                printf(">");
        }
-#undef N
 }
 
 static void
@@ -3388,7 +3386,6 @@ list_stations(int s)
 static const char *
 mesh_linkstate_string(uint8_t state)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        static const char *state_names[] = {
            [0] = "IDLE",
            [1] = "OPEN-TX",
@@ -3398,13 +3395,12 @@ mesh_linkstate_string(uint8_t state)
            [5] = "HOLDING",
        };
 
-       if (state >= N(state_names)) {
+       if (state >= nitems(state_names)) {
                static char buf[10];
                snprintf(buf, sizeof(buf), "#%u", state);
                return buf;
        } else
                return state_names[state];
-#undef N
 }
 
 static const char *
@@ -5261,12 +5257,10 @@ static struct afswtch af_ieee80211 = {
 static __constructor(101) void
 ieee80211_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(ieee80211_cmds);  i++)
+       for (i = 0; i < nitems(ieee80211_cmds);  i++)
                cmd_register(&ieee80211_cmds[i]);
        af_register(&af_ieee80211);
        clone_setdefcallback("wlan", wlan_create);
-#undef N
 }
index 4a78381..5c1ec0b 100644 (file)
@@ -62,7 +62,7 @@ setlaggproto(const char *val, int d, int s, const struct afswtch *afp)
        bzero(&ra, sizeof(ra));
        ra.ra_proto = LAGG_PROTO_MAX;
 
-       for (i = 0; i < (NELEM(lpr)); i++) {
+       for (i = 0; i < nitems(lpr); i++) {
                if (strcmp(val, lpr[i].lpr_name) == 0) {
                        ra.ra_proto = lpr[i].lpr_proto;
                        break;
@@ -163,7 +163,7 @@ lagg_status(int s)
        if (ioctl(s, SIOCGLAGG, &ra) == 0) {
                lp = (struct lacp_opreq *)&ra.ra_lacpreq;
 
-               for (i = 0; i < (NELEM(lpr)); i++) {
+               for (i = 0; i < nitems(lpr); i++) {
                        if (ra.ra_proto == lpr[i].lpr_proto) {
                                proto = lpr[i].lpr_name;
                                break;
@@ -209,7 +209,7 @@ lagg_status(int s)
 
                if (0 /* XXX */) {
                        printf("\tsupported aggregation protocols:\n");
-                       for (i = 0; i < (NELEM(lpr)); i++)
+                       for (i = 0; i < nitems(lpr); i++)
                                printf("\t\tlaggproto %s\n", lpr[i].lpr_name);
                }
        }
@@ -230,11 +230,9 @@ static struct afswtch af_lagg = {
 static __constructor(101) void
 lagg_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(lagg_cmds);  i++)
+       for (i = 0; i < nitems(lagg_cmds);  i++)
                cmd_register(&lagg_cmds[i]);
        af_register(&af_lagg);
-#undef N
 }
index db20187..1c5875f 100644 (file)
@@ -742,11 +742,9 @@ static struct afswtch af_media = {
 static __constructor(101) void
 ifmedia_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(media_cmds);  i++)
+       for (i = 0; i < nitems(media_cmds);  i++)
                cmd_register(&media_cmds[i]);
        af_register(&af_media);
-#undef N
 }
index 826a969..6bfcd2a 100644 (file)
@@ -153,12 +153,10 @@ static struct afswtch af_vlan = {
 static __constructor(101) void
 vlan_ctor(void)
 {
-#define        N(a)    (sizeof(a) / sizeof(a[0]))
        int i;
 
-       for (i = 0; i < N(vlan_cmds);  i++)
+       for (i = 0; i < nitems(vlan_cmds);  i++)
                cmd_register(&vlan_cmds[i]);
        af_register(&af_vlan);
        callback_register(vlan_cb, NULL);
-#undef N
 }
index 0bdee59..8d138c3 100644 (file)
@@ -213,7 +213,7 @@ decode_flag(struct mystate *mt, const char *p, int len)
        };
        int i;
 
-       for (i = 0; i < sizeof(flags)/sizeof(flags[0]); i++)
+       for (i = 0; i < nitems(flags); i++)
                if (len == flags[i].len && iseq(p, flags[i].name))
                        return flags[i].value;
        warnx("unknown flag \"%.*s\" at line %ld ignored",