Clean up poorly chosen loop control variable names
authorMarkus Armbruster <armbru@pond.sub.org>
Sat, 9 Jan 2010 13:51:59 +0000 (14:51 +0100)
committerMarkus Armbruster <armbru@pond.sub.org>
Tue, 19 Jan 2010 07:38:16 +0000 (08:38 +0100)
Calling a counting loop's control variable q or z is tasteless
bordering on actively misleading.

src/lib/commands/buy.c
src/lib/subs/mission.c

index 2f8dcbaafe44a5ddbadec8d7704f81d51be8c08c..8e00d961ac0b9fc8e3275c8c6a5a5b6f862231d7 100644 (file)
@@ -61,7 +61,7 @@ buy(void)
     struct trdstr tmpt;
     struct ichrstr *ip;
     int qty;
-    int o, n, q;
+    int o, n;
     coord x, y;
     char *p;
     float bid;
@@ -115,13 +115,13 @@ buy(void)
     would make him go broke.  Ken, I ought to skin you alive for making me code
     this part up.*/
     tally = 0.0;
-    for (q = 0; gettrade(q, &tmpt); q++) {
+    for (n = 0; gettrade(n, &tmpt); n++) {
        if (tmpt.trd_maxbidder == player->cnum &&
            tmpt.trd_unitid >= 0 && tmpt.trd_owner != player->cnum) {
            tally += tmpt.trd_price * tradetax;
        }
     }
-    for (q = 0; getcomm(q, &comt); q++) {
+    for (n = 0; getcomm(n, &comt); n++) {
        if (comt.com_maxbidder == player->cnum &&
            comt.com_owner != 0 && comt.com_owner != player->cnum) {
            tally += comt.com_price * comt.com_amount * buytax;
index 692acc883e6e784b5ead535a0e9277f1eb643fb2..e1f98df1db27216b94c00a0980cd6057e8096030 100644 (file)
@@ -92,11 +92,10 @@ ground_interdict(coord x, coord y, natid victim, char *s)
     int cn;
     int dam = 0, newdam, rel;
     struct genlist mi[MAXNOC];
-    int z;
 
     memset(mi, 0, sizeof(mi));
-    for (z = 1; z < MAXNOC; z++)
-       emp_initque((struct emp_qelem *)&mi[z]);
+    for (cn = 1; cn < MAXNOC; cn++)
+       emp_initque((struct emp_qelem *)&mi[cn]);
 
     build_mission_list(mi, x, y, MI_INTERDICT, victim);
 
@@ -174,12 +173,11 @@ unit_interdict(coord x, coord y, natid victim, char *s, int hardtarget,
     int cn;
     int dam = 0, newdam;
     struct genlist mi[MAXNOC];
-    int z;
     int osubs;
 
     memset(mi, 0, sizeof(mi));
-    for (z = 1; z < MAXNOC; z++)
-       emp_initque((struct emp_qelem *)&mi[z]);
+    for (cn = 1; cn < MAXNOC; cn++)
+       emp_initque((struct emp_qelem *)&mi[cn]);
 
     build_mission_list(mi, x, y, mission, victim);
 
@@ -220,11 +218,11 @@ off_support(coord x, coord y, natid victim, natid actee)
 {
     int dam = 0;
     struct genlist mi[MAXNOC];
-    int z;
+    int cn;
 
     memset(mi, 0, sizeof(mi));
-    for (z = 1; z < MAXNOC; z++)
-       emp_initque((struct emp_qelem *)&mi[z]);
+    for (cn = 1; cn < MAXNOC; cn++)
+       emp_initque((struct emp_qelem *)&mi[cn]);
 
     build_mission_list(mi, x, y, MI_SUPPORT, victim);
     build_mission_list(mi, x, y, MI_OSUPPORT, victim);
@@ -241,11 +239,11 @@ def_support(coord x, coord y, natid victim, natid actee)
 {
     int dam = 0;
     struct genlist mi[MAXNOC];
-    int z;
+    int cn;
 
     memset(mi, 0, sizeof(mi));
-    for (z = 1; z < MAXNOC; z++)
-       emp_initque((struct emp_qelem *)&mi[z]);
+    for (cn = 1; cn < MAXNOC; cn++)
+       emp_initque((struct emp_qelem *)&mi[cn]);
 
     build_mission_list(mi, x, y, MI_SUPPORT, victim);
     build_mission_list(mi, x, y, MI_DSUPPORT, victim);