ncurses 6.2 - patch 20210424

[ncurses.git] / ncurses / widechar / lib_get_wch.c

diff --git a/ncurses/widechar/lib_get_wch.c b/ncurses/widechar/lib_get_wch.c
index 68127e92e4136ad67dde972d38f7d7945fbff395..9f10d3bcf121063a791e411602c3801d699e5650 100644 (file)
--- a/ncurses/widechar/lib_get_wch.c
+++ b/ncurses/widechar/lib_get_wch.c
@@ -1,5 +1,6 @@
 /****************************************************************************
- * Copyright (c) 2002-2004,2005 Free Software Foundation, Inc.              *
+ * Copyright 2020,2021 Thomas E. Dickey                                     *
+ * Copyright 2002-2011,2016 Free Software Foundation, Inc.                  *
  *                                                                          *
  * Permission is hereby granted, free of charge, to any person obtaining a  *
  * copy of this software and associated documentation files (the            *
@@ -40,80 +41,81 @@
 #include <curses.priv.h>
 #include <ctype.h>
 
-MODULE_ID("$Id: lib_get_wch.c,v 1.12 2005/08/13 18:51:43 tom Exp $")
-
-#if HAVE_MBTOWC && HAVE_MBLEN
-#define reset_mbytes(state) mblen(NULL, 0), mbtowc(NULL, NULL, 0)
-#define count_mbytes(buffer,length,state) mblen(buffer,length)
-#define check_mbytes(wch,buffer,length,state) \
-       (int) mbtowc(&wch, buffer, length)
-#define state_unused
-#elif HAVE_MBRTOWC && HAVE_MBRLEN
-#define reset_mbytes(state) init_mb(state)
-#define count_mbytes(buffer,length,state) mbrlen(buffer,length,&state)
-#define check_mbytes(wch,buffer,length,state) \
-       (int) mbrtowc(&wch, buffer, length, &state)
-#else
-make an error
-#endif
+MODULE_ID("$Id: lib_get_wch.c,v 1.26 2021/04/17 16:12:54 tom Exp $")
 
 NCURSES_EXPORT(int)
 wget_wch(WINDOW *win, wint_t *result)
 {
+    SCREEN *sp;
     int code;
-    char buffer[(MB_LEN_MAX * 9) + 1]; /* allow some redundant shifts */
-    int status;
-    size_t count = 0;
-    unsigned long value;
-    wchar_t wch;
+    int value = 0;
 #ifndef state_unused
     mbstate_t state;
 #endif
 
-    T((T_CALLED("wget_wch(%p)"), win));
+    T((T_CALLED("wget_wch(%p)"), (void *) win));
 
     /*
      * We can get a stream of single-byte characters and KEY_xxx codes from
      * _nc_wgetch(), while we want to return a wide character or KEY_xxx code.
      */
-    for (;;) {
-       T(("reading %d of %d", count + 1, sizeof(buffer)));
-       code = _nc_wgetch(win, &value, TRUE);
-       if (code == ERR) {
-           break;
-       } else if (code == KEY_CODE_YES) {
-           /*
-            * If we were processing an incomplete multibyte character, return
-            * an error since we have a KEY_xxx code which interrupts it.  For
-            * some cases, we could improve this by writing a new version of
-            * lib_getch.c(!), but it is not clear whether the improvement
-            * would be worth the effort.
-            */
-           if (count != 0) {
-               ungetch((int) value);
+    _nc_lock_global(curses);
+    sp = _nc_screen_of(win);
+
+    if (sp != 0) {
+       size_t count = 0;
+
+       for (;;) {
+           char buffer[(MB_LEN_MAX * 9) + 1];  /* allow some redundant shifts */
+
+           T(("reading %d of %d", (int) count + 1, (int) sizeof(buffer)));
+           code = _nc_wgetch(win, &value, TRUE EVENTLIST_2nd((_nc_eventlist
+                                                              *) 0));
+           if (code == ERR) {
+               break;
+           } else if (code == KEY_CODE_YES) {
+               /*
+                * If we were processing an incomplete multibyte character,
+                * return an error since we have a KEY_xxx code which
+                * interrupts it.  For some cases, we could improve this by
+                * writing a new version of lib_getch.c(!), but it is not clear
+                * whether the improvement would be worth the effort.
+                */
+               if (count != 0) {
+                   safe_ungetch(SP_PARM, value);
+                   code = ERR;
+               }
+               break;
+           } else if (count + 1 >= sizeof(buffer)) {
+               safe_ungetch(SP_PARM, value);
                code = ERR;
-           }
-           break;
-       } else if (count + 1 >= sizeof(buffer)) {
-           ungetch((int) value);
-           code = ERR;
-           break;
-       } else {
-           buffer[count++] = UChar(value);
-           reset_mbytes(state);
-           status = count_mbytes(buffer, count, state);
-           if (status >= 0) {
+               break;
+           } else {
+               int status;
+
+               buffer[count++] = (char) UChar(value);
                reset_mbytes(state);
-               if (check_mbytes(wch, buffer, count, state) != status) {
-                   code = ERR; /* the two calls should match */
-                   ungetch((int) value);
+               status = count_mbytes(buffer, count, state);
+               if (status >= 0) {
+                   wchar_t wch;
+                   reset_mbytes(state);
+                   if (check_mbytes(wch, buffer, count, state) != status) {
+                       code = ERR;     /* the two calls should match */
+                       safe_ungetch(SP_PARM, value);
+                   }
+                   value = wch;
+                   break;
                }
-               value = wch;
-               break;
            }
        }
+    } else {
+       code = ERR;
     }
-    *result = value;
-    T(("result %#lo", value));
+
+    if (result != 0)
+       *result = (wint_t) value;
+
+    _nc_unlock_global(curses);
+    T(("result %#o", value));
     returnCode(code);
 }