Don't update more windows than necessary in resize_active_win().

[plomrogue] / src / windows.c

diff --git a/src/windows.c b/src/windows.c
index 41eaae9b5982c6ccb2fcd969217b0fec81a06465..a7bf88fc22ba063b46de8d6fc49aa6a4adf3ab83 100644 (file)
--- a/src/windows.c
+++ b/src/windows.c
@@ -81,8 +81,8 @@ static uint8_t refit_pad(struct WinMeta * wmeta)
     /* Only resize the pad if the rightmost window column has changed. */
     if (getmaxx(wmeta->padframe.curses_win) != lastwincol)
     {
-        if (lastwincol + 2 > UINT16_MAX) /* Abort if pad would grow beyond */
-        {                                /* yx_uint16 confines. */
+        if (lastwincol + 2 > UINT16_MAX)
+        {
             return 2;
         }
         return (ERR == wresize(wmeta->padframe.curses_win,
@@ -506,11 +506,11 @@ extern uint8_t resize_active_win(struct WinMeta * wmeta, struct yx_uint16 size)
         && size.y < wmeta->padframe.size.y)
     {
         wmeta->active->frame.size = size;
-        return update_wins(wmeta, wmeta->_chain_start); /* Following windows' */
-    }                                                   /* positioning may be */
-    return 0;                                           /* affected.          */
-}                                                       /* TODO: Why start at */
-                                                        /* chain_start then?  */
+        return update_wins(wmeta, wmeta->active); /* Positioning of following */
+    }                                             /* windows may be affected. */
+    return 0;
+}
+
 
 
 extern void cycle_active_win(struct WinMeta * wmeta, char dir)
@@ -547,8 +547,7 @@ extern void cycle_active_win(struct WinMeta * wmeta, char dir)
 extern uint8_t shift_active_win(struct WinMeta * wmeta, char dir)
 {
     if (   0 == wmeta->active                        /* No shifting with < 2  */
-        || wmeta->_chain_start == wmeta->_chain_end  /* windows visible or    */
-        || (dir != 'f' && dir != 'b'))               /* wrong direction char. */
+        || wmeta->_chain_start == wmeta->_chain_end) /* windows visible.      */
     {
         return 0;
     }