]> ncurses.scripts.mit.edu Git - ncurses.git/blobdiff - ncurses/tty/tty_update.c
ncurses 6.0 - patch 20170311
[ncurses.git] / ncurses / tty / tty_update.c
index 9be997389993818ed765078665f3b0844d809d97..4e43e2b016843ce0422ccba50c057a540153967b 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1998-2011,2012 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2016,2017 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            *
@@ -43,6 +43,8 @@
  *
  *-----------------------------------------------------------------*/
 
+#define NEW_PAIR_INTERNAL 1
+
 #include <curses.priv.h>
 
 #ifndef CUR
@@ -82,7 +84,7 @@
 
 #include <ctype.h>
 
-MODULE_ID("$Id: tty_update.c,v 1.272 2012/12/15 21:00:19 tom Exp $")
+MODULE_ID("$Id: tty_update.c,v 1.287 2017/02/28 22:13:45 tom Exp $")
 
 /*
  * This define controls the line-breakout optimization.  Every once in a
@@ -145,8 +147,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 +181,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 */
 
 /****************************************************************************
@@ -195,14 +196,87 @@ GoTo(NCURSES_SP_DCLx int const row, int const col)
     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");
 
     TINFO_MVCUR(NCURSES_SP_ARGx
                SP_PARM->_cursrow,
                SP_PARM->_curscol,
                row, col);
-    position_check(SP_PARM, SP_PARM->_cursrow, SP_PARM->_curscol, "GoTo2");
+    position_check(NCURSES_SP_ARGx
+                  SP_PARM->_cursrow,
+                  SP_PARM->_curscol, "GoTo2");
+}
+
+#if !NCURSES_WCWIDTH_GRAPHICS
+static bool
+is_wacs_value(unsigned ch)
+{
+    bool result;
+    switch (ch) {
+    case 0x00a3:               /* FALLTHRU - ncurses pound-sterling symbol */
+    case 0x00b0:               /* FALLTHRU - VT100 degree symbol */
+    case 0x00b1:               /* FALLTHRU - VT100 plus/minus */
+    case 0x00b7:               /* FALLTHRU - VT100 bullet */
+    case 0x03c0:               /* FALLTHRU - ncurses greek pi */
+    case 0x2190:               /* FALLTHRU - Teletype arrow pointing left */
+    case 0x2191:               /* FALLTHRU - Teletype arrow pointing up */
+    case 0x2192:               /* FALLTHRU - Teletype arrow pointing right */
+    case 0x2193:               /* FALLTHRU - Teletype arrow pointing down */
+    case 0x2260:               /* FALLTHRU - ncurses not-equal */
+    case 0x2264:               /* FALLTHRU - ncurses less-than-or-equal-to */
+    case 0x2265:               /* FALLTHRU - ncurses greater-than-or-equal-to */
+    case 0x23ba:               /* FALLTHRU - VT100 scan line 1 */
+    case 0x23bb:               /* FALLTHRU - ncurses scan line 3 */
+    case 0x23bc:               /* FALLTHRU - ncurses scan line 7 */
+    case 0x23bd:               /* FALLTHRU - VT100 scan line 9 */
+    case 0x2500:               /* FALLTHRU - VT100 horizontal line */
+    case 0x2501:               /* FALLTHRU - thick horizontal line */
+    case 0x2502:               /* FALLTHRU - VT100 vertical line */
+    case 0x2503:               /* FALLTHRU - thick vertical line */
+    case 0x250c:               /* FALLTHRU - VT100 upper left corner */
+    case 0x250f:               /* FALLTHRU - thick upper left corner */
+    case 0x2510:               /* FALLTHRU - VT100 upper right corner */
+    case 0x2513:               /* FALLTHRU - thick upper right corner */
+    case 0x2514:               /* FALLTHRU - VT100 lower left corner */
+    case 0x2517:               /* FALLTHRU - thick lower left corner */
+    case 0x2518:               /* FALLTHRU - VT100 lower right corner */
+    case 0x251b:               /* FALLTHRU - thick lower right corner */
+    case 0x251c:               /* FALLTHRU - VT100 tee pointing left */
+    case 0x2523:               /* FALLTHRU - thick tee pointing left */
+    case 0x2524:               /* FALLTHRU - VT100 tee pointing right */
+    case 0x252b:               /* FALLTHRU - thick tee pointing right */
+    case 0x252c:               /* FALLTHRU - VT100 tee pointing down */
+    case 0x2533:               /* FALLTHRU - thick tee pointing down */
+    case 0x2534:               /* FALLTHRU - VT100 tee pointing up */
+    case 0x253b:               /* FALLTHRU - thick tee pointing up */
+    case 0x253c:               /* FALLTHRU - VT100 large plus or crossover */
+    case 0x254b:               /* FALLTHRU - thick large plus or crossover */
+    case 0x2550:               /* FALLTHRU - double horizontal line */
+    case 0x2551:               /* FALLTHRU - double vertical line */
+    case 0x2554:               /* FALLTHRU - double upper left corner */
+    case 0x2557:               /* FALLTHRU - double upper right corner */
+    case 0x255a:               /* FALLTHRU - double lower left corner */
+    case 0x255d:               /* FALLTHRU - double lower right corner */
+    case 0x2560:               /* FALLTHRU - double tee pointing right */
+    case 0x2563:               /* FALLTHRU - double tee pointing left */
+    case 0x2566:               /* FALLTHRU - double tee pointing down */
+    case 0x2569:               /* FALLTHRU - double tee pointing up */
+    case 0x256c:               /* FALLTHRU - double large plus or crossover */
+    case 0x2592:               /* FALLTHRU - VT100 checker board (stipple) */
+    case 0x25ae:               /* FALLTHRU - Teletype solid square block */
+    case 0x25c6:               /* FALLTHRU - VT100 diamond */
+    case 0x2603:               /* FALLTHRU - Teletype lantern symbol */
+       result = TRUE;
+       break;
+    default:
+       result = FALSE;
+       break;
+    }
+    return result;
 }
+#endif
 
 static NCURSES_INLINE void
 PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
@@ -265,7 +339,11 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
 
     if ((AttrOf(attr) & A_ALTCHARSET)
        && SP_PARM->_acs_map != 0
-       && CharOfD(ch) < ACS_LEN) {
+       && ((CharOfD(ch) < ACS_LEN)
+#if !NCURSES_WCWIDTH_GRAPHICS
+           || is_wacs_value(CharOfD(ch))
+#endif
+       )) {
        my_ch = CHDEREF(ch);    /* work around const param */
 #if USE_WIDEC_SUPPORT
        /*
@@ -274,10 +352,22 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
         * character, and uses the wide-character mapping when we expect the
         * normal one to be broken (by mis-design ;-).
         */
-       if (SP_PARM->_screen_acs_fix
-           && SP_PARM->_screen_acs_map[CharOf(my_ch)]) {
-           RemAttr(attr, A_ALTCHARSET);
-           my_ch = _nc_wacs[CharOf(my_ch)];
+       if (SP_PARM->_screen_unicode
+           && _nc_wacs[CharOf(my_ch)].chars[0]) {
+           if (SP_PARM->_screen_acs_map[CharOf(my_ch)]) {
+               if (SP_PARM->_screen_acs_fix) {
+                   RemAttr(attr, A_ALTCHARSET);
+                   my_ch = _nc_wacs[CharOf(my_ch)];
+               }
+           } else {
+               RemAttr(attr, A_ALTCHARSET);
+               my_ch = _nc_wacs[CharOf(my_ch)];
+           }
+#if !NCURSES_WCWIDTH_GRAPHICS
+           if (!(AttrOf(attr) & A_ALTCHARSET)) {
+               chlen = 1;
+           }
+#endif /* !NCURSES_WCWIDTH_GRAPHICS */
        }
 #endif
        /*
@@ -299,6 +389,11 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
        }
        ch = CHREF(my_ch);
     }
+#if USE_WIDEC_SUPPORT && !NCURSES_WCWIDTH_GRAPHICS
+    else if (chlen > 1 && is_wacs_value(CharOfD(ch))) {
+       chlen = 1;
+    }
+#endif
     if (tilde_glitch && (CharOfD(ch) == L('~'))) {
        SetChar(tilde, L('`'), AttrOf(attr));
        ch = CHREF(tilde);
@@ -311,8 +406,7 @@ PutAttrChar(NCURSES_SP_DCLx CARG_CH_T ch)
 #endif
     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);
     }
 }
 
@@ -383,18 +477,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
@@ -448,7 +540,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");
@@ -468,7 +560,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");
 }
 
 /*
@@ -486,15 +580,17 @@ can_clear_with(NCURSES_SP_DCLx ARG_CH_T ch)
 
        if (!SP_PARM->_default_color)
            return FALSE;
-       if (SP_PARM->_default_fg != C_MASK || SP_PARM->_default_bg != C_MASK)
+       if (!(isDefaultColor(SP_PARM->_default_fg) &&
+             isDefaultColor(SP_PARM->_default_bg)))
            return FALSE;
        if ((pair = GetPair(CHDEREF(ch))) != 0) {
-           short fg, bg;
-           NCURSES_SP_NAME(pair_content) (NCURSES_SP_ARGx
-                                          (short) 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
+               || !(isDefaultColor(fg) && isDefaultColor(bg))) {
                return FALSE;
+           }
        }
 #else
        if (AttrOfD(ch) & A_COLOR)
@@ -556,8 +652,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,
@@ -584,7 +679,7 @@ EmitRange(NCURSES_SP_DCLx const NCURSES_CH_T * ntext, int num)
                                        TPARM_2(repeat_char,
                                                CharOf(ntext0),
                                                rep_count),
-                                       rep_count,
+                                       1,
                                        NCURSES_SP_NAME(_nc_outch));
                SP_PARM->_curscol += rep_count;
 
@@ -620,7 +715,7 @@ PutRange(NCURSES_SP_DCLx
         int row,
         int first, int last)
 {
-    int i, j, same;
+    int rc;
 
     TR(TRACE_CHARPUT, ("PutRange(%p, %p, %p, %d, %d, %d)",
                       (void *) SP_PARM,
@@ -630,6 +725,8 @@ PutRange(NCURSES_SP_DCLx
 
     if (otext != ntext
        && (last - first + 1) > SP_PARM->_inline_cost) {
+       int i, j, same;
+
        for (j = first, same = 0; j <= last; j++) {
            if (!same && isWidecExt(otext[j]))
                continue;
@@ -648,9 +745,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 */
@@ -1074,10 +1173,10 @@ ClrUpdate(NCURSES_SP_DCL0)
 static void
 ClrToEOL(NCURSES_SP_DCLx NCURSES_CH_T blank, int needclear)
 {
-    int j;
-
     if (CurScreen(SP_PARM) != 0
        && SP_PARM->_cursrow >= 0) {
+       int j;
+
        for (j = SP_PARM->_curscol; j < screen_columns(SP_PARM); j++) {
            if (j >= 0) {
                NCURSES_CH_T *cp =
@@ -1093,9 +1192,8 @@ ClrToEOL(NCURSES_SP_DCLx NCURSES_CH_T blank, int needclear)
 
     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)
@@ -1118,6 +1216,11 @@ ClrToEOS(NCURSES_SP_DCLx NCURSES_CH_T blank)
     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
@@ -1144,16 +1247,17 @@ ClrToEOS(NCURSES_SP_DCLx NCURSES_CH_T blank)
 static int
 ClrBottom(NCURSES_SP_DCLx int total)
 {
-    int row;
-    int col;
     int top = total;
     int last = min(screen_columns(SP_PARM), NewScreen(SP_PARM)->_maxx + 1);
     NCURSES_CH_T blank = NewScreen(SP_PARM)->_line[total - 1].text[last - 1];
-    bool ok;
 
     if (clr_eos && can_clear_with(NCURSES_SP_ARGx CHREF(blank))) {
+       int row;
 
        for (row = total - 1; row >= 0; row--) {
+           int col;
+           bool ok;
+
            for (col = 0, ok = TRUE; ok && col < last; col++) {
                ok = (CharEq(NewScreen(SP_PARM)->_line[row].text[col], blank));
            }
@@ -1244,8 +1348,8 @@ TransformLine(NCURSES_SP_DCLx int const lineno)
                    && unColor(oldLine[n]) == unColor(newLine[n])) {
                    if (oldPair < SP_PARM->_pair_limit
                        && newPair < SP_PARM->_pair_limit
-                       && (SP_PARM->_color_pairs[oldPair] ==
-                           SP_PARM->_color_pairs[newPair])) {
+                       && (isSamePair(SP_PARM->_color_pairs[oldPair],
+                                      SP_PARM->_color_pairs[newPair]))) {
                        SetPair(oldLine[n], GetPair(newLine[n]));
                    }
                }
@@ -1361,13 +1465,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)
@@ -1483,9 +1585,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);
@@ -1505,8 +1615,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 {
                    /*
@@ -1565,10 +1676,9 @@ 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");
@@ -1586,8 +1696,7 @@ ClearScreen(NCURSES_SP_DCLx NCURSES_CH_T blank)
            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);
        }
@@ -1630,41 +1739,38 @@ InsStr(NCURSES_SP_DCLx NCURSES_CH_T * line, int count)
        TPUTS_TRACE("parm_ich");
        NCURSES_SP_NAME(tputs) (NCURSES_SP_ARGx
                                TPARM_1(parm_ich, count),
-                               count,
+                               1,
                                NCURSES_SP_NAME(_nc_outch));
-       while (count) {
+       while (count > 0) {
            PutAttrChar(NCURSES_SP_ARGx CHREF(*line));
            line++;
            count--;
        }
     } else if (enter_insert_mode && exit_insert_mode) {
-       TPUTS_TRACE("enter_insert_mode");
-       NCURSES_SP_NAME(putp) (NCURSES_SP_ARGx enter_insert_mode);
-       while (count) {
+       NCURSES_PUTP2("enter_insert_mode", enter_insert_mode);
+       while (count > 0) {
            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);
+       while (count > 0) {
+           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");
 }
 
 /*
@@ -1677,8 +1783,6 @@ InsStr(NCURSES_SP_DCLx NCURSES_CH_T * line, int count)
 static void
 DelChar(NCURSES_SP_DCLx int count)
 {
-    int n;
-
     TR(TRACE_UPDATE, ("DelChar(%p, %d) called, position = (%ld,%ld)",
                      (void *) SP_PARM, count,
                      (long) NewScreen(SP_PARM)->_cury,
@@ -1688,12 +1792,13 @@ DelChar(NCURSES_SP_DCLx int count)
        TPUTS_TRACE("parm_dch");
        NCURSES_SP_NAME(tputs) (NCURSES_SP_ARGx
                                TPARM_1(parm_dch, count),
-                               count,
+                               1,
                                NCURSES_SP_NAME(_nc_outch));
     } else {
+       int n;
+
        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);
        }
     }
 }
@@ -1745,13 +1850,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);
@@ -1772,15 +1875,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;
@@ -1814,13 +1915,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);
@@ -1841,15 +1940,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;
@@ -1880,8 +1977,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
@@ -1890,16 +1986,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
@@ -1908,8 +2002,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);
        }
     }
 
@@ -1966,24 +2059,20 @@ 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;
        }
 
@@ -2016,15 +2105,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;
            }
@@ -2032,9 +2118,8 @@ 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;
        }
 
@@ -2105,23 +2190,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