ncurses 5.7 - patch 20100724
[ncurses.git] / test / ncurses.c
index f351122cfa9e40081b9ac2a55471c69db9e270d8..ddb732ef42b32fd9b7b83ec24126a26404999f19 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1998-2008,2009 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2009,2010 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,7 +40,7 @@ AUTHOR
    Author: Eric S. Raymond <esr@snark.thyrsus.com> 1993
            Thomas E. Dickey (beginning revision 1.27 in 1996).
 
-$Id: ncurses.c,v 1.347 2009/10/03 22:01:56 tom Exp $
+$Id: ncurses.c,v 1.360 2010/07/24 18:04:58 tom Exp $
 
 ***************************************************************************/
 
@@ -121,22 +121,22 @@ extern unsigned _nc_tracing;
 #if HAVE_WCSRTOMBS
 #define count_wchars(src, len, state)      wcsrtombs(0,   &src, len, state)
 #define trans_wchars(dst, src, len, state) wcsrtombs(dst, &src, len, state)
-#define reset_wchars(state) memset(&state, 0, sizeof(state))
+#define reset_wchars(state) init_mb(state)
 #elif HAVE_WCSTOMBS && HAVE_MBTOWC && HAVE_MBLEN
 #define count_wchars(src, len, state)      wcstombs(0,   src, len)
 #define trans_wchars(dst, src, len, state) wcstombs(dst, src, len)
-#define reset_wchars(state) mblen(NULL, 0), mbtowc(NULL, NULL, 0)
+#define reset_wchars(state) IGNORE_RC(mblen(NULL, 0)), IGNORE_RC(mbtowc(NULL, NULL, 0))
 #define state_unused
 #endif
 
 #if HAVE_MBSRTOWCS
 #define count_mbytes(src, len, state)      mbsrtowcs(0,   &src, len, state)
 #define trans_mbytes(dst, src, len, state) mbsrtowcs(dst, &src, len, state)
-#define reset_mbytes(state) memset(&state, 0, sizeof(state))
+#define reset_mbytes(state) init_mb(state)
 #elif HAVE_MBSTOWCS && HAVE_MBTOWC && HAVE_MBLEN
 #define count_mbytes(src, len, state)      mbstowcs(0,   src, len)
 #define trans_mbytes(dst, src, len, state) mbstowcs(dst, src, len)
-#define reset_mbytes(state) mblen(NULL, 0), mbtowc(NULL, NULL, 0)
+#define reset_mbytes(state) IGNORE_RC(mblen(NULL, 0)), IGNORE_RC(mbtowc(NULL, NULL, 0))
 #define state_unused
 #endif
 
@@ -171,14 +171,14 @@ static void
 do_h_line(int y, int x, chtype c, int to)
 {
     if ((to) > (x))
-       mvhline(y, x, c, (to) - (x));
+       MvHLine(y, x, c, (to) - (x));
 }
 
 static void
 do_v_line(int y, int x, chtype c, int to)
 {
     if ((to) > (y))
-       mvvline(y, x, c, (to) - (y));
+       MvVLine(y, x, c, (to) - (y));
 }
 
 static void
@@ -505,7 +505,7 @@ mouse_decode(MEVENT const *ep)
 {
     static char buf[80 + (5 * 10) + (32 * 15)];
 
-    (void) sprintf(buf, "id %2d  at (%2d, %2d, %2d) state %4lx = {",
+    (void) sprintf(buf, "id %2d at (%2d, %2d, %d) state %4lx = {",
                   ep->id, ep->x, ep->y, ep->z, (unsigned long) ep->bstate);
 
 #define SHOW(m, s) if ((ep->bstate & m)==m) {strcat(buf,s); strcat(buf, ", ");}
@@ -567,6 +567,42 @@ mouse_decode(MEVENT const *ep)
     (void) strcat(buf, "}");
     return (buf);
 }
+
+static void
+show_mouse(WINDOW *win)
+{
+    int y, x;
+    MEVENT event;
+    bool outside;
+    bool show_loc;
+
+    getmouse(&event);
+    outside = !wenclose(win, event.y, event.x);
+
+    if (outside) {
+       (void) wstandout(win);
+       waddstr(win, "KEY_MOUSE");
+       (void) wstandend(win);
+    } else {
+       waddstr(win, "KEY_MOUSE");
+    }
+    wprintw(win, ", %s", mouse_decode(&event));
+
+    if (outside)
+       win = stdscr;
+
+    show_loc = wmouse_trafo(win, &event.y, &event.x, FALSE);
+
+    if (show_loc) {
+       getyx(win, y, x);
+       wmove(win, event.y, event.x);
+       waddch(win, '*');
+       wmove(win, y, x);
+    }
+
+    if (outside)
+       wnoutrefresh(win);
+}
 #endif /* NCURSES_MOUSE_VERSION */
 
 /****************************************************************************
@@ -630,7 +666,7 @@ wgetch_help(WINDOW *win, GetchFlags flags)
                   && (flags[UChar(*help[n])] != FALSE));
        if (flg)
            (void) standout();
-       mvprintw(row, col, "%s", help[n]);
+       MvPrintw(row, col, "%s", help[n]);
        if (col == 0)
            clrtoeol();
        if (flg)
@@ -833,15 +869,7 @@ wgetch_test(unsigned level, WINDOW *win, int delay)
            wprintw(win, "Key pressed: %04o ", c);
 #ifdef NCURSES_MOUSE_VERSION
            if (c == KEY_MOUSE) {
-               int y, x;
-               MEVENT event;
-
-               getmouse(&event);
-               wprintw(win, "KEY_MOUSE, %s", mouse_decode(&event));
-               getyx(win, y, x);
-               move(event.y, event.x);
-               addch('*');
-               wmove(win, y, x);
+               show_mouse(win);
            } else
 #endif /* NCURSES_MOUSE_VERSION */
            if (c >= KEY_MIN) {
@@ -1006,7 +1034,7 @@ wget_wch_test(unsigned level, WINDOW *win, int delay)
     int incount = 0;
     GetchFlags flags;
     bool blocking = (delay < 0);
-    int y, x, code;
+    int code;
     char *temp;
 
     init_getch(win, flags);
@@ -1103,14 +1131,7 @@ wget_wch_test(unsigned level, WINDOW *win, int delay)
            wprintw(win, "Key pressed: %04o ", (int) c);
 #ifdef NCURSES_MOUSE_VERSION
            if (c == KEY_MOUSE) {
-               MEVENT event;
-
-               getmouse(&event);
-               wprintw(win, "KEY_MOUSE, %s", mouse_decode(&event));
-               getyx(win, y, x);
-               move(event.y, event.x);
-               addch('*');
-               wmove(win, y, x);
+               show_mouse(win);
            } else
 #endif /* NCURSES_MOUSE_VERSION */
            if (code == KEY_CODE_YES) {
@@ -1215,29 +1236,29 @@ attr_legend(WINDOW *helpwin)
     int row = 1;
     int col = 1;
 
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "ESC to exit.");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "^L repaints.");
     ++row;
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "Modify the test strings:");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  A digit sets gaps on each side of displayed attributes");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  </> shifts the text left/right. ");
     ++row;
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "Toggles:");
     if (use_colors) {
-       mvwprintw(helpwin, row++, col,
+       MvWPrintw(helpwin, row++, col,
                  "  f/F/b/F toggle foreground/background background color");
-       mvwprintw(helpwin, row++, col,
+       MvWPrintw(helpwin, row++, col,
                  "  t/T     toggle text/background color attribute");
     }
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  a/A     toggle ACS (alternate character set) mapping");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row, col,
              "  v/V     toggle video attribute to combine with each line");
 }
 
@@ -1339,9 +1360,9 @@ show_attr(int row, int skip, bool arrow, chtype attr, const char *name)
     chtype test = attr & (chtype) (~A_ALTCHARSET);
 
     if (arrow)
-       mvprintw(row, 5, "-->");
-    mvprintw(row, 8, "%s mode:", name);
-    mvprintw(row, 24, "|");
+       MvPrintw(row, 5, "-->");
+    MvPrintw(row, 8, "%s mode:", name);
+    MvPrintw(row, 24, "|");
     if (skip)
        printw("%*s", skip, " ");
     /*
@@ -1530,7 +1551,7 @@ attr_test(void)
        erase();
 
        box(stdscr, 0, 0);
-       mvaddstr(0, 20, "Character attribute test display");
+       MvAddStr(0, 20, "Character attribute test display");
 
        for (j = 0; j < SIZEOF(attrs_to_test); ++j) {
            bool arrow = (j == k);
@@ -1541,10 +1562,10 @@ attr_test(void)
                            attrs_to_test[j].name);
        }
 
-       mvprintw(row, 8,
+       MvPrintw(row, 8,
                 "This terminal does %shave the magic-cookie glitch",
                 get_xmc() > -1 ? "" : "not ");
-       mvprintw(row + 1, 8, "Enter '?' for help.");
+       MvPrintw(row + 1, 8, "Enter '?' for help.");
        show_color_attr(fg, bg, tx);
        printw("  ACS (%d)", ac != 0);
 
@@ -1627,9 +1648,9 @@ wide_show_attr(int row, int skip, bool arrow, chtype attr, short pair, const cha
     chtype test = attr & ~WA_ALTCHARSET;
 
     if (arrow)
-       mvprintw(row, 5, "-->");
-    mvprintw(row, 8, "%s mode:", name);
-    mvprintw(row, 24, "|");
+       MvPrintw(row, 5, "-->");
+    MvPrintw(row, 8, "%s mode:", name);
+    MvPrintw(row, 24, "|");
     if (skip)
        printw("%*s", skip, " ");
 
@@ -1653,10 +1674,10 @@ wide_show_attr(int row, int skip, bool arrow, chtype attr, short pair, const cha
        attr_t old_attr;
        short old_pair;
 
-       attr_get(&old_attr, &old_pair, 0);
-       attr_set(attr, pair, 0);
+       (void) attr_get(&old_attr, &old_pair, 0);
+       (void) attr_set(attr, pair, 0);
        addwstr(wide_attr_test_string);
-       attr_set(old_attr, old_pair, 0);
+       (void) attr_set(old_attr, old_pair, 0);
     }
     if (skip)
        printw("%*s", skip, " ");
@@ -1802,7 +1823,7 @@ wide_attr_test(void)
        erase();
 
        box_set(stdscr, 0, 0);
-       mvaddstr(0, 20, "Character attribute test display");
+       MvAddStr(0, 20, "Character attribute test display");
 
        for (j = 0; j < SIZEOF(attrs_to_test); ++j) {
            row = wide_show_attr(row, n, j == k,
@@ -1813,10 +1834,10 @@ wide_attr_test(void)
                                 attrs_to_test[j].name);
        }
 
-       mvprintw(row, 8,
+       MvPrintw(row, 8,
                 "This terminal does %shave the magic-cookie glitch",
                 get_xmc() > -1 ? "" : "not ");
-       mvprintw(row + 1, 8, "Enter '?' for help.");
+       MvPrintw(row + 1, 8, "Enter '?' for help.");
        show_color_attr(fg, bg, tx);
        printw("  ACS (%d)", ac != 0);
 
@@ -1882,35 +1903,35 @@ color_legend(WINDOW *helpwin, bool wide)
     int row = 1;
     int col = 1;
 
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "ESC to exit.");
     ++row;
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "Use up/down arrow to scroll through the display if it is");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "longer than one screen. Control/N and Control/P can be used");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "in place of up/down arrow.  Use pageup/pagedown to scroll a");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "full screen; control/B and control/F can be used here.");
     ++row;
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "Toggles:");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  a/A     toggle altcharset off/on");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  b/B     toggle bold off/on");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  n/N     toggle text/number on/off");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  r/R     toggle reverse on/off");
-    mvwprintw(helpwin, row++, col,
+    MvWPrintw(helpwin, row++, col,
              "  w/W     toggle width between 8/16 colors");
 #if USE_WIDEC_SUPPORT
     if (wide) {
-       mvwprintw(helpwin, row++, col,
+       MvWPrintw(helpwin, row++, col,
                  "Wide characters:");
-       mvwprintw(helpwin, row++, col,
+       MvWPrintw(helpwin, row, col,
                  "  x/X     toggle text between ASCII and wide-character");
     }
 #else
@@ -1974,11 +1995,11 @@ color_test(void)
                      min_colors ? " besides 'default'" : "");
 
        clrtobot();
-       (void) mvprintw(top + 1, 0,
-                       "%dx%d matrix of foreground/background colors, bold *%s*\n",
-                       row_limit,
-                       per_row,
-                       opt_bold ? "on" : "off");
+       MvPrintw(top + 1, 0,
+                "%dx%d matrix of foreground/background colors, bold *%s*\n",
+                row_limit,
+                per_row,
+                opt_bold ? "on" : "off");
 
        /* show color names/numbers across the top */
        for (i = 0; i < per_row; i++)
@@ -2117,7 +2138,6 @@ color_test(void)
 static void
 wide_color_test(void)
 {
-    int c;
     int i;
     int top = 0, width;
     int base_row = 0;
@@ -2178,11 +2198,11 @@ wide_color_test(void)
                      min_colors ? " besides 'default'" : "");
 
        clrtobot();
-       (void) mvprintw(top + 1, 0,
-                       "%dx%d matrix of foreground/background colors, bold *%s*\n",
-                       row_limit,
-                       per_row,
-                       opt_bold ? "on" : "off");
+       MvPrintw(top + 1, 0,
+                "%dx%d matrix of foreground/background colors, bold *%s*\n",
+                row_limit,
+                per_row,
+                opt_bold ? "on" : "off");
 
        /* show color names/numbers across the top */
        for (i = 0; i < per_row; i++)
@@ -2213,7 +2233,7 @@ wide_color_test(void)
                    }
                }
                addnwstr(buffer, width);
-               attr_set(A_NORMAL, 0, NULL);
+               (void) attr_set(A_NORMAL, 0, NULL);
 
                if ((i % per_row) == 0 && InxToFG(i) == min_colors) {
                    show_color_name(row, 0, InxToBG(i), opt_wide);
@@ -2224,7 +2244,7 @@ wide_color_test(void)
            }
        }
 
-       switch (c = wGetchar(stdscr)) {
+       switch (wGetchar(stdscr)) {
        case 'a':
            opt_acsc = FALSE;
            break;
@@ -2376,13 +2396,13 @@ color_edit(void)
     for (i = 0; i < max_colors; i++)
        init_pair((short) i, (short) COLOR_WHITE, (short) i);
 
-    mvprintw(LINES - 2, 0, "Number: %d", value);
+    MvPrintw(LINES - 2, 0, "Number: %d", value);
 
     do {
        short red, green, blue;
 
        attron(A_BOLD);
-       mvaddstr(0, 20, "Color RGB Value Editing");
+       MvAddStr(0, 20, "Color RGB Value Editing");
        attroff(A_BOLD);
 
        for (i = (short) top_color;
@@ -2391,7 +2411,7 @@ color_edit(void)
            char numeric[80];
 
            sprintf(numeric, "[%d]", i);
-           mvprintw(2 + i - top_color, 0, "%c %-8s:",
+           MvPrintw(2 + i - top_color, 0, "%c %-8s:",
                     (i == current ? '>' : ' '),
                     (i < (int) SIZEOF(the_color_names)
                      ? the_color_names[i] : numeric));
@@ -2425,9 +2445,9 @@ color_edit(void)
                   scaled_rgb(blue));
        }
 
-       mvaddstr(LINES - 3, 0,
+       MvAddStr(LINES - 3, 0,
                 "Use up/down to select a color, left/right to change fields.");
-       mvaddstr(LINES - 2, 0,
+       MvAddStr(LINES - 2, 0,
                 "Modify field by typing nnn=, nnn-, or nnn+.  ? for help.");
 
        move(2 + current - top_color, 0);
@@ -2520,6 +2540,8 @@ color_edit(void)
        case 'm':
            endwin();
            main_menu(FALSE);
+           for (i = 0; i < max_colors; i++)
+               init_pair((short) i, (short) COLOR_WHITE, (short) i);
            refresh();
            break;
 
@@ -2540,7 +2562,7 @@ color_edit(void)
        if (current - top_color >= page_size)
            top_color = current - (page_size - 1);
 
-       mvprintw(LINES - 1, 0, "Number: %d", value);
+       MvPrintw(LINES - 1, 0, "Number: %d", value);
        clrtoeol();
     } while
        (!isQuit(this_c));
@@ -2581,8 +2603,9 @@ slk_help(void)
        ,"[12345678] -- set label; labels are numbered 1 through 8"
        ,"e          -- erase stdscr (should not erase labels)"
        ,"s          -- test scrolling of shortened screen"
+       ,"v/V        -- cycle through video attributes"
 #if HAVE_SLK_COLOR
-       ,"F/B        -- cycle through foreground/background colors"
+       ,"F/f/B/b    -- cycle through foreground/background colors"
 #endif
        ,"ESC        -- return to main menu"
        ,""
@@ -2600,14 +2623,100 @@ slk_help(void)
     refresh();
 }
 
+/****************************************************************************
+ *
+ * Alternate character-set stuff
+ *
+ ****************************************************************************/
+/* *INDENT-OFF* */
+static struct {
+    chtype attr;
+    const char *name;
+} attrs_to_cycle[] = {
+    { A_NORMAL,                "normal" },
+    { A_BOLD,          "bold" },
+    { A_BLINK,         "blink" },
+    { A_REVERSE,       "reverse" },
+    { A_UNDERLINE,     "underline" },
+};
+/* *INDENT-ON* */
+
+static bool
+cycle_attr(int ch, unsigned *at_code, chtype *attr)
+{
+    bool result = TRUE;
+
+    switch (ch) {
+    case 'v':
+       if ((*at_code += 1) >= SIZEOF(attrs_to_cycle))
+           *at_code = 0;
+       break;
+    case 'V':
+       if (*at_code == 0)
+           *at_code = SIZEOF(attrs_to_cycle) - 1;
+       else
+           *at_code -= 1;
+       break;
+    default:
+       result = FALSE;
+       break;
+    }
+    if (result)
+       *attr = attrs_to_cycle[*at_code].attr;
+    return result;
+}
+
+static bool
+cycle_colors(int ch, int *fg, int *bg, short *pair)
+{
+    bool result = FALSE;
+
+    if (use_colors) {
+       result = TRUE;
+       switch (ch) {
+       case 'F':
+           if ((*fg -= 1) < 0)
+               *fg = COLORS - 1;
+           break;
+       case 'f':
+           if ((*fg += 1) >= COLORS)
+               *fg = 0;
+           break;
+       case 'B':
+           if ((*bg -= 1) < 0)
+               *bg = COLORS - 1;
+           break;
+       case 'b':
+           if ((*bg += 1) >= COLORS)
+               *bg = 0;
+           break;
+       default:
+           result = FALSE;
+           break;
+       }
+       if (result) {
+           *pair = (short) (*fg != COLOR_BLACK || *bg != COLOR_BLACK);
+           if (*pair != 0) {
+               *pair = 1;
+               if (init_pair(*pair, (short) *fg, (short) *bg) == ERR) {
+                   result = FALSE;
+               }
+           }
+       }
+    }
+    return result;
+}
+
 #if HAVE_SLK_COLOR
 static void
 call_slk_color(short fg, short bg)
 {
     init_pair(1, bg, fg);
     slk_color(1);
-    mvprintw(SLK_WORK, 0, "Colors %d/%d\n", fg, bg);
+    MvPrintw(SLK_WORK, 0, "Colors %d/%d\n", fg, bg);
     clrtoeol();
+    slk_touch();
+    slk_noutrefresh();
     refresh();
 }
 #endif
@@ -2619,9 +2728,12 @@ slk_test(void)
     int c, fmt = 1;
     char buf[9];
     char *s;
+    chtype attr = A_NORMAL;
+    unsigned at_code = 0;
 #if HAVE_SLK_COLOR
-    short fg = COLOR_BLACK;
-    short bg = COLOR_WHITE;
+    int fg = COLOR_BLACK;
+    int bg = COLOR_WHITE;
+    short pair = 0;
 #endif
 
     c = CTRL('l');
@@ -2637,7 +2749,7 @@ slk_test(void)
        case CTRL('l'):
            erase();
            attron(A_BOLD);
-           mvaddstr(0, 20, "Soft Key Exerciser");
+           MvAddStr(0, 20, "Soft Key Exerciser");
            attroff(A_BOLD);
 
            slk_help();
@@ -2652,7 +2764,7 @@ slk_test(void)
            break;
 
        case 's':
-           mvprintw(SLK_WORK, 0, "Press Q to stop the scrolling-test: ");
+           MvPrintw(SLK_WORK, 0, "Press Q to stop the scrolling-test: ");
            while ((c = Getchar()) != 'Q' && (c != ERR))
                addch((chtype) c);
            break;
@@ -2681,7 +2793,7 @@ slk_test(void)
        case '6':
        case '7':
        case '8':
-           (void) mvaddstr(SLK_WORK, 0, "Please enter the label value: ");
+           MvAddStr(SLK_WORK, 0, "Please enter the label value: ");
            strcpy(buf, "");
            if ((s = slk_label(c - '0')) != 0) {
                strncpy(buf, s, 8);
@@ -2696,20 +2808,6 @@ slk_test(void)
        case case_QUIT:
            goto done;
 
-#if HAVE_SLK_COLOR
-       case 'F':
-           if (use_colors) {
-               fg = (short) ((fg + 1) % COLORS);
-               call_slk_color(fg, bg);
-           }
-           break;
-       case 'B':
-           if (use_colors) {
-               bg = (short) ((bg + 1) % COLORS);
-               call_slk_color(fg, bg);
-           }
-           break;
-#endif
 #if defined(NCURSES_VERSION) && defined(KEY_RESIZE) && HAVE_WRESIZE
        case KEY_RESIZE:
            wnoutrefresh(stdscr);
@@ -2717,7 +2815,24 @@ slk_test(void)
 #endif
 
        default:
+           if (cycle_attr(c, &at_code, &attr)) {
+               slk_attrset(attr);
+               slk_touch();
+               slk_noutrefresh();
+               break;
+           }
+#if HAVE_SLK_COLOR
+           if (cycle_colors(c, &fg, &bg, &pair)) {
+               if (use_colors) {
+                   call_slk_color(fg, bg);
+               } else {
+                   beep();
+               }
+               break;
+           }
+#endif
            beep();
+           break;
        }
     } while (!isQuit(c = Getchar()));
 
@@ -2736,8 +2851,11 @@ wide_slk_test(void)
     int c, fmt = 1;
     wchar_t buf[SLKLEN + 1];
     char *s;
-    short fg = COLOR_BLACK;
-    short bg = COLOR_WHITE;
+    chtype attr = A_NORMAL;
+    unsigned at_code = 0;
+    int fg = COLOR_BLACK;
+    int bg = COLOR_WHITE;
+    short pair = 0;
 
     c = CTRL('l');
     if (use_colors) {
@@ -2749,7 +2867,7 @@ wide_slk_test(void)
        case CTRL('l'):
            erase();
            attr_on(WA_BOLD, NULL);
-           mvaddstr(0, 20, "Soft Key Exerciser");
+           MvAddStr(0, 20, "Soft Key Exerciser");
            attr_off(WA_BOLD, NULL);
 
            slk_help();
@@ -2764,7 +2882,7 @@ wide_slk_test(void)
            break;
 
        case 's':
-           mvprintw(SLK_WORK, 0, "Press Q to stop the scrolling-test: ");
+           MvPrintw(SLK_WORK, 0, "Press Q to stop the scrolling-test: ");
            while ((c = Getchar()) != 'Q' && (c != ERR))
                addch((chtype) c);
            break;
@@ -2793,7 +2911,7 @@ wide_slk_test(void)
        case '6':
        case '7':
        case '8':
-           (void) mvaddstr(SLK_WORK, 0, "Please enter the label value: ");
+           MvAddStr(SLK_WORK, 0, "Please enter the label value: ");
            *buf = 0;
            if ((s = slk_label(c - '0')) != 0) {
                char *temp = strdup(s);
@@ -2849,7 +2967,24 @@ wide_slk_test(void)
            break;
 #endif
        default:
+           if (cycle_attr(c, &at_code, &attr)) {
+               slk_attr_set(attr, (fg || bg), NULL);
+               slk_touch();
+               slk_noutrefresh();
+               break;
+           }
+#if HAVE_SLK_COLOR
+           if (cycle_colors(c, &fg, &bg, &pair)) {
+               if (use_colors) {
+                   call_slk_color(fg, bg);
+               } else {
+                   beep();
+               }
+               break;
+           }
+#endif
            beep();
+           break;
        }
     } while (!isQuit(c = Getchar()));
 
@@ -2861,90 +2996,6 @@ wide_slk_test(void)
 #endif
 #endif /* SLK_INIT */
 
-/****************************************************************************
- *
- * Alternate character-set stuff
- *
- ****************************************************************************/
-/* *INDENT-OFF* */
-static struct {
-    chtype attr;
-    const char *name;
-} attrs_to_cycle[] = {
-    { A_NORMAL,                "normal" },
-    { A_BOLD,          "bold" },
-    { A_BLINK,         "blink" },
-    { A_REVERSE,       "reverse" },
-    { A_UNDERLINE,     "underline" },
-};
-/* *INDENT-ON* */
-
-static bool
-cycle_attr(int ch, unsigned *at_code, chtype *attr)
-{
-    bool result = TRUE;
-
-    switch (ch) {
-    case 'v':
-       if ((*at_code += 1) >= SIZEOF(attrs_to_cycle))
-           *at_code = 0;
-       break;
-    case 'V':
-       if (*at_code == 0)
-           *at_code = SIZEOF(attrs_to_cycle) - 1;
-       else
-           *at_code -= 1;
-       break;
-    default:
-       result = FALSE;
-       break;
-    }
-    if (result)
-       *attr = attrs_to_cycle[*at_code].attr;
-    return result;
-}
-
-static bool
-cycle_colors(int ch, int *fg, int *bg, short *pair)
-{
-    bool result = FALSE;
-
-    if (use_colors) {
-       result = TRUE;
-       switch (ch) {
-       case 'F':
-           if ((*fg -= 1) < 0)
-               *fg = COLORS - 1;
-           break;
-       case 'f':
-           if ((*fg += 1) >= COLORS)
-               *fg = 0;
-           break;
-       case 'B':
-           if ((*bg -= 1) < 0)
-               *bg = COLORS - 1;
-           break;
-       case 'b':
-           if ((*bg += 1) >= COLORS)
-               *bg = 0;
-           break;
-       default:
-           result = FALSE;
-           break;
-       }
-       if (result) {
-           *pair = (short) (*fg != COLOR_BLACK || *bg != COLOR_BLACK);
-           if (*pair != 0) {
-               *pair = 1;
-               if (init_pair(*pair, (short) *fg, (short) *bg) == ERR) {
-                   result = FALSE;
-               }
-           }
-       }
-    }
-    return result;
-}
-
 /* ISO 6429:  codes 0x80 to 0x9f may be control characters that cause the
  * terminal to perform functions.  The remaining codes can be graphic.
  */
@@ -2958,7 +3009,7 @@ show_upper_chars(unsigned first, int repeat, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvprintw(0, 20, "Display of %s Character Codes %d to %d",
+    MvPrintw(0, 20, "Display of %s Character Codes %d to %d",
             C1 ? "C1" : "GR", first, last);
     attroff(A_BOLD);
     refresh();
@@ -2969,7 +3020,7 @@ show_upper_chars(unsigned first, int repeat, attr_t attr, short pair)
        int col = ((int) (code - first) / 16) * COLS / 2;
        char tmp[80];
        sprintf(tmp, "%3u (0x%x)", code, code);
-       mvprintw(row, col, "%*s: ", COLS / 4, tmp);
+       MvPrintw(row, col, "%*s: ", COLS / 4, tmp);
 
        do {
            if (C1)
@@ -2996,19 +3047,19 @@ show_pc_chars(int repeat, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvprintw(0, 20, "Display of PC Character Codes");
+    MvPrintw(0, 20, "Display of PC Character Codes");
     attroff(A_BOLD);
     refresh();
 
     for (code = 0; code < 16; ++code) {
-       mvprintw(2, (int) code * PC_COLS + 8, "%X", code);
+       MvPrintw(2, (int) code * PC_COLS + 8, "%X", code);
     }
     for (code = 0; code < 256; code++) {
        int count = repeat;
        int row = 3 + (int) (code / 16) + (code >= 128);
        int col = 8 + (int) (code % 16) * PC_COLS;
        if ((code % 16) == 0)
-           mvprintw(row, 0, "0x%02x:", code);
+           MvPrintw(row, 0, "0x%02x:", code);
        move(row, col);
        do {
            switch (code) {
@@ -3038,7 +3089,7 @@ show_box_chars(int repeat, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvaddstr(0, 20, "Display of the ACS Line-Drawing Set");
+    MvAddStr(0, 20, "Display of the ACS Line-Drawing Set");
     attroff(A_BOLD);
     refresh();
     /* *INDENT-OFF* */
@@ -3051,13 +3102,13 @@ show_box_chars(int repeat, attr_t attr, short pair)
            colored_chtype(ACS_URCORNER, attr, pair),
             colored_chtype(ACS_LLCORNER, attr, pair),
            colored_chtype(ACS_LRCORNER, attr, pair));
-    mvhline(LINES / 2, 0,        colored_chtype(ACS_HLINE, attr, pair), COLS);
-    mvvline(0,         COLS / 2, colored_chtype(ACS_VLINE, attr, pair), LINES);
-    mvaddch(0,         COLS / 2, colored_chtype(ACS_TTEE,  attr, pair));
-    mvaddch(LINES / 2, COLS / 2, colored_chtype(ACS_PLUS,  attr, pair));
-    mvaddch(LINES - 1, COLS / 2, colored_chtype(ACS_BTEE,  attr, pair));
-    mvaddch(LINES / 2, 0,        colored_chtype(ACS_LTEE,  attr, pair));
-    mvaddch(LINES / 2, COLS - 1, colored_chtype(ACS_RTEE,  attr, pair));
+    MvHLine(LINES / 2, 0,        colored_chtype(ACS_HLINE, attr, pair), COLS);
+    MvVLine(0,         COLS / 2, colored_chtype(ACS_VLINE, attr, pair), LINES);
+    MvAddCh(0,         COLS / 2, colored_chtype(ACS_TTEE,  attr, pair));
+    MvAddCh(LINES / 2, COLS / 2, colored_chtype(ACS_PLUS,  attr, pair));
+    MvAddCh(LINES - 1, COLS / 2, colored_chtype(ACS_BTEE,  attr, pair));
+    MvAddCh(LINES / 2, 0,        colored_chtype(ACS_LTEE,  attr, pair));
+    MvAddCh(LINES / 2, COLS - 1, colored_chtype(ACS_RTEE,  attr, pair));
     /* *INDENT-ON* */
 
 }
@@ -3069,7 +3120,7 @@ show_1_acs(int n, int repeat, const char *name, chtype code)
     int row = 2 + (n % height);
     int col = (n / height) * COLS / 2;
 
-    mvprintw(row, col, "%*s : ", COLS / 4, name);
+    MvPrintw(row, col, "%*s : ", COLS / 4, name);
     do {
        addch(code);
     } while (--repeat > 0);
@@ -3086,7 +3137,7 @@ show_acs_chars(int repeat, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvaddstr(0, 20, "Display of the ACS Character Set");
+    MvAddStr(0, 20, "Display of the ACS Character Set");
     attroff(A_BOLD);
     refresh();
 
@@ -3131,7 +3182,7 @@ show_acs_chars(int repeat, attr_t attr, short pair)
     n = show_1_acs(n, repeat, BOTH(ACS_S1));
     n = show_1_acs(n, repeat, BOTH(ACS_S3));
     n = show_1_acs(n, repeat, BOTH(ACS_S7));
-    n = show_1_acs(n, repeat, BOTH(ACS_S9));
+    (void) show_1_acs(n, repeat, BOTH(ACS_S9));
 #endif
 }
 
@@ -3214,18 +3265,18 @@ acs_display(void)
        else
            show_upper_chars((unsigned) (digit * 32 + 128), repeat, attr, pair);
 
-       mvprintw(LINES - 3, 0,
+       MvPrintw(LINES - 3, 0,
                 "Note: ANSI terminals may not display C1 characters.");
-       mvprintw(LINES - 2, 0,
+       MvPrintw(LINES - 2, 0,
                 "Select: a=ACS, x=box, %s0=C1, 1-3,+/- non-ASCII, </> repeat, ESC=quit",
                 pch_kludge);
        if (use_colors) {
-           mvprintw(LINES - 1, 0,
+           MvPrintw(LINES - 1, 0,
                     "v/V, f/F, b/B cycle through video attributes (%s) and color %d/%d.",
                     attrs_to_cycle[at_code].name,
                     fg, bg);
        } else {
-           mvprintw(LINES - 1, 0,
+           MvPrintw(LINES - 1, 0,
                     "v/V cycles through video attributes (%s).",
                     attrs_to_cycle[at_code].name);
        }
@@ -3265,7 +3316,7 @@ show_upper_widechars(int first, int repeat, int space, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvprintw(0, 20, "Display of Character Codes %d to %d", first, last);
+    MvPrintw(0, 20, "Display of Character Codes %d to %d", first, last);
     attroff(A_BOLD);
 
     for (code = first; (int) code <= last; code++) {
@@ -3279,7 +3330,7 @@ show_upper_widechars(int first, int repeat, int space, attr_t attr, short pair)
        memset(&codes, 0, sizeof(codes));
        codes[0] = code;
        sprintf(tmp, "%3ld (0x%lx)", (long) code, (long) code);
-       mvprintw(row, col, "%*s: ", COLS / 4, tmp);
+       MvPrintw(row, col, "%*s: ", COLS / 4, tmp);
        setcchar(&temp, codes, attr, pair, 0);
        do {
            /*
@@ -3314,7 +3365,7 @@ show_1_wacs(int n, int repeat, const char *name, const cchar_t *code)
     int row = 2 + (n % height);
     int col = (n / height) * COLS / 2;
 
-    mvprintw(row, col, "%*s : ", COLS / 4, name);
+    MvPrintw(row, col, "%*s : ", COLS / 4, name);
     while (--repeat >= 0) {
        add_wch(code);
     }
@@ -3336,7 +3387,7 @@ show_wacs_chars(int repeat, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvaddstr(0, 20, "Display of the Wide-ACS Character Set");
+    MvAddStr(0, 20, "Display of the Wide-ACS Character Set");
     attroff(A_BOLD);
     refresh();
 
@@ -3378,7 +3429,7 @@ show_wacs_chars(int repeat, attr_t attr, short pair)
     n = show_1_wacs(n, repeat, BOTH2(WACS_S1));
     n = show_1_wacs(n, repeat, BOTH2(WACS_S3));
     n = show_1_wacs(n, repeat, BOTH2(WACS_S7));
-    n = show_1_wacs(n, repeat, BOTH2(WACS_S9));
+    (void) show_1_wacs(n, repeat, BOTH2(WACS_S9));
 #endif
 }
 
@@ -3396,7 +3447,7 @@ show_wacs_chars_double(int repeat, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvaddstr(0, 20, "Display of the Wide-ACS Character Set");
+    MvAddStr(0, 20, "Display of the Wide-ACS Character Set");
     attroff(A_BOLD);
     refresh();
 
@@ -3438,7 +3489,7 @@ show_wacs_chars_double(int repeat, attr_t attr, short pair)
     n = show_1_wacs(n, repeat, BOTH2(WACS_S1));
     n = show_1_wacs(n, repeat, BOTH2(WACS_S3));
     n = show_1_wacs(n, repeat, BOTH2(WACS_S7));
-    n = show_1_wacs(n, repeat, BOTH2(WACS_S9));
+    (void) show_1_wacs(n, repeat, BOTH2(WACS_S9));
 #endif
 }
 #endif
@@ -3457,7 +3508,7 @@ show_wacs_chars_thick(int repeat, attr_t attr, short pair)
 
     erase();
     attron(A_BOLD);
-    mvaddstr(0, 20, "Display of the Wide-ACS Character Set");
+    MvAddStr(0, 20, "Display of the Wide-ACS Character Set");
     attroff(A_BOLD);
     refresh();
 
@@ -3499,7 +3550,7 @@ show_wacs_chars_thick(int repeat, attr_t attr, short pair)
     n = show_1_wacs(n, repeat, BOTH2(WACS_S1));
     n = show_1_wacs(n, repeat, BOTH2(WACS_S3));
     n = show_1_wacs(n, repeat, BOTH2(WACS_S7));
-    n = show_1_wacs(n, repeat, BOTH2(WACS_S9));
+    (void) show_1_wacs(n, repeat, BOTH2(WACS_S9));
 #endif
 }
 #endif
@@ -3516,7 +3567,7 @@ show_wbox_chars(int repeat, attr_t attr, short pair)
     (void) repeat;
     erase();
     attron(A_BOLD);
-    mvaddstr(0, 20, "Display of the Wide-ACS Line-Drawing Set");
+    MvAddStr(0, 20, "Display of the Wide-ACS Line-Drawing Set");
     attroff(A_BOLD);
     refresh();
 
@@ -3530,13 +3581,13 @@ show_wbox_chars(int repeat, attr_t attr, short pair)
                MERGE_ATTR(6, WACS_LLCORNER),
                MERGE_ATTR(7, WACS_LRCORNER));
     /* *INDENT-OFF* */
-    mvhline_set(LINES / 2, 0,        MERGE_ATTR(0, WACS_HLINE), COLS);
-    mvvline_set(0,         COLS / 2, MERGE_ATTR(0, WACS_VLINE), LINES);
-    mvadd_wch(0,           COLS / 2, MERGE_ATTR(0, WACS_TTEE));
-    mvadd_wch(LINES / 2,   COLS / 2, MERGE_ATTR(0, WACS_PLUS));
-    mvadd_wch(LINES - 1,   COLS / 2, MERGE_ATTR(0, WACS_BTEE));
-    mvadd_wch(LINES / 2,   0,        MERGE_ATTR(0, WACS_LTEE));
-    mvadd_wch(LINES / 2,   COLS - 1, MERGE_ATTR(0, WACS_RTEE));
+    (void) mvhline_set(LINES / 2, 0,        MERGE_ATTR(0, WACS_HLINE), COLS);
+    (void) mvvline_set(0,         COLS / 2, MERGE_ATTR(0, WACS_VLINE), LINES);
+    (void) mvadd_wch(0,           COLS / 2, MERGE_ATTR(0, WACS_TTEE));
+    (void) mvadd_wch(LINES / 2,   COLS / 2, MERGE_ATTR(0, WACS_PLUS));
+    (void) mvadd_wch(LINES - 1,   COLS / 2, MERGE_ATTR(0, WACS_BTEE));
+    (void) mvadd_wch(LINES / 2,   0,        MERGE_ATTR(0, WACS_LTEE));
+    (void) mvadd_wch(LINES / 2,   COLS - 1, MERGE_ATTR(0, WACS_RTEE));
     /* *INDENT-ON* */
 
 }
@@ -3551,10 +3602,10 @@ show_2_wacs(int n, const char *name, const char *code, attr_t attr, short pair)
     int col = (n / height) * COLS / 2;
     char temp[80];
 
-    mvprintw(row, col, "%*s : ", COLS / 4, name);
-    attr_set(attr, pair, 0);
+    MvPrintw(row, col, "%*s : ", COLS / 4, name);
+    (void) attr_set(attr, pair, 0);
     addstr(strcpy(temp, code));
-    attr_set(A_NORMAL, 0, 0);
+    (void) attr_set(A_NORMAL, 0, 0);
     return n + 1;
 }
 
@@ -3568,7 +3619,7 @@ show_utf8_chars(int repeat, attr_t attr, short pair)
     (void) repeat;
     erase();
     attron(A_BOLD);
-    mvaddstr(0, 20, "Display of the Wide-ACS Character Set");
+    MvAddStr(0, 20, "Display of the Wide-ACS Character Set");
     attroff(A_BOLD);
     refresh();
     /* *INDENT-OFF* */
@@ -3608,7 +3659,7 @@ show_utf8_chars(int repeat, attr_t attr, short pair)
     n = SHOW_UTF8(n, "WACS_S1",                "\342\216\272");
     n = SHOW_UTF8(n, "WACS_S3",                "\342\216\273");
     n = SHOW_UTF8(n, "WACS_S7",                "\342\216\274");
-    n = SHOW_UTF8(n, "WACS_S9",                "\342\216\275");
+    (void) SHOW_UTF8(n, "WACS_S9",     "\342\216\275");
     /* *INDENT-ON* */
 
 }
@@ -3684,15 +3735,15 @@ wide_acs_display(void)
        else
            show_upper_widechars(digit * 32 + 128, repeat, space, attr, pair);
 
-       mvprintw(LINES - 3, 0,
+       MvPrintw(LINES - 3, 0,
                 "Select: a/d/t WACS, x box, u UTF-8, 0-9,+/- non-ASCII, </> repeat, ESC=quit");
        if (use_colors) {
-           mvprintw(LINES - 2, 0,
+           MvPrintw(LINES - 2, 0,
                     "v/V, f/F, b/B cycle through video attributes (%s) and color %d/%d.",
                     attrs_to_cycle[at_code].name,
                     fg, bg);
        } else {
-           mvprintw(LINES - 2, 0,
+           MvPrintw(LINES - 2, 0,
                     "v/V cycles through video attributes (%s).",
                     attrs_to_cycle[at_code].name);
        }
@@ -3724,58 +3775,58 @@ test_sgr_attributes(void)
        }
        bkgdset(normal);
        erase();
-       mvprintw(1, 20, "Graphic rendition test pattern:");
+       MvPrintw(1, 20, "Graphic rendition test pattern:");
 
-       mvprintw(4, 1, "vanilla");
+       MvPrintw(4, 1, "vanilla");
 
 #define set_sgr(mask) bkgdset((normal^(mask)));
        set_sgr(A_BOLD);
-       mvprintw(4, 40, "bold");
+       MvPrintw(4, 40, "bold");
 
        set_sgr(A_UNDERLINE);
-       mvprintw(6, 6, "underline");
+       MvPrintw(6, 6, "underline");
 
        set_sgr(A_BOLD | A_UNDERLINE);
-       mvprintw(6, 45, "bold underline");
+       MvPrintw(6, 45, "bold underline");
 
        set_sgr(A_BLINK);
-       mvprintw(8, 1, "blink");
+       MvPrintw(8, 1, "blink");
 
        set_sgr(A_BLINK | A_BOLD);
-       mvprintw(8, 40, "bold blink");
+       MvPrintw(8, 40, "bold blink");
 
        set_sgr(A_UNDERLINE | A_BLINK);
-       mvprintw(10, 6, "underline blink");
+       MvPrintw(10, 6, "underline blink");
 
        set_sgr(A_BOLD | A_UNDERLINE | A_BLINK);
-       mvprintw(10, 45, "bold underline blink");
+       MvPrintw(10, 45, "bold underline blink");
 
        set_sgr(A_REVERSE);
-       mvprintw(12, 1, "negative");
+       MvPrintw(12, 1, "negative");
 
        set_sgr(A_BOLD | A_REVERSE);
-       mvprintw(12, 40, "bold negative");
+       MvPrintw(12, 40, "bold negative");
 
        set_sgr(A_UNDERLINE | A_REVERSE);
-       mvprintw(14, 6, "underline negative");
+       MvPrintw(14, 6, "underline negative");
 
        set_sgr(A_BOLD | A_UNDERLINE | A_REVERSE);
-       mvprintw(14, 45, "bold underline negative");
+       MvPrintw(14, 45, "bold underline negative");
 
        set_sgr(A_BLINK | A_REVERSE);
-       mvprintw(16, 1, "blink negative");
+       MvPrintw(16, 1, "blink negative");
 
        set_sgr(A_BOLD | A_BLINK | A_REVERSE);
-       mvprintw(16, 40, "bold blink negative");
+       MvPrintw(16, 40, "bold blink negative");
 
        set_sgr(A_UNDERLINE | A_BLINK | A_REVERSE);
-       mvprintw(18, 6, "underline blink negative");
+       MvPrintw(18, 6, "underline blink negative");
 
        set_sgr(A_BOLD | A_UNDERLINE | A_BLINK | A_REVERSE);
-       mvprintw(18, 45, "bold underline blink negative");
+       MvPrintw(18, 45, "bold underline blink negative");
 
        bkgdset(normal);
-       mvprintw(LINES - 2, 1, "%s background. ", pass == 0 ? "Dark" :
+       MvPrintw(LINES - 2, 1, "%s background. ", pass == 0 ? "Dark" :
                 "Light");
        clrtoeol();
        Pause();
@@ -3923,7 +3974,7 @@ transient(FRAME * curp, NCURSES_CONST char *msg)
 {
     newwin_legend(curp);
     if (msg) {
-       mvaddstr(LINES - 1, 0, msg);
+       MvAddStr(LINES - 1, 0, msg);
        refresh();
        napms(1000);
     }
@@ -4013,10 +4064,10 @@ static void
 outerbox(pair ul, pair lr, bool onoff)
 /* draw or erase a box *outside* the given pair of corners */
 {
-    mvaddch(ul.y - 1, lr.x - 1, onoff ? ACS_ULCORNER : ' ');
-    mvaddch(ul.y - 1, lr.x + 1, onoff ? ACS_URCORNER : ' ');
-    mvaddch(lr.y + 1, lr.x + 1, onoff ? ACS_LRCORNER : ' ');
-    mvaddch(lr.y + 1, ul.x - 1, onoff ? ACS_LLCORNER : ' ');
+    MvAddCh(ul.y - 1, lr.x - 1, onoff ? ACS_ULCORNER : ' ');
+    MvAddCh(ul.y - 1, lr.x + 1, onoff ? ACS_URCORNER : ' ');
+    MvAddCh(lr.y + 1, lr.x + 1, onoff ? ACS_LRCORNER : ' ');
+    MvAddCh(lr.y + 1, ul.x - 1, onoff ? ACS_LLCORNER : ' ');
     move(ul.y - 1, ul.x);
     hline(onoff ? ACS_HLINE : ' ', lr.x - ul.x + 1);
     move(ul.y, ul.x - 1);
@@ -4041,7 +4092,7 @@ getwindow(void)
     if ((tmp = selectcell(2, 1, LINES - BOTLINES - 2, COLS - 2)) == (pair *) 0)
        return ((WINDOW *) 0);
     memcpy(&ul, tmp, sizeof(pair));
-    mvaddch(ul.y - 1, ul.x - 1, ACS_ULCORNER);
+    MvAddCh(ul.y - 1, ul.x - 1, ACS_ULCORNER);
     move(0, 0);
     clrtoeol();
     addstr("Use arrows to move cursor, anything else to mark corner 2");
@@ -4110,7 +4161,7 @@ acs_and_scroll(void)
 {
     int c;
     FRAME *current = (FRAME *) 0, *neww;
-    WINDOW *usescr = stdscr;
+    WINDOW *usescr;
 #if HAVE_PUTWIN && HAVE_GETWIN
     FILE *fp;
 #endif
@@ -4310,7 +4361,7 @@ acs_and_scroll(void)
                        break;
                    x = getmaxx(frame_win(current)) - 1;
                }
-               mvwdelch(frame_win(current), y, x);
+               (void) mvwdelch(frame_win(current), y, x);
            }
            break;
 
@@ -4711,12 +4762,13 @@ panner_legend(int line)
        "Number repeats.  Toggle legend:? filler:a timer:t scrollmark:s."
     };
     int n = ((int) SIZEOF(legend) - (LINES - line));
-    if (line < LINES && (n >= 0)) {
-       move(line, 0);
-       if (show_panner_legend)
-           printw("%s", legend[n]);
-       clrtoeol();
-       return show_panner_legend;
+    if (n >= 0) {
+       if (move(line, 0) != ERR) {
+           if (show_panner_legend)
+               printw("%s", legend[n]);
+           clrtoeol();
+           return show_panner_legend;
+       }
     }
     return FALSE;
 }
@@ -4954,7 +5006,7 @@ panner(WINDOW *pad,
            break;
        }
 
-       mvaddch(top_y - 1, top_x - 1, ACS_ULCORNER);
+       MvAddCh(top_y - 1, top_x - 1, ACS_ULCORNER);
        do_v_line(top_y, top_x - 1, ACS_VLINE, porty);
        do_h_line(top_y - 1, top_x, ACS_HLINE, portx);
 
@@ -4992,9 +5044,9 @@ panner(WINDOW *pad,
        } else
            do_v_line(top_y, portx - 1, ACS_VLINE, porty);
 
-       mvaddch(top_y - 1, portx - 1, ACS_URCORNER);
-       mvaddch(porty - 1, top_x - 1, ACS_LLCORNER);
-       mvaddch(porty - 1, portx - 1, ACS_LRCORNER);
+       MvAddCh(top_y - 1, portx - 1, ACS_URCORNER);
+       MvAddCh(porty - 1, top_x - 1, ACS_LLCORNER);
+       MvAddCh(porty - 1, portx - 1, ACS_LRCORNER);
 
        if (!pending_pan) {
 #if HAVE_GETTIMEOFDAY
@@ -5142,7 +5194,7 @@ Continue(WINDOW *win)
 {
     noecho();
     wmove(win, 10, 1);
-    mvwaddstr(win, 10, 1, " Press any key to continue");
+    MvWAddStr(win, 10, 1, " Press any key to continue");
     wrefresh(win);
     wGetchar(win);
 }
@@ -5171,7 +5223,7 @@ flushinp_test(WINDOW *win)
 #endif
     (void) wattrset(subWin, A_BOLD);
     box(subWin, ACS_VLINE, ACS_HLINE);
-    mvwaddstr(subWin, 2, 1, "This is a subwindow");
+    MvWAddStr(subWin, 2, 1, "This is a subwindow");
     wrefresh(win);
 
     /*
@@ -5182,15 +5234,15 @@ flushinp_test(WINDOW *win)
      * character to return without needing a newline. - T.Dickey 1997/10/11.
      */
     cbreak();
-    mvwaddstr(win, 0, 1, "This is a test of the flushinp() call.");
+    MvWAddStr(win, 0, 1, "This is a test of the flushinp() call.");
 
-    mvwaddstr(win, 2, 1, "Type random keys for 5 seconds.");
-    mvwaddstr(win, 3, 1,
+    MvWAddStr(win, 2, 1, "Type random keys for 5 seconds.");
+    MvWAddStr(win, 3, 1,
              "These should be discarded (not echoed) after the subwindow goes away.");
     wrefresh(win);
 
     for (i = 0; i < 5; i++) {
-       mvwprintw(subWin, 1, 1, "Time = %d", i);
+       MvWPrintw(subWin, 1, 1, "Time = %d", i);
        wrefresh(subWin);
        napms(1000);
        flushinp();
@@ -5202,19 +5254,19 @@ flushinp_test(WINDOW *win)
     wrefresh(win);
     napms(1000);
 
-    mvwaddstr(win, 2, 1,
+    MvWAddStr(win, 2, 1,
              "If you were still typing when the window timer expired,");
-    mvwaddstr(win, 3, 1,
+    MvWAddStr(win, 3, 1,
              "or else you typed nothing at all while it was running,");
-    mvwaddstr(win, 4, 1,
+    MvWAddStr(win, 4, 1,
              "test was invalid.  You'll see garbage or nothing at all. ");
-    mvwaddstr(win, 6, 1, "Press a key");
+    MvWAddStr(win, 6, 1, "Press a key");
     wmove(win, 9, 10);
     wrefresh(win);
     echo();
     wGetchar(win);
     flushinp();
-    mvwaddstr(win, 12, 0,
+    MvWAddStr(win, 12, 0,
              "If you see any key other than what you typed, flushinp() is broken.");
     Continue(win);
 
@@ -5298,12 +5350,12 @@ menu_test(void)
 #ifdef NCURSES_MOUSE_VERSION
     mousemask(ALL_MOUSE_EVENTS, (mmask_t *) 0);
 #endif
-    mvaddstr(0, 0, "This is the menu test:");
-    mvaddstr(2, 0, "  Use up and down arrow to move the select bar.");
-    mvaddstr(3, 0, "  'n' and 'p' act like arrows.");
-    mvaddstr(4, 0,
+    MvAddStr(0, 0, "This is the menu test:");
+    MvAddStr(2, 0, "  Use up and down arrow to move the select bar.");
+    MvAddStr(3, 0, "  'n' and 'p' act like arrows.");
+    MvAddStr(4, 0,
             "  'b' and 'f' scroll up/down (page), 'u' and 'd' (line).");
-    mvaddstr(5, 0, "  Press return to exit.");
+    MvAddStr(5, 0, "  Press return to exit.");
     refresh();
 
     for (ap = animals; *ap; ap++) {
@@ -5334,8 +5386,8 @@ menu_test(void)
        continue;
     }
 
-    (void) mvprintw(LINES - 2, 0,
-                   "You chose: %s\n", item_name(current_item(m)));
+    MvPrintw(LINES - 2, 0,
+            "You chose: %s\n", item_name(current_item(m)));
     (void) addstr("Press any key to continue...");
     wGetchar(stdscr);
 
@@ -5456,11 +5508,11 @@ trace_set(void)
     int n;
     WINDOW *menuwin;
 
-    mvaddstr(0, 0, "Interactively set trace level:");
-    mvaddstr(2, 0, "  Press space bar to toggle a selection.");
-    mvaddstr(3, 0, "  Use up and down arrow to move the select bar.");
-    mvaddstr(4, 0, "  Press return to set the trace level.");
-    mvprintw(6, 0, "(Current trace level is %s)", tracetrace(_nc_tracing));
+    MvAddStr(0, 0, "Interactively set trace level:");
+    MvAddStr(2, 0, "  Press space bar to toggle a selection.");
+    MvAddStr(3, 0, "  Use up and down arrow to move the select bar.");
+    MvAddStr(4, 0, "  Press return to set the trace level.");
+    MvPrintw(6, 0, "(Current trace level is %s)", tracetrace(_nc_tracing));
 
     refresh();
 
@@ -5504,8 +5556,8 @@ trace_set(void)
     trace(newtrace);
     Trace(("trace level interactively set to %s", tracetrace(_nc_tracing)));
 
-    (void) mvprintw(LINES - 2, 0,
-                   "Trace level is %s\n", tracetrace(_nc_tracing));
+    MvPrintw(LINES - 2, 0,
+            "Trace level is %s\n", tracetrace(_nc_tracing));
     (void) addstr("Press any key to continue...");
     wGetchar(stdscr);
 
@@ -5711,7 +5763,7 @@ form_virtualize(FORM * f, WINDOW *w)
            }
        }
     }
-    mvprintw(0, COLS - 6, "(%s)", mode == REQ_INS_MODE ? "INS" : "OVL");
+    MvPrintw(0, COLS - 6, "(%s)", mode == REQ_INS_MODE ? "INS" : "OVL");
 
     /*
      * Force the field that the user is typing into to be in reverse video,
@@ -5863,7 +5915,7 @@ demo_forms(void)
     addstr("^K   -- clear to end of field  ^X  -- clear field\n");
     addstr("Arrow keys move within a field as you would expect. ^] toggles overlay mode.");
 
-    mvaddstr(4, 57, "Forms Entry Test");
+    MvAddStr(4, 57, "Forms Entry Test");
 
     refresh();
 
@@ -5890,7 +5942,7 @@ demo_forms(void)
     secure =
        f[n++] = make_field(5, 30, 1, 9, TRUE);
     set_field_type(f[n - 1], fty_passwd);
-    f[n++] = (FIELD *) 0;
+    f[n] = (FIELD *) 0;
 
     if ((form = new_form(f)) != 0) {
 
@@ -5902,7 +5954,7 @@ demo_forms(void)
        while (!finished) {
            switch (form_driver(form, c = form_virtualize(form, w))) {
            case E_OK:
-               mvaddstr(5, 57, field_buffer(secure, 1));
+               MvAddStr(5, 57, field_buffer(secure, 1));
                clrtoeol();
                refresh();
                break;
@@ -6470,7 +6522,7 @@ rip_footer(WINDOW *win, int cols)
     wbkgd(win, A_REVERSE);
     werase(win);
     wmove(win, 0, 0);
-    wprintw(win, "footer: window %p, %d columns", win, cols);
+    wprintw(win, "footer: window %p, %d columns", (void *) win, cols);
     wnoutrefresh(win);
     return OK;
 }
@@ -6481,7 +6533,7 @@ rip_header(WINDOW *win, int cols)
     wbkgd(win, A_REVERSE);
     werase(win);
     wmove(win, 0, 0);
-    wprintw(win, "header: window %p, %d columns", win, cols);
+    wprintw(win, "header: window %p, %d columns", (void *) win, cols);
     wnoutrefresh(win);
     return OK;
 }
@@ -6545,8 +6597,8 @@ main_menu(bool top)
        (void) fflush(stdout);  /* necessary under SVr4 curses */
 
        /*
-        * This used to be an 'fgets()' call.  However (on Linux, at least)
-        * mixing stream I/O and 'read()' (used in the library) causes the
+        * This used to be an 'fgets()' call (until 1996/10).  However with
+        * some runtime libraries, mixing stream I/O and 'read()' causes the
         * input stream to be flushed when switching between the two.
         */
        command = 0;