Slightly re-structured and re-styled a for-loop for greater understandability.

[plomrogue] / src / windows.c

diff --git a/src/windows.c b/src/windows.c
index 373d2212a1626b11be3f63f0de7e156054dbfe43..362e10a475a3063a463ab3ce4fdb1e7d26ef24c9 100644 (file)
--- a/src/windows.c
+++ b/src/windows.c
@@ -438,11 +438,11 @@ extern void shift_active_win(struct WinMeta * wmeta, char dir)
 
         /* Suspend all visible windows. */
         uint16_t i, i_max;
-        for (i_max = 1, w_p = wmeta->chain_start;
+        for (w_p = wmeta->chain_start, i_max = 1;
              w_p != wmeta->chain_end;
-             i_max++)
+             w_p = w_p->next)
         {
-            w_p = w_p->next;
+            i_max++;
         }
         struct Win ** wins = malloc(i_max * sizeof(struct Win *));
         for (i = 0, w_p = wmeta->chain_start; i < i_max; i++)
@@ -575,7 +575,7 @@ extern void draw_scroll_hint(struct Frame * frame, uint16_t pos, uint32_t dist,
         unit = unit_cols;
     }
     char * scrolldsc = malloc((4 * sizeof(char)) + strlen(more) + strlen(unit)
-                              + 10);  /* 10 = uint32 max strlen */
+                              + 10);                /* 10 = uint32 max strlen */
     sprintf(scrolldsc, " %d %s %s ", dist, more, unit);
 
     /* Decide on offset of the description text inside the scroll hint line. */