home · contact · privacy
Re-wrote shift_active_win() to manipulate chain by merely changing the chain pointers...
[plomrogue] / src / windows.c
index 86cf1fecde03da528ceec6bab03170c85e68113a..ec176ed7ea2f11414a43c97feeedf4438881e754 100644 (file)
@@ -1,25 +1,30 @@
 /* windows.c */
 
 #include "windows.h"
-#include <stdint.h>    /* for uint16_t, uint32_t */
-#include <ncurses.h>   /* for LOTS of stuff */
+#include <stdint.h>    /* for uint8_t, uint16_t, uint32_t */
+#include <ncurses.h>   /* for typedefs WINDOW, chtype, wresize(), getmaxx(), */
+                       /* getmaxy(), supbad(), delwin(), mvwaddch(),         */
+                       /* mvwaddstr(), newpad(), wnoutrefres(), erase(),     */
+                       /* werase(), pnoutrefresh(), doupdate()               */
 #include <stdlib.h>    /* for malloc(), free() */
-#include <string.h>    /* for strlen(), memcpy() */
+#include <string.h>    /* for strlen(), strnlen(), memcpy() */
 #include "yx_uint16.h" /* for yx_uint16 coordinates */
 
 
 
 /* Fit virtual screen's width to minimum width demanded by current windows'
- * geometries.
+ * geometries. Returns 0 on success, 1 on (pad memory allocation) error.
  */
-static void refit_pad(struct WinMeta * wmeta);
+static uint8_t refit_pad(struct WinMeta * wmeta);
 
 
 
 /* Update geometry (sizes, positions) of window "w" and its successors in the
  * window chain. For the positioning algorithm place_win() is used.
+ *
+ * update_wins() returns 0 on success, 1 on (pad/window memory alloc.) error.
  */
-static void update_wins(struct WinMeta * wmeta, struct Win * w);
+static uint8_t update_wins(struct WinMeta * wmeta, struct Win * w);
 static void place_win(struct WinMeta * wmeta, struct Win * w);
 
 
@@ -53,7 +58,13 @@ static void draw_wins_bordercorners(struct Win * w, WINDOW * pad);
 
 
 
-static void refit_pad(struct WinMeta * wmeta)
+/* Shift active window forwards / backwards in window chain. */
+static void shift_win_forward(struct WinMeta * wmeta);
+static void shift_win_backward(struct WinMeta * wmeta);
+
+
+
+static uint8_t refit_pad(struct WinMeta * wmeta)
 {
     /* Determine rightmost window column. */
     uint16_t lastwincol = 0;
@@ -70,33 +81,44 @@ static void refit_pad(struct WinMeta * wmeta)
     /* Only resize the pad if the rightmost window column has changed. */
     if (getmaxx(wmeta->padframe.curses_win) != lastwincol)
     {
-        wresize(wmeta->padframe.curses_win,
-                getmaxy(wmeta->padframe.curses_win), lastwincol + 2);
+        return (ERR == wresize(wmeta->padframe.curses_win,
+                               getmaxy(wmeta->padframe.curses_win),
+                               lastwincol + 2));
     }
+    return 0;
 }
 
 
 
-static void update_wins (struct WinMeta * wmeta, struct Win * w)
+static uint8_t update_wins(struct WinMeta * wmeta, struct Win * w)
 {
     if (0 != w->frame.curses_win)
     {
-        destroy_win (w);
+        destroy_win(w);
     }
     place_win(wmeta, w);
-    refit_pad(wmeta);
-    w->frame.curses_win = subpad(wmeta->padframe.curses_win,
-                                 w->frame.size.y, w->frame.size.x,
-                                 w->start.y, w->start.x);
+    if (0 != refit_pad(wmeta))
+    {
+        return 1;
+    }
+    WINDOW * test = subpad(wmeta->padframe.curses_win,
+                           w->frame.size.y, w->frame.size.x,
+                           w->start.y, w->start.x);
+    if (NULL == test)
+    {
+        return 1;
+    }
+    w->frame.curses_win = test;
     if (0 != w->next)
     {
-        update_wins (wmeta, w->next);
+        return update_wins(wmeta, w->next);
     }
+    return 0;
 }
 
 
 
-static void place_win (struct WinMeta * wmeta, struct Win * w)
+static void place_win(struct WinMeta * wmeta, struct Win * w)
 {
     /* First window goes into the upper-left corner. */
     w->start.x = 0;
@@ -162,7 +184,7 @@ static void place_win (struct WinMeta * wmeta, struct Win * w)
 
 
 
-static void destroy_win (struct Win * w)
+static void destroy_win(struct Win * w)
 {
     delwin(w->frame.curses_win);
     w->frame.curses_win = 0;
@@ -170,12 +192,12 @@ static void destroy_win (struct Win * w)
 
 
 
-static void draw_wins (struct Win * w)
+static void draw_wins(struct Win * w)
 {
     w->draw(w);
     if (0 != w->next)
     {
-        draw_wins (w->next);
+        draw_wins(w->next);
     }
 }
 
@@ -232,7 +254,7 @@ static void draw_wins_borderlines(struct Win * w, struct Win * w_active,
     draw_win_borderlines(w, active, pad);
     if (0 != w->next)
     {
-        draw_wins_borderlines (w->next, w_active, pad);
+        draw_wins_borderlines(w->next, w_active, pad);
     }
 }
 
@@ -253,23 +275,111 @@ static void draw_wins_bordercorners(struct Win * w, WINDOW * pad)
 
 
 
-extern struct WinMeta init_win_meta(WINDOW * screen)
+static void shift_win_forward(struct WinMeta * wmeta)
 {
-    struct WinMeta wmeta;
-    wmeta.screen              = screen;
-    wmeta.padframe.size.y     = getmaxy(screen);
-    wmeta.padframe.size.x     = getmaxx(screen);
-    wmeta.chain_start         = 0;
-    wmeta.chain_end           = 0;
-    wmeta.pad_offset          = 0;
-    wmeta.padframe.curses_win = newpad(wmeta.padframe.size.y, 1);
-    wmeta.active              = 0;
-    return wmeta;
+    if (wmeta->active == wmeta->chain_end)
+    {
+        wmeta->chain_end = wmeta->active->prev;
+        wmeta->chain_end->next = 0;
+        wmeta->active->next = wmeta->chain_start;
+        wmeta->active->next->prev = wmeta->active;
+        wmeta->chain_start = wmeta->active;
+        wmeta->chain_start->prev = 0;
+    }
+    else
+    {
+        struct Win * old_prev = wmeta->active->prev;
+        struct Win * old_next = wmeta->active->next;
+        if (wmeta->chain_end == wmeta->active->next)
+        {
+            wmeta->chain_end = wmeta->active;
+            wmeta->active->next = 0;
+        }
+        else
+        {
+            wmeta->active->next = old_next->next;
+            wmeta->active->next->prev = wmeta->active;
+        }
+        if (wmeta->chain_start == wmeta->active)
+        {
+            wmeta->chain_start = old_next;
+        }
+        else
+        {
+            old_prev->next = old_next;
+        }
+        old_next->prev = old_prev;
+        old_next->next = wmeta->active;
+        wmeta->active->prev = old_next;
+    }
+}
+
+
+
+static void shift_win_backward(struct WinMeta * wmeta)
+{
+    if (wmeta->active == wmeta->chain_start)
+    {
+        wmeta->chain_start = wmeta->active->next;
+        wmeta->chain_start->prev = 0;
+        wmeta->active->prev = wmeta->chain_end;
+        wmeta->active->prev->next = wmeta->active;
+        wmeta->chain_end = wmeta->active;
+        wmeta->chain_end->next = 0;
+    }
+    else
+    {
+        struct Win * old_prev = wmeta->active->prev;
+        struct Win * old_next = wmeta->active->next;
+        if (wmeta->chain_start == wmeta->active->prev)
+        {
+            wmeta->chain_start = wmeta->active;
+            wmeta->active->prev = 0;
+        }
+        else
+        {
+            wmeta->active->prev = old_prev->prev;
+            wmeta->active->prev->next = wmeta->active;
+        }
+        if (wmeta->chain_end == wmeta->active)
+        {
+            wmeta->chain_end = old_prev;
+        }
+        else
+        {
+            old_next->prev = old_prev;
+        }
+        old_prev->next = old_next;
+        old_prev->prev = wmeta->active;
+        wmeta->active->next = old_prev;
+    }
+}
+
+
+
+extern uint8_t init_win_meta(WINDOW * screen, struct WinMeta * wmeta)
+{
+    wmeta->screen              = screen;
+    wmeta->padframe.size.y     = getmaxy(screen);
+    wmeta->padframe.size.x     = getmaxx(screen);
+    wmeta->chain_start         = 0;
+    wmeta->chain_end           = 0;
+    wmeta->pad_offset          = 0;
+    WINDOW * test;
+    test = newpad(wmeta->padframe.size.y, 1);
+    if (NULL == test)
+    {
+        return 1;
+    }
+    wmeta->padframe.curses_win = test;
+    wmeta->active              = 0;
+    return 0;
 }
 
 
 
 extern struct Win init_win(struct WinMeta * wmeta, char * title,
+                           uint16_t height, uint16_t width,
                            void * data, void * func)
 {
     struct Win w;
@@ -277,16 +387,30 @@ extern struct Win init_win(struct WinMeta * wmeta, char * title,
     w.next             = 0;
     w.frame.curses_win = 0;
     w.title            = title;
-    w.frame.size.x     = 20;
-    w.frame.size.y     = wmeta->padframe.size.y - 1;
     w.data             = data;
     w.draw             = func;
+    if (width > 0)
+    {
+        w.frame.size.x = width;
+    }
+    else
+    {
+        w.frame.size.x = 1;
+    }
+    if (height > 0 && height <= wmeta->padframe.size.y - 1)
+    {
+        w.frame.size.y = height;
+    }
+    else
+    {
+        w.frame.size.y = wmeta->padframe.size.y - 1;
+    }
     return w;
 }
 
 
 
-extern void append_win(struct WinMeta * wmeta, struct Win * w)
+extern uint8_t append_win(struct WinMeta * wmeta, struct Win * w)
 {
     if (0 != wmeta->chain_start)
     {
@@ -299,12 +423,12 @@ extern void append_win(struct WinMeta * wmeta, struct Win * w)
         wmeta->chain_start = w;
     }
     wmeta->chain_end = w;
-    update_wins(wmeta, w);
+    return update_wins(wmeta, w);
 }
 
 
 
-extern void suspend_win(struct WinMeta * wmeta, struct Win * w)
+extern uint8_t suspend_win(struct WinMeta * wmeta, struct Win * w)
 {
     destroy_win(w);
 
@@ -324,10 +448,13 @@ extern void suspend_win(struct WinMeta * wmeta, struct Win * w)
         {
             wmeta->active = w->next;
         }
-        update_wins(wmeta, w->next); /* Positioning of successor windows may  */
-        pad_refitted = 1;            /* be affected / need correction. Note   */
-    }                                /* that update_wins() already refits the */
-    else                             /* pad, voiding later need for that.     */
+        if (0 != update_wins(wmeta, w->next)) /* Positioning of successor    */
+        {                                     /* windows may be affected /    */
+            return 1;                         /* need correction.             */
+        }                                     /* Note that update_wins()      */
+        pad_refitted = 1;                     /* already refits the  pad,     */
+    }                                         /* voiding later need for that. */
+    else
     {
         wmeta->chain_end = w->prev;
         if (wmeta->active == w)
@@ -341,8 +468,9 @@ extern void suspend_win(struct WinMeta * wmeta, struct Win * w)
 
     if (0 == pad_refitted)
     {
-        refit_pad(wmeta);
+        return refit_pad(wmeta);
     }
+    return 0;
 }
 
 
@@ -360,17 +488,18 @@ extern void reset_pad_offset(struct WinMeta * wmeta, uint16_t new_offset)
 
 
 
-extern void resize_active_win(struct WinMeta * wmeta, struct yx_uint16 size)
+extern uint8_t resize_active_win(struct WinMeta * wmeta, struct yx_uint16 size)
 {
     if (0 != wmeta->active
         && size.x > 0 && size.y > 0
         && size.y < wmeta->padframe.size.y)
     {
         wmeta->active->frame.size = size;
-        update_wins(wmeta, wmeta->chain_start);   /* Positioning of successor */
-    }                                             /* windows may be affected. */
-}
-
+        return update_wins(wmeta, wmeta->chain_start); /* Succeeding windows' */
+    }                                                  /* positioning may be  */
+    return 0;                                          /* affected.           */
+}                                                      /* TODO: Why start at  */
+                                                       /* chain_start then?   */
 
 
 extern void cycle_active_win(struct WinMeta * wmeta, char dir)
@@ -404,11 +533,11 @@ extern void cycle_active_win(struct WinMeta * wmeta, char dir)
 
 
 
-extern void shift_active_win(struct WinMeta * wmeta, char dir)
+extern uint8_t shift_active_win_old(struct WinMeta * wmeta, char dir)
 {
-    if (0 != wmeta->active                        /* No shifting with less    */
-        && wmeta->chain_start != wmeta->chain_end /* than one window visible. */
-        && (dir == 'f' || dir == 'b'))
+    if (0 != wmeta->active                         /* No shifting with less   */
+        && wmeta->chain_start != wmeta->chain_end  /* than 2 windows visible. */
+        && (dir == 'f' || dir == 'b'))             /* and wrong dir char.     */
     {
         struct Win * w_shift = wmeta->active, * w_p, * w_p_next;
 
@@ -429,6 +558,10 @@ extern void shift_active_win(struct WinMeta * wmeta, char dir)
             i_max++;
         }
         struct Win ** wins = malloc(i_max * sizeof(struct Win *));
+        if (NULL == wins)
+        {
+            return 1;
+        }
         for (i = 0, w_p = wmeta->chain_start;
              i < i_max;
              i++)
@@ -444,19 +577,31 @@ extern void shift_active_win(struct WinMeta * wmeta, char dir)
         {
             if (dir == 'f')
             {
-                append_win(wmeta, w_shift);
+                if (0 != append_win(wmeta, w_shift))
+                {
+                    return 1;
+                }
                 for (i = 0; i < i_max - 1; i++)
                 {
-                    append_win(wmeta, wins[i]);
+                    if (0 != append_win(wmeta, wins[i]))
+                    {
+                        return 1;
+                    }
                 }
             }
             else
             {
                 for (i = 1; i < i_max; i++)
                 {
-                    append_win(wmeta, wins[i]);
+                    if (0 != append_win(wmeta, wins[i]))
+                    {
+                        return 1;
+                    }
+                }
+                if (0 != append_win(wmeta, w_shift))
+                {
+                    return 1;
                 }
-                append_win(wmeta, w_shift);
             }
         }
         else
@@ -466,13 +611,19 @@ extern void shift_active_win(struct WinMeta * wmeta, char dir)
                 if (   (dir == 'f' && w_shift == wins[i])
                     || (dir == 'b' && w_shift == wins[i+1]))
                 {
-                    append_win(wmeta, wins[i+1]);
-                    append_win(wmeta, wins[i]);
+                    if (   0 != append_win(wmeta, wins[i+1])
+                        || 0 != append_win(wmeta, wins[i]))
+                    {
+                        return 1;
+                    }
                     i++;
                 }
                 else
                 {
-                    append_win(wmeta, wins[i]);
+                    if (0 != append_win(wmeta, wins[i]))
+                    {
+                        return 1;
+                    }
                 }
             }
         }
@@ -480,6 +631,28 @@ extern void shift_active_win(struct WinMeta * wmeta, char dir)
 
         wmeta->active = w_shift;  /* Otherwise lastly appended win is active. */
     }
+    return 0;
+}
+
+
+
+extern void shift_active_win(struct WinMeta * wmeta, char dir)
+{
+    if (   0 == wmeta->active
+        || wmeta->chain_start == wmeta->chain_end
+        || (dir != 'f' && dir != 'b'))
+    {
+        return;
+    }
+    if ('f' == dir)
+    {
+        shift_win_forward(wmeta);
+    }
+    else
+    {
+        shift_win_backward(wmeta);
+    }
+    update_wins(wmeta, wmeta->chain_start);
 }
 
 
@@ -527,8 +700,8 @@ extern void draw_all_wins(struct WinMeta * wmeta)
 
 
 
-extern void draw_scroll_hint(struct Frame * frame, uint16_t pos, uint32_t dist,
-                             char dir)
+extern uint8_t draw_scroll_hint(struct Frame * frame, uint16_t pos,
+                                uint32_t dist, char dir)
 {
     /* Decide on alignment (vertical/horizontal?), thereby scroll hint text. */
     char * more = "more";
@@ -543,6 +716,10 @@ extern void draw_scroll_hint(struct Frame * frame, uint16_t pos, uint32_t dist,
     }
     char * scrolldsc = malloc((4 * sizeof(char)) + strlen(more) + strlen(unit)
                               + 10);                /* 10 = uint32 max strlen */
+    if (NULL == scrolldsc)
+    {
+        return 1;
+    }
     sprintf(scrolldsc, " %d %s %s ", dist, more, unit);
 
     /* Decide on offset of the description text inside the scroll hint line. */
@@ -575,4 +752,5 @@ extern void draw_scroll_hint(struct Frame * frame, uint16_t pos, uint32_t dist,
     }
 
     free(scrolldsc);
+    return 0;
 }