Minor formatting change.

[plomrogue] / src / windows.c

diff --git a/src/windows.c b/src/windows.c
index 46124c5639a8b4c0b984e274adc573d5aa4bf235..861b737681b2b1b6c4ae5b92273e4523df7ae668 100644 (file)
--- a/src/windows.c
+++ b/src/windows.c
@@ -436,16 +436,18 @@ extern void shift_active_win(struct WinMeta * wmeta, char dir)
             wrap = 1;
         }
 
-        /* Suspend all visible windows. */
+        /* Suspend all visible windows, remember their order in wins[]. */
         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++)
+        for (i = 0, w_p = wmeta->chain_start;
+             i < i_max;
+             i++)
         {
             w_p_next = w_p->next;
             suspend_win(wmeta, w_p);
@@ -492,7 +494,7 @@ extern void shift_active_win(struct WinMeta * wmeta, char dir)
         }
         free(wins);
 
-        wmeta->active = w_shift;      /* TODO: Is this necessary? If so, why? */
+        wmeta->active = w_shift;  /* Otherwise lastly appended win is active. */
     }
 }
 
@@ -519,7 +521,7 @@ extern void draw_all_wins(struct WinMeta * wmeta)
             n_wins++;
         }
         struct Corners * all_corners = malloc(sizeof(struct Corners) * n_wins);
-        draw_wins_borders (wmeta->chain_start, wmeta->active, all_corners, 0);
+        draw_wins_borders(wmeta->chain_start, wmeta->active, all_corners, 0);
         for (i = 0; i < n_wins; i++)
         {
             mvwaddch(wmeta->pad.curses_win,