Add hidden key handler window

After BUG#6 had apparently been closed with the screen locks introduced
in commit a80f8dcf2c6eb3b54658218bc081ee9694204dd5, some people still
had problems with random characters appearing in the notification bar.
This was obviously caused by wgetch() refreshing the screen if the
status panel was changed. From wgetch(3):

    If the window is not a pad, and it has been moved or modified since
    the last call to wrefresh, wrefresh will be called before another
    character is read.

Since the wgetch(3) isn't thread-safe, there were race conditions
between the notification bar thread drawing to the notification bar and
wgetch() updating the screen. Introduce a (hidden) window that handles
all key presses and never gets changed in order to avoid this.

Also, call wins_wrefresh() explicitly in status_mesg(), since we can no
longer rely on wgetch() updating windows automatically.

Fixes reopened BUG#6. Note that this is a hotfix -- FR#26 has been
opened to ensure we fix this properly in the next major release.

Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>

1 files changed, 6 insertions, 6 deletions

diff --git a/src/io.c b/src/io.c
index bef4c55..1572a49 100644
--- a/src/io.c
+++ b/src/io.c
@@ -175,7 +175,7 @@ static FILE *get_export_stream(enum export_type type)
     stream = fopen(stream_name, "w");
     if (stream == NULL) {
       status_mesg(wrong_name, press_enter);
-      wgetch(win[STA].p);
+      wgetch(win[KEY].p);
     }
   }
   mem_free(stream_name);
@@ -421,7 +421,7 @@ void io_save_cal(enum save_display display)
   /* Print a message telling data were saved */
   if (ui_mode == UI_CURSES && conf.system_dialogs) {
     status_mesg(save_success, enter);
-    wgetch(win[STA].p);
+    wgetch(win[KEY].p);
   }
 
   pthread_mutex_unlock(&io_save_mutex);
@@ -893,7 +893,7 @@ void io_startup_screen(int no_data_file)
     status_mesg(_("Welcome to Calcurse. Missing data files were created."),
                 enter);
 
-  wgetch(win[STA].p);
+  wgetch(win[KEY].p);
 }
 
 /* Export calcurse data. */
@@ -928,7 +928,7 @@ void io_export_data(enum export_type type)
 
   if (conf.system_dialogs && ui_mode == UI_CURSES) {
     status_mesg(success, enter);
-    wgetch(win[STA].p);
+    wgetch(win[KEY].p);
   }
 }
 
@@ -952,7 +952,7 @@ static FILE *get_import_stream(enum import_type type)
     stream = fopen(stream_name, "r");
     if (stream == NULL) {
       status_mesg(wrong_file, press_enter);
-      wgetch(win[STA].p);
+      wgetch(win[KEY].p);
     }
   }
   mem_free(stream_name);
@@ -1030,7 +1030,7 @@ void io_import_data(enum import_type type, const char *stream_name)
              stats_str[1], stats_str[2], stats_str[3],
              _("Press [ENTER] to continue"));
     status_mesg(read, stat);
-    wgetch(win[STA].p);
+    wgetch(win[KEY].p);
   } else if (ui_mode == UI_CMDLINE) {
     printf(proc_report, stats.lines);
     printf("\n%s / %s / %s / %s\n", stats_str[0], stats_str[1],