ncurses 5.9 - patch 20140823
[ncurses.git] / ncurses / tty / tty_update.c
index b91053bced85d6e5034fcd5f25e55169a418fe12..e66f71605ee27254ca40868502f2b2d946020b66 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1998-2008,2009 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2013,2014 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            *
@@ -82,7 +82,7 @@
 
 #include <ctype.h>
 
-MODULE_ID("$Id: tty_update.c,v 1.256 2009/05/17 00:13:49 tom Exp $")
+MODULE_ID("$Id: tty_update.c,v 1.280 2014/08/23 19:25:18 tom Exp $")
 
 /*
  * This define controls the line-breakout optimization.  Every once in a
@@ -145,8 +145,7 @@ position_check(NCURSES_SP_DCLx int expected_y, int expected_x, char *legend)
 
     NCURSES_SP_NAME(_nc_flush) (NCURSES_SP_ARG);
     memset(buf, '\0', sizeof(buf));
-    NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx "\033[6n"); /* only works on ANSI-compatibles */
-    NCURSES_SP_NAME(_nc_flush) (NCURSES_SP_ARG);
+    NCURSES_PUTP2_FLUSH("cpr", "\033[6n");     /* only works on ANSI-compatibles */
     *(s = buf) = 0;
     do {
        int ask = sizeof(buf) - 1 - (s - buf);
@@ -180,7 +179,7 @@ position_check(NCURSES_SP_DCLx int expected_y, int expected_x, char *legend)
     }
 }
 #else
-#define position_check(sp, expected_y, expected_x, legend)     /* nothing */
+#define position_check(expected_y, expected_x, legend) /* nothing */
 #endif /* POSITION_DEBUG */
 
 /****************************************************************************
@@ -192,13 +191,20 @@ position_check(NCURSES_SP_DCLx int expected_y, int expected_x, char *legend)
 static NCURSES_INLINE void
 GoTo(NCURSES_SP_DCLx int const row, int const col)
 {
-    TR(TRACE_MOVE, ("GoTo(%d, %d) from (%d, %d)",
-                   row, col, SP_PARM->_cursrow, SP_PARM->_curscol));
+    TR(TRACE_MOVE, ("GoTo(%p, %d, %d) from (%d, %d)",
+                   (void *) SP_PARM, row, col, SP_PARM->_cursrow, SP_PARM->_curscol));
 
-    position_check(SP_PARM, SP_PARM->_cursrow, SP_PARM->_curscol, "GoTo");
+    position_check(NCURSES_SP_ARGx
+                  SP_PARM->_cursrow,
+                  SP_PARM->_curscol, "GoTo");
 
-    mvcur(SP_PARM->_cursrow, SP_PARM->_curscol, row, col);
-    position_check(SP_PARM, SP_PARM->_cursrow, SP_PARM->_curscol, "GoTo2");
+    TINFO_MVCUR(NCURSES_SP_ARGx
+               SP_PARM->_cursrow,
+               SP_PARM->_curscol,
+               row, col);
+    position_check(NCURSES_SP_ARGx
+                  SP_PARM->_cursrow,
+                  SP_PARM->_curscol, "GoTo2");
 }
 
 static NCURSES_INLINE void
@@ -206,7 +212,9 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
 {
     int chlen = 1;
     NCURSES_CH_T my_ch;
+#if USE_WIDEC_SUPPORT
     PUTC_DATA;
+#endif
     NCURSES_CH_T tilde;
     NCURSES_CH_T attr = CHDEREF(ch);
 
@@ -273,6 +281,11 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
            && SP_PARM->_screen_acs_map[CharOf(my_ch)]) {
            RemAttr(attr, A_ALTCHARSET);
            my_ch = _nc_wacs[CharOf(my_ch)];
+       } else if (SP_PARM->_screen_unicode
+                  && !SP_PARM->_screen_acs_map[CharOf(my_ch)]
+                  && _nc_wacs[CharOf(my_ch)].chars[0]) {
+           RemAttr(attr, A_ALTCHARSET);
+           my_ch = _nc_wacs[CharOf(my_ch)];
        }
 #endif
        /*
@@ -285,13 +298,12 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
            int j = CharOfD(ch);
            chtype temp = UChar(SP_PARM->_acs_map[j]);
 
-           if (!(SP_PARM->_screen_acs_map[j])) {
+           if (temp != 0) {
+               SetChar(my_ch, temp, AttrOf(attr));
+           } else {
+               my_ch = CHDEREF(ch);
                RemAttr(attr, A_ALTCHARSET);
-               if (temp == 0)
-                   temp = ' ';
            }
-           if (temp != 0)
-               SetChar(my_ch, temp, AttrOf(attr));
        }
        ch = CHREF(my_ch);
     }
@@ -301,22 +313,13 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
     }
 
     UpdateAttrs(SP_PARM, attr);
+    PUTC(CHDEREF(ch));
 #if !USE_WIDEC_SUPPORT
-    /* FIXME - we do this special case for signal handling, should see how to
-     * make it work for wide characters.
-     */
-    if (SP_PARM->_outch != 0) {
-       SP_PARM->_outch(NCURSES_SP_ARGx UChar(ch));
-    } else
+    COUNT_OUTCHARS(1);
 #endif
-    {
-       PUTC(CHDEREF(ch), SP_PARM->_ofp);       /* macro's fastest... */
-       COUNT_OUTCHARS(1);
-    }
     SP_PARM->_curscol += chlen;
     if (char_padding) {
-       TPUTS_TRACE("char_padding");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx char_padding);
+       NCURSES_PUTP2("char_padding", char_padding);
     }
 }
 
@@ -339,7 +342,7 @@ check_pending(NCURSES_SP_DCL0)
        struct pollfd fds[1];
        fds[0].fd = SP_PARM->_checkfd;
        fds[0].events = POLLIN;
-       if (poll(fds, 1, 0) > 0) {
+       if (poll(fds, (size_t) 1, 0) > 0) {
            have_pending = TRUE;
        }
 #elif defined(__BEOS__)
@@ -387,18 +390,16 @@ PutCharLR(NCURSES_SP_DCLx const ARG_CH_T ch)
        PutAttrChar(NCURSES_SP_ARGx ch);
     } else if (enter_am_mode && exit_am_mode) {
        /* we can suppress automargin */
-       TPUTS_TRACE("exit_am_mode");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx exit_am_mode);
+       NCURSES_PUTP2("exit_am_mode", exit_am_mode);
 
        PutAttrChar(NCURSES_SP_ARGx ch);
        SP_PARM->_curscol--;
-       position_check(SP_PARM,
+       position_check(NCURSES_SP_ARGx
                       SP_PARM->_cursrow,
                       SP_PARM->_curscol,
                       "exit_am_mode");
 
-       TPUTS_TRACE("enter_am_mode");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx enter_am_mode);
+       NCURSES_PUTP2("enter_am_mode", enter_am_mode);
     } else if ((enter_insert_mode && exit_insert_mode)
               || insert_character || parm_ich) {
        GoTo(NCURSES_SP_ARGx
@@ -452,7 +453,7 @@ wrap_cursor(NCURSES_SP_DCL0)
     } else {
        SP_PARM->_curscol--;
     }
-    position_check(SP_PARM,
+    position_check(NCURSES_SP_ARGx
                   SP_PARM->_cursrow,
                   SP_PARM->_curscol,
                   "wrap_cursor");
@@ -472,7 +473,9 @@ PutChar(NCURSES_SP_DCLx const ARG_CH_T ch)
     if (SP_PARM->_curscol >= screen_columns(SP_PARM))
        wrap_cursor(NCURSES_SP_ARG);
 
-    position_check(SP_PARM, SP_PARM->_cursrow, SP_PARM->_curscol, "PutChar");
+    position_check(NCURSES_SP_ARGx
+                  SP_PARM->_cursrow,
+                  SP_PARM->_curscol, "PutChar");
 }
 
 /*
@@ -493,10 +496,13 @@ can_clear_with(NCURSES_SP_DCLx ARG_CH_T ch)
        if (SP_PARM->_default_fg != C_MASK || SP_PARM->_default_bg != C_MASK)
            return FALSE;
        if ((pair = GetPair(CHDEREF(ch))) != 0) {
-           short fg, bg;
-           NCURSES_SP_NAME(pair_content) (NCURSES_SP_ARGx pair, &fg, &bg);
-           if (fg != C_MASK || bg != C_MASK)
+           NCURSES_COLOR_T fg, bg;
+           if (NCURSES_SP_NAME(pair_content) (NCURSES_SP_ARGx
+                                              (short) pair,
+                                              &fg, &bg) == ERR
+               || (fg != C_MASK || bg != C_MASK)) {
                return FALSE;
+           }
        }
 #else
        if (AttrOfD(ch) & A_COLOR)
@@ -558,7 +564,7 @@ EmitRange(NCURSES_SP_DCLx const NCURSES_CH_T * ntext, int num)
                && runcount > SP_PARM->_ech_cost + SP_PARM->_cup_ch_cost
                && can_clear_with(NCURSES_SP_ARGx CHREF(ntext0))) {
                UpdateAttrs(SP_PARM, ntext0);
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx TPARM_1(erase_chars, runcount));
+               NCURSES_PUTP2("erase_chars", TPARM_1(erase_chars, runcount));
 
                /*
                 * If this is the last part of the given interval,
@@ -622,9 +628,13 @@ PutRange(NCURSES_SP_DCLx
         int first, int last)
 {
     int i, j, same;
+    int rc;
 
     TR(TRACE_CHARPUT, ("PutRange(%p, %p, %p, %d, %d, %d)",
-                      SP_PARM, otext, ntext, row, first, last));
+                      (void *) SP_PARM,
+                      (const void *) otext,
+                      (const void *) ntext,
+                      row, first, last));
 
     if (otext != ntext
        && (last - first + 1) > SP_PARM->_inline_cost) {
@@ -646,9 +656,11 @@ PutRange(NCURSES_SP_DCLx
         * Always return 1 for the next GoTo() after a PutRange() if we found
         * identical characters at end of interval
         */
-       return (same == 0 ? i : 1);
+       rc = (same == 0 ? i : 1);
+    } else {
+       rc = EmitRange(NCURSES_SP_ARGx ntext + first, last - first + 1);
     }
-    return EmitRange(NCURSES_SP_ARGx ntext + first, last - first + 1);
+    return rc;
 }
 
 /* leave unbracketed here so 'indent' works */
@@ -658,7 +670,7 @@ PutRange(NCURSES_SP_DCLx
                if_USE_SCROLL_HINTS(win->_line[row].oldindex = row)
 
 NCURSES_EXPORT(int)
-NCURSES_SP_NAME(doupdate) (NCURSES_SP_DCL0)
+TINFO_DOUPDATE(NCURSES_SP_DCL0)
 {
     int i;
     int nonempty;
@@ -666,7 +678,10 @@ NCURSES_SP_NAME(doupdate) (NCURSES_SP_DCL0)
     struct tms before, after;
 #endif /* USE_TRACE_TIMES */
 
-    T((T_CALLED("doupdate()")));
+    T((T_CALLED("_nc_tinfo:doupdate(%p)"), (void *) SP_PARM));
+
+    if (SP_PARM == 0)
+       returnCode(ERR);
 
 #if !USE_REENTRANT
     /*
@@ -806,9 +821,8 @@ NCURSES_SP_NAME(doupdate) (NCURSES_SP_DCL0)
                    /* find end of span, if it's onscreen */
                    for (m = i; m < screen_lines(SP_PARM); m++) {
                        for (; n < screen_columns(SP_PARM); n++) {
-                           attr_t testattr
-                           = AttrOf(NewScreen(SP_PARM)->_line[m].text[n]);
-
+                           attr_t testattr =
+                           AttrOf(NewScreen(SP_PARM)->_line[m].text[n]);
                            if ((testattr & SP_PARM->_xmc_triggers) == rattr) {
                                end_onscreen = TRUE;
                                TR(TRACE_ATTRS,
@@ -870,8 +884,7 @@ NCURSES_SP_NAME(doupdate) (NCURSES_SP_DCL0)
                    /* turn off new attributes over span */
                    for (p = i; p < screen_lines(SP_PARM); p++) {
                        for (; q < screen_columns(SP_PARM); q++) {
-                           attr_t testattr =
-                           AttrOf(NewScreen(SP_PARM)->_line[p].text[q]);
+                           attr_t testattr = AttrOf(newscr->_line[p].text[q]);
                            if ((testattr & SP_PARM->_xmc_triggers) == rattr)
                                goto foundend;
                            RemAttr(NewScreen(SP_PARM)->_line[p].text[q], turnon);
@@ -921,7 +934,7 @@ NCURSES_SP_NAME(doupdate) (NCURSES_SP_DCL0)
        nonempty = min(screen_lines(SP_PARM), NewScreen(SP_PARM)->_maxy + 1);
 
        if (SP_PARM->_scrolling) {
-           _nc_scroll_optimize();
+           NCURSES_SP_NAME(_nc_scroll_optimize) (NCURSES_SP_ARG);
        }
 
        nonempty = ClrBottom(NCURSES_SP_ARGx nonempty);
@@ -938,8 +951,8 @@ NCURSES_SP_NAME(doupdate) (NCURSES_SP_DCL0)
            }
 
            /*
-            * newscr.line[i].firstchar is normally set
-            * by wnoutrefresh.  curscr.line[i].firstchar
+            * newscr->line[i].firstchar is normally set
+            * by wnoutrefresh.  curscr->line[i].firstchar
             * is normally set by _nc_scroll_window in the
             * vertical-movement optimization code,
             */
@@ -1003,11 +1016,11 @@ NCURSES_SP_NAME(doupdate) (NCURSES_SP_DCL0)
     returnCode(OK);
 }
 
-#if NCURSES_SP_FUNCS
+#if NCURSES_SP_FUNCS && !defined(USE_TERM_DRIVER)
 NCURSES_EXPORT(int)
 doupdate(void)
 {
-    return NCURSES_SP_NAME(doupdate) (CURRENT_SCREEN);
+    return TINFO_DOUPDATE(CURRENT_SCREEN);
 }
 #endif
 
@@ -1022,14 +1035,14 @@ doupdate(void)
  *     in the wbkgd() call.  Assume 'stdscr' for this case.
  */
 #define BCE_ATTRS (A_NORMAL|A_COLOR)
-#define BCE_BKGD(win) (((win) == CurScreen(SP_PARM) ? StdScreen(SP_PARM) : (win))->_nc_bkgd)
+#define BCE_BKGD(sp,win) (((win) == CurScreen(sp) ? StdScreen(sp) : (win))->_nc_bkgd)
 
 static NCURSES_INLINE NCURSES_CH_T
 ClrBlank(NCURSES_SP_DCLx WINDOW *win)
 {
     NCURSES_CH_T blank = blankchar;
     if (back_color_erase)
-       AddAttr(blank, (AttrOf(BCE_BKGD(win)) & BCE_ATTRS));
+       AddAttr(blank, (AttrOf(BCE_BKGD(SP_PARM, win)) & BCE_ATTRS));
     return blank;
 }
 
@@ -1069,7 +1082,7 @@ ClrUpdate(NCURSES_SP_DCL0)
 */
 
 static void
-ClrToEOL(NCURSES_SP_DCLx NCURSES_CH_T blank, bool needclear)
+ClrToEOL(NCURSES_SP_DCLx NCURSES_CH_T blank, int needclear)
 {
     int j;
 
@@ -1086,15 +1099,12 @@ ClrToEOL(NCURSES_SP_DCLx NCURSES_CH_T blank, bool needclear)
                }
            }
        }
-    } else {
-       needclear = TRUE;
     }
 
     if (needclear) {
        UpdateAttrs(SP_PARM, blank);
-       TPUTS_TRACE("clr_eol");
        if (clr_eol && SP_PARM->_el_cost <= (screen_columns(SP_PARM) - SP_PARM->_curscol)) {
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx clr_eol);
+           NCURSES_PUTP2("clr_eol", clr_eol);
        } else {
            int count = (screen_columns(SP_PARM) - SP_PARM->_curscol);
            while (count-- > 0)
@@ -1114,12 +1124,14 @@ ClrToEOS(NCURSES_SP_DCLx NCURSES_CH_T blank)
 {
     int row, col;
 
-    if (0 == SP_PARM)
-       return;
-
     row = SP_PARM->_cursrow;
     col = SP_PARM->_curscol;
 
+    if (row < 0)
+       row = 0;
+    if (col < 0)
+       col = 0;
+
     UpdateAttrs(SP_PARM, blank);
     TPUTS_TRACE("clr_eos");
     NCURSES_SP_NAME(tputs) (NCURSES_SP_ARGx
@@ -1191,7 +1203,7 @@ ClrBottom(NCURSES_SP_DCLx int total)
 #define xmc_turn_on(sp,a,b) ((((a)^(b)) & ~(a) & (sp)->_xmc_triggers) != 0)
 #endif
 
-#define xmc_new(sp,r,c) NewScreen(SP_PARM)->_line[r].text[c]
+#define xmc_new(sp,r,c) NewScreen(sp)->_line[r].text[c]
 #define xmc_turn_off(sp,a,b) xmc_turn_on(sp,b,a)
 #endif /* USE_XMC_SUPPORT */
 
@@ -1199,7 +1211,7 @@ ClrBottom(NCURSES_SP_DCLx int total)
 **     TransformLine(lineno)
 **
 **     Transform the given line in curscr to the one in newscr, using
-**     Insert/Delete Character if _nc_idcok && has_ic().
+**     Insert/Delete Character if idcok && has_ic().
 **
 **             firstChar = position of first different character in line
 **             oLastChar = position of last different character in old line
@@ -1222,7 +1234,7 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
     int n;
     bool attrchanged = FALSE;
 
-    TR(TRACE_UPDATE, (T_CALLED("TransformLine(%p, %d)"), SP_PARM, lineno));
+    TR(TRACE_UPDATE, (T_CALLED("TransformLine(%p, %d)"), (void *) SP_PARM, lineno));
 
     /* copy new hash value to old one */
     if (SP_PARM->oldhash && SP_PARM->newhash)
@@ -1271,8 +1283,8 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
     if (attrchanged) {         /* we may have to disregard the whole line */
        GoTo(NCURSES_SP_ARGx lineno, firstChar);
        ClrToEOL(NCURSES_SP_ARGx
-                ClrBlank(NCURSES_SP_ARGx CurScreen(SP_PARM)),
-                FALSE);
+                ClrBlank(NCURSES_SP_ARGx
+                         CurScreen(SP_PARM)), FALSE);
        PutRange(NCURSES_SP_ARGx
                 oldLine, newLine, lineno, 0,
                 screen_columns(SP_PARM) - 1);
@@ -1363,13 +1375,11 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
                        && SP_PARM->_el_cost <= SP_PARM->_el1_cost) {
                        GoTo(NCURSES_SP_ARGx lineno, 0);
                        UpdateAttrs(SP_PARM, blank);
-                       TPUTS_TRACE("clr_eol");
-                       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx clr_eol);
+                       NCURSES_PUTP2("clr_eol", clr_eol);
                    } else {
                        GoTo(NCURSES_SP_ARGx lineno, nFirstChar - 1);
                        UpdateAttrs(SP_PARM, blank);
-                       TPUTS_TRACE("clr_bol");
-                       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx clr_bol);
+                       NCURSES_PUTP2("clr_bol", clr_bol);
                    }
 
                    while (firstChar < nFirstChar)
@@ -1408,7 +1418,7 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
                         nLastChar);
                memcpy(oldLine + firstChar,
                       newLine + firstChar,
-                      (nLastChar - firstChar + 1) * sizeof(NCURSES_CH_T));
+                      (unsigned) (nLastChar - firstChar + 1) * sizeof(NCURSES_CH_T));
            }
            TR(TRACE_UPDATE, (T_RETURN("")));
            return;
@@ -1432,7 +1442,8 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
            ClrToEOL(NCURSES_SP_ARGx blank, FALSE);
        } else if ((nLastChar != oLastChar)
                   && (!CharEq(newLine[nLastChar], oldLine[oLastChar])
-                      || !(_nc_idcok && has_ic()))) {
+                      || !(SP_PARM->_nc_sp_idcok
+                           && NCURSES_SP_NAME(has_ic) (NCURSES_SP_ARG)))) {
            GoTo(NCURSES_SP_ARGx lineno, firstChar);
            if ((oLastChar - nLastChar) > SP_PARM->_el_cost) {
                if (PutRange(NCURSES_SP_ARGx
@@ -1484,9 +1495,17 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
            if (oLastChar < nLastChar) {
                int m = max(nLastNonblank, oLastNonblank);
 #if USE_WIDEC_SUPPORT
-               while (isWidecExt(newLine[n + 1]) && n) {
-                   --n;
-                   --oLastChar;
+               if (n) {
+                   while (isWidecExt(newLine[n + 1]) && n) {
+                       --n;
+                       --oLastChar;    /* increase cost */
+                   }
+               } else if (n >= firstChar &&
+                          isWidecBase(newLine[n])) {
+                   while (isWidecExt(newLine[n + 1])) {
+                       ++n;
+                       ++oLastChar;    /* decrease cost */
+                   }
                }
 #endif
                GoTo(NCURSES_SP_ARGx lineno, n + 1);
@@ -1506,8 +1525,9 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
                if (DelCharCost(SP_PARM, oLastChar - nLastChar)
                    > SP_PARM->_el_cost + nLastNonblank - (n + 1)) {
                    if (PutRange(NCURSES_SP_ARGx oldLine, newLine, lineno,
-                                n + 1, nLastNonblank))
-                         GoTo(NCURSES_SP_ARGx lineno, nLastNonblank + 1);
+                                n + 1, nLastNonblank)) {
+                       GoTo(NCURSES_SP_ARGx lineno, nLastNonblank + 1);
+                   }
                    ClrToEOL(NCURSES_SP_ARGx blank, FALSE);
                } else {
                    /*
@@ -1529,7 +1549,7 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
     if (screen_columns(SP_PARM) > firstChar)
        memcpy(oldLine + firstChar,
               newLine + firstChar,
-              (screen_columns(SP_PARM) - firstChar) * sizeof(NCURSES_CH_T));
+              (unsigned) (screen_columns(SP_PARM) - firstChar) * sizeof(NCURSES_CH_T));
     TR(TRACE_UPDATE, (T_RETURN("")));
     return;
 }
@@ -1553,7 +1573,7 @@ ClearScreen(NCURSES_SP_DCLx NCURSES_CH_T blank)
     if (SP_PARM->_coloron
        && !SP_PARM->_default_color) {
        NCURSES_SP_NAME(_nc_do_color) (NCURSES_SP_ARGx
-                                      GET_SCREEN_PAIR(SP_PARM),
+                                      (short) GET_SCREEN_PAIR(SP_PARM),
                                       0,
                                       FALSE,
                                       NCURSES_SP_NAME(_nc_outch));
@@ -1566,17 +1586,15 @@ ClearScreen(NCURSES_SP_DCLx NCURSES_CH_T blank)
     if (fast_clear) {
        if (clear_screen) {
            UpdateAttrs(SP_PARM, blank);
-           TPUTS_TRACE("clear_screen");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx clear_screen);
+           NCURSES_PUTP2("clear_screen", clear_screen);
            SP_PARM->_cursrow = SP_PARM->_curscol = 0;
-           position_check(SP_PARM,
+           position_check(NCURSES_SP_ARGx
                           SP_PARM->_cursrow,
                           SP_PARM->_curscol,
                           "ClearScreen");
        } else if (clr_eos) {
            SP_PARM->_cursrow = SP_PARM->_curscol = -1;
            GoTo(NCURSES_SP_ARGx 0, 0);
-
            UpdateAttrs(SP_PARM, blank);
            TPUTS_TRACE("clr_eos");
            NCURSES_SP_NAME(tputs) (NCURSES_SP_ARGx
@@ -1585,12 +1603,10 @@ ClearScreen(NCURSES_SP_DCLx NCURSES_CH_T blank)
                                    NCURSES_SP_NAME(_nc_outch));
        } else if (clr_eol) {
            SP_PARM->_cursrow = SP_PARM->_curscol = -1;
-
            UpdateAttrs(SP_PARM, blank);
            for (i = 0; i < screen_lines(SP_PARM); i++) {
                GoTo(NCURSES_SP_ARGx i, 0);
-               TPUTS_TRACE("clr_eol");
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx clr_eol);
+               NCURSES_PUTP2("clr_eol", clr_eol);
            }
            GoTo(NCURSES_SP_ARGx 0, 0);
        }
@@ -1622,7 +1638,9 @@ ClearScreen(NCURSES_SP_DCLx NCURSES_CH_T blank)
 static void
 InsStr(NCURSES_SP_DCLx NCURSES_CH_T * line, int count)
 {
-    TR(TRACE_UPDATE, ("InsStr(%p, %p,%d) called", SP_PARM, line, count));
+    TR(TRACE_UPDATE, ("InsStr(%p, %p,%d) called",
+                     (void *) SP_PARM,
+                     (void *) line, count));
 
     /* Prefer parm_ich as it has the smallest cost - no need to shift
      * the whole line on each character. */
@@ -1639,33 +1657,30 @@ InsStr(NCURSES_SP_DCLx NCURSES_CH_T * line, int count)
            count--;
        }
     } else if (enter_insert_mode && exit_insert_mode) {
-       TPUTS_TRACE("enter_insert_mode");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx enter_insert_mode);
+       NCURSES_PUTP2("enter_insert_mode", enter_insert_mode);
        while (count) {
            PutAttrChar(NCURSES_SP_ARGx CHREF(*line));
            if (insert_padding) {
-               TPUTS_TRACE("insert_padding");
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx insert_padding);
+               NCURSES_PUTP2("insert_padding", insert_padding);
            }
            line++;
            count--;
        }
-       TPUTS_TRACE("exit_insert_mode");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx exit_insert_mode);
+       NCURSES_PUTP2("exit_insert_mode", exit_insert_mode);
     } else {
        while (count) {
-           TPUTS_TRACE("insert_character");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx insert_character);
+           NCURSES_PUTP2("insert_character", insert_character);
            PutAttrChar(NCURSES_SP_ARGx CHREF(*line));
            if (insert_padding) {
-               TPUTS_TRACE("insert_padding");
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx insert_padding);
+               NCURSES_PUTP2("insert_padding", insert_padding);
            }
            line++;
            count--;
        }
     }
-    position_check(SP_PARM, SP_PARM->_cursrow, SP_PARM->_curscol, "InsStr");
+    position_check(NCURSES_SP_ARGx
+                  SP_PARM->_cursrow,
+                  SP_PARM->_curscol, "InsStr");
 }
 
 /*
@@ -1681,7 +1696,7 @@ DelChar(NCURSES_SP_DCLx int count)
     int n;
 
     TR(TRACE_UPDATE, ("DelChar(%p, %d) called, position = (%ld,%ld)",
-                     SP_PARM, count,
+                     (void *) SP_PARM, count,
                      (long) NewScreen(SP_PARM)->_cury,
                      (long) NewScreen(SP_PARM)->_curx));
 
@@ -1693,8 +1708,7 @@ DelChar(NCURSES_SP_DCLx int count)
                                NCURSES_SP_NAME(_nc_outch));
     } else {
        for (n = 0; n < count; n++) {
-           TPUTS_TRACE("delete_character");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx delete_character);
+           NCURSES_PUTP2("delete_character", delete_character);
        }
     }
 }
@@ -1746,13 +1760,11 @@ scroll_csr_forward(NCURSES_SP_DCLx
     if (n == 1 && scroll_forward && top == miny && bot == maxy) {
        GoTo(NCURSES_SP_ARGx bot, 0);
        UpdateAttrs(SP_PARM, blank);
-       TPUTS_TRACE("scroll_forward");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx scroll_forward);
+       NCURSES_PUTP2("scroll_forward", scroll_forward);
     } else if (n == 1 && delete_line && bot == maxy) {
        GoTo(NCURSES_SP_ARGx top, 0);
        UpdateAttrs(SP_PARM, blank);
-       TPUTS_TRACE("delete_line");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx delete_line);
+       NCURSES_PUTP2("delete_line", delete_line);
     } else if (parm_index && top == miny && bot == maxy) {
        GoTo(NCURSES_SP_ARGx bot, 0);
        UpdateAttrs(SP_PARM, blank);
@@ -1773,15 +1785,13 @@ scroll_csr_forward(NCURSES_SP_DCLx
        GoTo(NCURSES_SP_ARGx bot, 0);
        UpdateAttrs(SP_PARM, blank);
        for (i = 0; i < n; i++) {
-           TPUTS_TRACE("scroll_forward");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx scroll_forward);
+           NCURSES_PUTP2("scroll_forward", scroll_forward);
        }
     } else if (delete_line && bot == maxy) {
        GoTo(NCURSES_SP_ARGx top, 0);
        UpdateAttrs(SP_PARM, blank);
        for (i = 0; i < n; i++) {
-           TPUTS_TRACE("delete_line");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx delete_line);
+           NCURSES_PUTP2("delete_line", delete_line);
        }
     } else
        return ERR;
@@ -1815,13 +1825,11 @@ scroll_csr_backward(NCURSES_SP_DCLx
     if (n == 1 && scroll_reverse && top == miny && bot == maxy) {
        GoTo(NCURSES_SP_ARGx top, 0);
        UpdateAttrs(SP_PARM, blank);
-       TPUTS_TRACE("scroll_reverse");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx scroll_reverse);
+       NCURSES_PUTP2("scroll_reverse", scroll_reverse);
     } else if (n == 1 && insert_line && bot == maxy) {
        GoTo(NCURSES_SP_ARGx top, 0);
        UpdateAttrs(SP_PARM, blank);
-       TPUTS_TRACE("insert_line");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx insert_line);
+       NCURSES_PUTP2("insert_line", insert_line);
     } else if (parm_rindex && top == miny && bot == maxy) {
        GoTo(NCURSES_SP_ARGx top, 0);
        UpdateAttrs(SP_PARM, blank);
@@ -1842,15 +1850,13 @@ scroll_csr_backward(NCURSES_SP_DCLx
        GoTo(NCURSES_SP_ARGx top, 0);
        UpdateAttrs(SP_PARM, blank);
        for (i = 0; i < n; i++) {
-           TPUTS_TRACE("scroll_reverse");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx scroll_reverse);
+           NCURSES_PUTP2("scroll_reverse", scroll_reverse);
        }
     } else if (insert_line && bot == maxy) {
        GoTo(NCURSES_SP_ARGx top, 0);
        UpdateAttrs(SP_PARM, blank);
        for (i = 0; i < n; i++) {
-           TPUTS_TRACE("insert_line");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx insert_line);
+           NCURSES_PUTP2("insert_line", insert_line);
        }
     } else
        return ERR;
@@ -1881,8 +1887,7 @@ scroll_idl(NCURSES_SP_DCLx int n, int del, int ins, NCURSES_CH_T blank)
     GoTo(NCURSES_SP_ARGx del, 0);
     UpdateAttrs(SP_PARM, blank);
     if (n == 1 && delete_line) {
-       TPUTS_TRACE("delete_line");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx delete_line);
+       NCURSES_PUTP2("delete_line", delete_line);
     } else if (parm_delete_line) {
        TPUTS_TRACE("parm_delete_line");
        NCURSES_SP_NAME(tputs) (NCURSES_SP_ARGx
@@ -1891,16 +1896,14 @@ scroll_idl(NCURSES_SP_DCLx int n, int del, int ins, NCURSES_CH_T blank)
                                NCURSES_SP_NAME(_nc_outch));
     } else {                   /* if (delete_line) */
        for (i = 0; i < n; i++) {
-           TPUTS_TRACE("delete_line");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx delete_line);
+           NCURSES_PUTP2("delete_line", delete_line);
        }
     }
 
     GoTo(NCURSES_SP_ARGx ins, 0);
     UpdateAttrs(SP_PARM, blank);
     if (n == 1 && insert_line) {
-       TPUTS_TRACE("insert_line");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx insert_line);
+       NCURSES_PUTP2("insert_line", insert_line);
     } else if (parm_insert_line) {
        TPUTS_TRACE("parm_insert_line");
        NCURSES_SP_NAME(tputs) (NCURSES_SP_ARGx
@@ -1909,8 +1912,7 @@ scroll_idl(NCURSES_SP_DCLx int n, int del, int ins, NCURSES_CH_T blank)
                                NCURSES_SP_NAME(_nc_outch));
     } else {                   /* if (insert_line) */
        for (i = 0; i < n; i++) {
-           TPUTS_TRACE("insert_line");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx insert_line);
+           NCURSES_PUTP2("insert_line", insert_line);
        }
     }
 
@@ -1933,12 +1935,18 @@ NCURSES_SP_NAME(_nc_scrolln) (NCURSES_SP_DCLx
                              int maxy)
 /* scroll region from top to bot by n lines */
 {
-    NCURSES_CH_T blank = ClrBlank(NCURSES_SP_ARGx StdScreen(SP_PARM));
+    NCURSES_CH_T blank;
     int i;
     bool cursor_saved = FALSE;
     int res;
 
-    TR(TRACE_MOVE, ("mvcur_scrolln(%d, %d, %d, %d)", n, top, bot, maxy));
+    TR(TRACE_MOVE, ("_nc_scrolln(%p, %d, %d, %d, %d)",
+                   (void *) SP_PARM, n, top, bot, maxy));
+
+    if (!IsValidScreen(SP_PARM))
+       return (ERR);
+
+    blank = ClrBlank(NCURSES_SP_ARGx StdScreen(SP_PARM));
 
 #if USE_XMC_SUPPORT
     /*
@@ -1961,28 +1969,24 @@ NCURSES_SP_NAME(_nc_scrolln) (NCURSES_SP_DCLx
                 && (SP_PARM->_cursrow == bot || SP_PARM->_cursrow == bot - 1))
                && save_cursor && restore_cursor) {
                cursor_saved = TRUE;
-               TPUTS_TRACE("save_cursor");
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx save_cursor);
+               NCURSES_PUTP2("save_cursor", save_cursor);
            }
-           TPUTS_TRACE("change_scroll_region");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx
-                                  TPARM_2(change_scroll_region, top, bot));
+           NCURSES_PUTP2("change_scroll_region",
+                         TPARM_2(change_scroll_region, top, bot));
            if (cursor_saved) {
-               TPUTS_TRACE("restore_cursor");
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx restore_cursor);
+               NCURSES_PUTP2("restore_cursor", restore_cursor);
            } else {
                SP_PARM->_cursrow = SP_PARM->_curscol = -1;
            }
 
            res = scroll_csr_forward(NCURSES_SP_ARGx n, top, bot, top, bot, blank);
 
-           TPUTS_TRACE("change_scroll_region");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx
-                                  TPARM_2(change_scroll_region, 0, maxy));
+           NCURSES_PUTP2("change_scroll_region",
+                         TPARM_2(change_scroll_region, 0, maxy));
            SP_PARM->_cursrow = SP_PARM->_curscol = -1;
        }
 
-       if (res == ERR && _nc_idlok)
+       if (res == ERR && SP_PARM->_nc_sp_idlok)
            res = scroll_idl(NCURSES_SP_ARGx n, top, bot - n + 1, blank);
 
        /*
@@ -2011,15 +2015,12 @@ NCURSES_SP_NAME(_nc_scrolln) (NCURSES_SP_DCLx
                    SP_PARM->_cursrow == top - 1)
                && save_cursor && restore_cursor) {
                cursor_saved = TRUE;
-               TPUTS_TRACE("save_cursor");
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx save_cursor);
+               NCURSES_PUTP2("save_cursor", save_cursor);
            }
-           TPUTS_TRACE("change_scroll_region");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx
-                                  TPARM_2(change_scroll_region, top, bot));
+           NCURSES_PUTP2("change_scroll_region",
+                         TPARM_2(change_scroll_region, top, bot));
            if (cursor_saved) {
-               TPUTS_TRACE("restore_cursor");
-               NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx restore_cursor);
+               NCURSES_PUTP2("restore_cursor", restore_cursor);
            } else {
                SP_PARM->_cursrow = SP_PARM->_curscol = -1;
            }
@@ -2027,13 +2028,12 @@ NCURSES_SP_NAME(_nc_scrolln) (NCURSES_SP_DCLx
            res = scroll_csr_backward(NCURSES_SP_ARGx
                                      -n, top, bot, top, bot, blank);
 
-           TPUTS_TRACE("change_scroll_region");
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx
-                                  TPARM_2(change_scroll_region, 0, maxy));
+           NCURSES_PUTP2("change_scroll_region",
+                         TPARM_2(change_scroll_region, 0, maxy));
            SP_PARM->_cursrow = SP_PARM->_curscol = -1;
        }
 
-       if (res == ERR && _nc_idlok)
+       if (res == ERR && SP_PARM->_nc_sp_idlok)
            res = scroll_idl(NCURSES_SP_ARGx -n, bot + n + 1, top, blank);
 
        /*
@@ -2052,7 +2052,10 @@ NCURSES_SP_NAME(_nc_scrolln) (NCURSES_SP_DCLx
     if (res == ERR)
        return (ERR);
 
-    _nc_scroll_window(CurScreen(SP_PARM), n, top, bot, blank);
+    _nc_scroll_window(CurScreen(SP_PARM), n,
+                     (NCURSES_SIZE_T) top,
+                     (NCURSES_SIZE_T) bot,
+                     blank);
 
     /* shift hash values too - they can be reused */
     NCURSES_SP_NAME(_nc_scroll_oldhash) (NCURSES_SP_ARGx n, top, bot);
@@ -2079,7 +2082,7 @@ NCURSES_SP_NAME(_nc_screen_resume) (NCURSES_SP_DCL0)
 
     /* reset color pairs and definitions */
     if (SP_PARM->_coloron || SP_PARM->_color_defs)
-       _nc_reset_colors();
+       NCURSES_SP_NAME(_nc_reset_colors) (NCURSES_SP_ARG);
 
     /* restore user-defined colors, if any */
     if (SP_PARM->_color_defs < 0) {
@@ -2087,7 +2090,8 @@ NCURSES_SP_NAME(_nc_screen_resume) (NCURSES_SP_DCL0)
        SP_PARM->_color_defs = -(SP_PARM->_color_defs);
        for (n = 0; n < SP_PARM->_color_defs; ++n) {
            if (SP_PARM->_color_table[n].init) {
-               NCURSES_SP_NAME(init_color) (NCURSES_SP_ARGx n,
+               NCURSES_SP_NAME(init_color) (NCURSES_SP_ARGx
+                                            (short) n,
                                             SP_PARM->_color_table[n].r,
                                             SP_PARM->_color_table[n].g,
                                             SP_PARM->_color_table[n].b);
@@ -2096,23 +2100,25 @@ NCURSES_SP_NAME(_nc_screen_resume) (NCURSES_SP_DCL0)
     }
 
     if (exit_attribute_mode)
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx exit_attribute_mode);
+       NCURSES_PUTP2("exit_attribute_mode", exit_attribute_mode);
     else {
        /* turn off attributes */
        if (exit_alt_charset_mode)
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx exit_alt_charset_mode);
+           NCURSES_PUTP2("exit_alt_charset_mode", exit_alt_charset_mode);
        if (exit_standout_mode)
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx exit_standout_mode);
+           NCURSES_PUTP2("exit_standout_mode", exit_standout_mode);
        if (exit_underline_mode)
-           NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx exit_underline_mode);
+           NCURSES_PUTP2("exit_underline_mode", exit_underline_mode);
     }
     if (exit_insert_mode)
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx exit_insert_mode);
-    if (enter_am_mode && exit_am_mode)
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx
-                              (auto_right_margin
-                               ? enter_am_mode
-                               : exit_am_mode));
+       NCURSES_PUTP2("exit_insert_mode", exit_insert_mode);
+    if (enter_am_mode && exit_am_mode) {
+       if (auto_right_margin) {
+           NCURSES_PUTP2("enter_am_mode", enter_am_mode);
+       } else {
+           NCURSES_PUTP2("exit_am_mode", exit_am_mode);
+       }
+    }
 }
 
 #if NCURSES_SP_FUNCS
@@ -2141,30 +2147,33 @@ _nc_screen_init(void)
 NCURSES_EXPORT(void)
 NCURSES_SP_NAME(_nc_screen_wrap) (NCURSES_SP_DCL0)
 {
-    if (SP_PARM == 0)
-       return;
+    if (SP_PARM != 0) {
 
-    UpdateAttrs(SP_PARM, normal);
+       UpdateAttrs(SP_PARM, normal);
 #if NCURSES_EXT_FUNCS
-    if (SP_PARM->_coloron
-       && !SP_PARM->_default_color) {
-       static const NCURSES_CH_T blank = NewChar(BLANK_TEXT);
-       SP_PARM->_default_color = TRUE;
-       NCURSES_SP_NAME(_nc_do_color) (NCURSES_SP_ARGx
-                                      -1, 0,
-                                      FALSE,
-                                      NCURSES_SP_NAME(_nc_outch));
-       SP_PARM->_default_color = FALSE;
-
-       mvcur(SP_PARM->_cursrow,
-             SP_PARM->_curscol,
-             screen_lines(SP_PARM) - 1, 0);
-
-       ClrToEOL(NCURSES_SP_ARGx blank, TRUE);
-    }
+       if (SP_PARM->_coloron
+           && !SP_PARM->_default_color) {
+           static const NCURSES_CH_T blank = NewChar(BLANK_TEXT);
+           SP_PARM->_default_color = TRUE;
+           NCURSES_SP_NAME(_nc_do_color) (NCURSES_SP_ARGx
+                                          -1,
+                                          0,
+                                          FALSE,
+                                          NCURSES_SP_NAME(_nc_outch));
+           SP_PARM->_default_color = FALSE;
+
+           TINFO_MVCUR(NCURSES_SP_ARGx
+                       SP_PARM->_cursrow,
+                       SP_PARM->_curscol,
+                       screen_lines(SP_PARM) - 1,
+                       0);
+
+           ClrToEOL(NCURSES_SP_ARGx blank, TRUE);
+       }
 #endif
-    if (SP_PARM->_color_defs) {
-       _nc_reset_colors();
+       if (SP_PARM->_color_defs) {
+           NCURSES_SP_NAME(_nc_reset_colors) (NCURSES_SP_ARG);
+       }
     }
 }