mild refactoring, clean up windows properly upon exit

Makefile

@@ -21,7 +21,7 @@ EXECUTABLE := $(TARGET)
 
 CC      := gcc
 LD      := gcc
-CFLAGS  := -std=c99 -pedantic -Wall -Wextra -O2 -ggdb
+CFLAGS  := -std=c99 -pedantic -Wall -Wextra -Wno-unused-parameter -O2 -ggdb
 LDFLAGS := $(CFLAGS) -lncurses
 
 all: $(EXECUTABLE)

src/bbs.c

@@ -3,20 +3,47 @@
 #include <curses.h>
 #include <string.h>
 #include <locale.h>
-
 #include "globals.h"
 #include "screens.h"
 
-const unsigned int GETCH_TIMEOUT = 10; /* in ms */
-
 char error_message[MAX_ERROR_MESSAGE_SIZE];
 
+unsigned int terminal_width;
+unsigned int terminal_height;
+unsigned int old_terminal_width = 0;
+unsigned int old_terminal_height = 0;
+
+enum Status status = STATUS_NEED_REFRESH; /* refresh screens[active_screen].window on first loop */
+char input = ' ';
+
+unsigned int screen_before_error = HOME;
+enum ActiveScreen active_screen = HOME;
+
 static void draw_screen (struct Screen *screen, char *input) {
     screen->draw_screen(screen, input);
-    box(screen->win, 0, 0);
+    wattron(screen->win, A_STANDOUT);
-    /* wattron(screen->win, A_STANDOUT); */
     mvwprintw(screen->win, 0, 1, " %s ", screen->name);
-    /* wattroff(screen->win, A_STANDOUT); */
+    wattroff(screen->win, A_STANDOUT);
+}
+
+static void resize_active_screen() {
+    wclear(screens[active_screen].win);
+    wresize(screens[active_screen].win, terminal_height, terminal_width);
+    mvwin(screens[active_screen].win, 0, 0);
+    wrefresh(screens[active_screen].win);
+    old_terminal_width = terminal_width;
+    old_terminal_height = terminal_height;
+
+    if (terminal_width < 80 || terminal_height < 24) {
+        if (active_screen != ERROR) {
+            strcpy(error_message, "This program expects at least 80 rows by 24 columns.");
+            screen_before_error = active_screen;
+        };
+        active_screen = ERROR;
+    } else if (active_screen != screen_before_error) {
+        active_screen = screen_before_error;
+    };
+    status = STATUS_NEED_REFRESH;
 }
 
 int main() {
@@ -24,24 +51,15 @@ int main() {
     initscr();                   /* start ncurses */
     noecho();                    /* hide keyboard input */
     curs_set(0);                 /* disable cursor */
+    keypad(stdscr, TRUE);        /* enable extra keys */
     timeout(GETCH_TIMEOUT);      /* set timeout for getch() */
-    setlocale(LC_CTYPE, ""); /* set locale, UTF8 support is enabled here */
+    setlocale(LC_CTYPE, "");     /* set locale to current locale */
-
-    unsigned int terminal_width;
-    unsigned int terminal_height;
-    unsigned int old_terminal_width = 0;
-    unsigned int old_terminal_height = 0;
-
-    enum Status status = STATUS_NEED_REFRESH; /* refresh screens[active_screen].window on first loop */
-    char input = ' ';
 
     strcpy(error_message, "");
 
     getmaxyx(stdscr, terminal_height, terminal_width);
 
-    unsigned int screen_before_error = HOME;
 
-    enum ActiveScreen active_screen = HOME;
 
     /* could be it's own init func within the screen's file. */
     screens[HOME].win = newwin(terminal_height, terminal_width, 0, 0);
@@ -57,27 +75,10 @@ int main() {
                 status = STATUS_NEED_REFRESH;
             };
         };
+
         getmaxyx(stdscr, terminal_height, terminal_width);
-
-        /* resize active screen's window */
         if (old_terminal_width != terminal_width || old_terminal_height != terminal_height) {
-            wclear(screens[active_screen].win);
+            resize_active_screen();
-            wresize(screens[active_screen].win, terminal_height, terminal_width);
-            mvwin(screens[active_screen].win, 0, 0);
-            wrefresh(screens[active_screen].win);
-            old_terminal_width = terminal_width;
-            old_terminal_height = terminal_height;
-
-            if (terminal_width < 80 || terminal_height < 24) {
-                if (active_screen != ERROR) {
-                    strcpy(error_message, "This program expects at least 80 rows by 24 columns.");
-                    screen_before_error = active_screen;
-                };
-                active_screen = ERROR;
-            } else if (active_screen != screen_before_error) {
-                active_screen = screen_before_error;
-            };
-            status = STATUS_NEED_REFRESH;
         };
 
         /* refresh active screen's window */
@@ -90,7 +91,8 @@ int main() {
     };
 
     /* clean up */
-    delwin(screens[active_screen].win); /* TODO: delete all windows in screens[] */
+    delwin(screens[0].win); /* could be in a destructor function in the screen's file */
+    delwin(screens[1].win);
     nocbreak();
     endwin();   /* ends curses mode */
     curs_set(1);

src/error.c

@@ -3,6 +3,8 @@
 #include <curses.h>
 #include "screens.h"
 
-void draw_error(struct Screen *screen, __attribute__((unused)) char *input)  {
+/* char *input is unused */
+void draw_error(struct Screen *screen, char *input) {
+    box(screen->win, 0, 0);
     mvwprintw(screen->win, 1, 2, "%s", error_message);
 }

src/globals.h

@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: MIT */
 
+#define GETCH_TIMEOUT 10          /* in ms */
 #define MAX_SCREEN_NAME_SIZE 20   /* size of the screen name array */
 #define MAX_ERROR_MESSAGE_SIZE 60 /* size of error_message array */
 

src/home.c

@@ -14,12 +14,13 @@ void draw_home(struct Screen *screen, char *input)  {
     "  \\__,_|_| |_| |_|_.__/ \\___|_|  |___/ .__/|_|\\__,_|\\___\\___(_)_| |_|\\___|\\__|\n"
     "                                     | |                                      \n"
     "                                     |_|                                      \n";
-
     mvwprintw(screen->win, 1, 2, "Thank you for visiting:");
     mvwprintw(screen->win, 2, 1, "%s", banner);
     mvwprintw(screen->win, 10, 1, "Your current input is: %s", input);
+    wclrtoeol(screen->win); /* clear rest of line to prevent longer inputs from sticking around */
     mvwprintw(screen->win, 11, 1, "Git hash: %s", GIT_HASH);
     mvwprintw(screen->win, 12, 1, "Git tag: %s", GIT_TAG);
     mvwprintw(screen->win, 13, 1, "CC version: %s", CC_VERSION);
     mvwprintw(screen->win, 14, 1, "Build date: %s", BUILD_DATE);
+    box(screen->win, 0, 0);
 }