Moved allocation and freeing of Win structs into windows library.

diff --git a/src/rexit.c b/src/rexit.c
index 204961331c889553bb9b247ef86bbd9d7b675cba..85e2bfc328798fdb3085bfa9676118084073176a 100644 (file)
--- a/src/rexit.c
+++ b/src/rexit.c
@@ -10,8 +10,7 @@
 #include "map.h" /* for Map struct */
 #include "keybindings.h" /* for KeysWinData, KeyBinding structs */
 #include "command_db.h" /* for free_command_db() */
-#include "windows.h" /* for Win struct */
-#include "wincontrol.h" /* for free_win() */
+#include "windows.h" /* for Win struct, free_win() */
 
 
 

diff --git a/src/wincontrol.c b/src/wincontrol.c
index 793b75e2d6205b896656da6ca6cf077189b77e6d..74c9c2a297d6fc9e0fcca3c1afdd83855cd56e2c 100644 (file)
--- a/src/wincontrol.c
+++ b/src/wincontrol.c
@@ -15,14 +15,6 @@
 
 
 
-extern void free_win(struct Win * win)
-{
-    free(win->_title);
-    free(win);
-}
-
-
-
 extern struct Win * init_win_from_file(struct World * world, char * w_name,
                                        void (* f) (struct Win *))
 {
@@ -54,10 +46,10 @@ extern struct Win * init_win_from_file(struct World * world, char * w_name,
     exit_err(fclose(file), world, err);
 
     struct WinMeta * wmeta = world->wins.meta;
-    struct Win * w = malloc(sizeof(struct Win));
-    init_win(wmeta, w, title, height, width, world, f);
+    struct Win * wp; // = malloc(sizeof(struct Win));
+    init_win(wmeta, &wp, title, height, width, world, f);
     free(title);
-    return w;
+    return wp;
 }
 
 

diff --git a/src/wincontrol.h b/src/wincontrol.h
index cb7d0ba85b7cb932b638e56244c1076da911e7c3..29a6ddfef472cb3da1fae9594b9da786c0674d9f 100644 (file)
--- a/src/wincontrol.h
+++ b/src/wincontrol.h
@@ -16,12 +16,6 @@ struct World;
 
 
 
-/* Free allocated memory for an initialized Win struct. */
-extern void free_win(struct Win * win);
-
-
-
-
 /* Wrapper around init_win() that reads the desired window size and title from a
  * file at the path prefixing the provided win name "w_name" with
  * "config/windows/". "f"() is the window drawing function (Win._draw()).

diff --git a/src/windows.c b/src/windows.c
index bb78a8bee0624ea791aadffc5084201b00abeefb..f75544bd4753fa06d64f46382499a8b92f7fbfec 100644 (file)
--- a/src/windows.c
+++ b/src/windows.c
@@ -388,10 +388,15 @@ extern uint8_t init_win_meta(WINDOW * screen, struct WinMeta * wmeta)
 
 
 
-extern uint8_t init_win(struct WinMeta * wmeta, struct Win * w, char * title,
+extern uint8_t init_win(struct WinMeta * wmeta, struct Win ** wp, char * title,
                         int16_t height, int16_t width,
                         void * data, void * func)
 {
+    struct Win * w = malloc(sizeof(struct Win));
+    if (NULL == w)
+    {
+        return 1;
+    }
     w->_prev             = 0;
     w->_next             = 0;
     w->frame.curses_win  = 0;
@@ -427,11 +432,20 @@ extern uint8_t init_win(struct WinMeta * wmeta, struct Win * w, char * title,
     {
         w->frame.size.y = wmeta->padframe.size.y - 1;
     }
+    *wp = w;
     return 0;
 }
 
 
 
+extern void free_win(struct Win * win)
+{
+    free(win->_title);
+    free(win);
+}
+
+
+
 extern uint8_t append_win(struct WinMeta * wmeta, struct Win * w)
 {
     if (0 != wmeta->_chain_start)

diff --git a/src/windows.h b/src/windows.h
index 0e3f79bfd63dba060fb15fc3a7b27a9b8aa3c872..a40835d9c591925b3d080b8747495c9b21a5c3ed 100644 (file)
--- a/src/windows.h
+++ b/src/windows.h
@@ -113,12 +113,17 @@ extern uint8_t init_win_meta(WINDOW * screen, struct WinMeta * wmeta);
  *
  * Other members of the Win struct are initialized to 0.
  */
-extern uint8_t init_win(struct WinMeta * wmeta, struct Win * w, char * title,
+extern uint8_t init_win(struct WinMeta * wmeta, struct Win ** w, char * title,
                         int16_t height, int16_t width,
                         void * data, void * func);
 
 
 
+/* Free allocated memory for an initialized Win struct. */
+extern void free_win(struct Win * win);
+
+
+
 /* Append/suspend window "w" to/from chain of visible windows below "wmeta".
  * Appended windows will become active. Suspended active windows will move the
  * active window selection to their successor in the window chain or, failing