ncurses 5.6 - patch 20080920
[ncurses.git] / ncurses / tty / tty_update.c
index 408ebe3be7d05c8fa60f5dd0ad37d64b03f1e86c..6a3a0c8e9341ece94e67edc7ec78db47a5845c82 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1998-2004,2005 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2007,2008 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            *
 
 #include <curses.priv.h>
 
+#if defined __HAIKU__ && defined __BEOS__
+#undef __BEOS__
+#endif
+
 #ifdef __BEOS__
 #undef false
 #undef true
@@ -74,7 +78,7 @@
 #include <ctype.h>
 #include <term.h>
 
-MODULE_ID("$Id: tty_update.c,v 1.220 2005/08/13 17:12:32 tom Exp $")
+MODULE_ID("$Id: tty_update.c,v 1.246 2008/08/30 20:08:19 tom Exp $")
 
 /*
  * This define controls the line-breakout optimization.  Every once in a
@@ -89,7 +93,8 @@ MODULE_ID("$Id: tty_update.c,v 1.220 2005/08/13 17:12:32 tom Exp $")
 
 #define FILL_BCE() (SP->_coloron && !SP->_default_color && !back_color_erase)
 
-static const NCURSES_CH_T normal = NewChar2(BLANK_TEXT, BLANK_ATTR);
+static const NCURSES_CH_T blankchar = NewChar(BLANK_TEXT);
+static NCURSES_CH_T normal = NewChar(BLANK_TEXT);
 
 /*
  * Enable checking to see if doupdate and friends are tracking the true
@@ -98,7 +103,7 @@ static const NCURSES_CH_T normal = NewChar2(BLANK_TEXT, BLANK_ATTR);
  */
 /* #define POSITION_DEBUG */
 
-static inline NCURSES_CH_T ClrBlank(WINDOW *win);
+static NCURSES_INLINE NCURSES_CH_T ClrBlank(WINDOW *win);
 static int ClrBottom(int total);
 static void ClearScreen(NCURSES_CH_T blank);
 static void ClrUpdate(void);
@@ -166,7 +171,7 @@ position_check(int expected_y, int expected_x, char *legend)
  *
  ****************************************************************************/
 
-static inline void
+static NCURSES_INLINE void
 GoTo(int const row, int const col)
 {
     TR(TRACE_MOVE, ("GoTo(%d, %d) from (%d, %d)",
@@ -178,7 +183,7 @@ GoTo(int const row, int const col)
     position_check(SP->_cursrow, SP->_curscol, "GoTo2");
 }
 
-static inline void
+static NCURSES_INLINE void
 PutAttrChar(CARG_CH_T ch)
 {
     int chlen = 1;
@@ -203,11 +208,29 @@ PutAttrChar(CARG_CH_T ch)
      * on the screen.  It should be at least one.
      */
     if ((chlen = wcwidth(CharOf(CHDEREF(ch)))) <= 0) {
-       static NCURSES_CH_T blank = NewChar(BLANK_TEXT);
+       static const NCURSES_CH_T blank = NewChar(BLANK_TEXT);
 
+       /*
+        * If the character falls into any of these special cases, do
+        * not force the result to a blank:
+        *
+        * a) it is printable (this works around a bug in wcwidth()).
+        * b) use_legacy_coding() has been called to modify the treatment
+        *    of codes 128-255.
+        * c) the acs_map[] has been initialized to allow codes 0-31
+        *    to be rendered.  This supports Linux console's "PC"
+        *    characters.  Codes 128-255 are allowed though this is
+        *    not checked.
+        */
        if (is8bits(CharOf(CHDEREF(ch)))
            && (isprint(CharOf(CHDEREF(ch)))
-               || (SP->_legacy_coding && CharOf(CHDEREF(ch)) >= 160))) {
+               || (SP->_legacy_coding > 0 && CharOf(CHDEREF(ch)) >= 160)
+               || (SP->_legacy_coding > 1 && CharOf(CHDEREF(ch)) >= 128)
+               || (AttrOf(attr) & A_ALTCHARSET
+                   && ((CharOfD(ch) < ACS_LEN
+                        && SP->_acs_map != 0
+                        && SP->_acs_map[CharOfD(ch)] != 0)
+                       || (CharOfD(ch) >= 128))))) {
            ;
        } else {
            ch = CHREF(blank);
@@ -244,8 +267,11 @@ PutAttrChar(CARG_CH_T ch)
            int j = CharOfD(ch);
            chtype temp = UChar(SP->_acs_map[j]);
 
-           if (!(SP->_screen_acs_map[j]))
+           if (!(SP->_screen_acs_map[j])) {
                RemAttr(attr, A_ALTCHARSET);
+               if (temp == 0)
+                   temp = ' ';
+           }
            if (temp != 0)
                SetChar(my_ch, temp, AttrOf(attr));
        }
@@ -267,7 +293,7 @@ PutAttrChar(CARG_CH_T ch)
 #endif
     {
        PUTC(CHDEREF(ch), SP->_ofp);    /* macro's fastest... */
-       TRACE_OUTCHARS(1);
+       COUNT_OUTCHARS(1);
     }
     SP->_curscol += chlen;
     if (char_padding) {
@@ -361,38 +387,48 @@ PutCharLR(const ARG_CH_T ch)
     }
 }
 
+/*
+ * Wrap the cursor position, i.e., advance to the beginning of the next line.
+ */
 static void
 wrap_cursor(void)
 {
     if (eat_newline_glitch) {
        /*
-        * xenl can manifest two different ways.  The vt100
-        * way is that, when you'd expect the cursor to wrap,
-        * it stays hung at the right margin (on top of the
-        * character just emitted) and doesn't wrap until the
-        * *next* graphic char is emitted.  The c100 way is
-        * to ignore LF received just after an am wrap.
+        * xenl can manifest two different ways.  The vt100 way is that, when
+        * you'd expect the cursor to wrap, it stays hung at the right margin
+        * (on top of the character just emitted) and doesn't wrap until the
+        * *next* graphic char is emitted.  The c100 way is to ignore LF
+        * received just after an am wrap.
         *
-        * An aggressive way to handle this would be to
-        * emit CR/LF after the char and then assume the wrap
-        * is done, you're on the first position of the next
-        * line, and the terminal out of its weird state.
-        * Here it's safe to just tell the code that the
-        * cursor is in hyperspace and let the next mvcur()
-        * call straighten things out.
+        * An aggressive way to handle this would be to emit CR/LF after the
+        * char and then assume the wrap is done, you're on the first position
+        * of the next line, and the terminal out of its weird state.  Here
+        * it's safe to just tell the code that the cursor is in hyperspace and
+        * let the next mvcur() call straighten things out.
         */
        SP->_curscol = -1;
        SP->_cursrow = -1;
     } else if (auto_right_margin) {
        SP->_curscol = 0;
        SP->_cursrow++;
+       /*
+        * We've actually moved - but may have to work around problems with
+        * video attributes not working.
+        */
+       if (!move_standout_mode && AttrOf(SCREEN_ATTRS(SP))) {
+           TR(TRACE_CHARPUT, ("turning off (%#lx) %s before wrapping",
+                              (unsigned long) AttrOf(SCREEN_ATTRS(SP)),
+                              _traceattr(AttrOf(SCREEN_ATTRS(SP)))));
+           (void) VIDATTR(A_NORMAL, 0);
+       }
     } else {
        SP->_curscol--;
     }
     position_check(SP->_cursrow, SP->_curscol, "wrap_cursor");
 }
 
-static inline void
+static NCURSES_INLINE void
 PutChar(const ARG_CH_T ch)
 /* insert character, handling automargin stuff */
 {
@@ -413,7 +449,7 @@ PutChar(const ARG_CH_T ch)
  * as A_REVERSE.  All attribute flags which don't affect appearance of a space
  * or can be output by clearing (A_COLOR in case of bce-terminal) are excluded.
  */
-static inline bool
+static NCURSES_INLINE bool
 can_clear_with(ARG_CH_T ch)
 {
     if (!back_color_erase && SP->_coloron) {
@@ -456,6 +492,8 @@ EmitRange(const NCURSES_CH_T * ntext, int num)
 {
     int i;
 
+    TR(TRACE_CHARPUT, ("EmitRange %d:%s", num, _nc_viscbuf(ntext, num)));
+
     if (erase_chars || repeat_char) {
        while (num > 0) {
            int runcount;
@@ -488,7 +526,7 @@ EmitRange(const NCURSES_CH_T * ntext, int num)
                && runcount > SP->_ech_cost + SP->_cup_ch_cost
                && can_clear_with(CHREF(ntext0))) {
                UpdateAttrs(ntext0);
-               putp(tparm(erase_chars, runcount));
+               putp(TPARM_1(erase_chars, runcount));
 
                /*
                 * If this is the last part of the given interval,
@@ -508,7 +546,7 @@ EmitRange(const NCURSES_CH_T * ntext, int num)
                    rep_count--;
 
                UpdateAttrs(ntext0);
-               tputs(tparm(repeat_char, CharOf(ntext0), rep_count),
+               tputs(TPARM_2(repeat_char, CharOf(ntext0), rep_count),
                      rep_count, _nc_outch);
                SP->_curscol += rep_count;
 
@@ -590,13 +628,18 @@ doupdate(void)
 
     T((T_CALLED("doupdate()")));
 
+    if (curscr == 0
+       || newscr == 0)
+       returnCode(ERR);
+
 #ifdef TRACE
-    if (_nc_tracing & TRACE_UPDATE) {
+    if (USE_TRACEF(TRACE_UPDATE)) {
        if (curscr->_clear)
            _tracef("curscr is clear");
        else
            _tracedump("curscr", curscr);
        _tracedump("newscr", newscr);
+       _nc_unlock_global(tracef);
     }
 #endif /* TRACE */
 
@@ -606,7 +649,7 @@ doupdate(void)
        SP->_fifohold--;
 
 #if USE_SIZECHANGE
-    if (SP->_endwin || SP->_sig_winch) {
+    if (SP->_endwin || _nc_handle_sigwinch(SP)) {
        /*
         * This is a transparent extension:  XSI does not address it,
         * and applications need not know that ncurses can do it.
@@ -615,7 +658,7 @@ doupdate(void)
         * (this can happen in an xterm, for example), and resize the
         * ncurses data structures accordingly.
         */
-       _nc_update_screensize();
+       _nc_update_screensize(SP);
     }
 #endif
 
@@ -632,22 +675,20 @@ doupdate(void)
     }
 #if USE_TRACE_TIMES
     /* zero the metering machinery */
-    _nc_outchars = 0;
+    RESET_OUTCHARS();
     (void) times(&before);
 #endif /* USE_TRACE_TIMES */
 
     /*
-     * This is the support for magic-cookie terminals.  The
-     * theory: we scan the virtual screen looking for attribute
-     * turnons.  Where we find one, check to make sure it's
-     * realizable by seeing if the required number of
-     * un-attributed blanks are present before and after the
-     * attributed range; try to shift the range boundaries over
-     * blanks (not changing the screen display) so this becomes
-     * true.  If it is, shift the beginning attribute change
-     * appropriately (the end one, if we've gotten this far, is
-     * guaranteed room for its cookie). If not, nuke the added
-     * attributes out of the span.
+     * This is the support for magic-cookie terminals.  The theory:  we scan
+     * the virtual screen looking for attribute turnons.  Where we find one,
+     * check to make sure it's realizable by seeing if the required number of
+     * un-attributed blanks are present before and after the attributed range;
+     * try to shift the range boundaries over blanks (not changing the screen
+     * display) so this becomes true.  If it is, shift the beginning attribute
+     * change appropriately (the end one, if we've gotten this far, is
+     * guaranteed room for its cookie).  If not, nuke the added attributes out
+     * of the span.
      */
 #if USE_XMC_SUPPORT
     if (magic_cookie_glitch > 0) {
@@ -657,11 +698,13 @@ doupdate(void)
        for (i = 0; i < screen_lines; i++) {
            for (j = 0; j < screen_columns; j++) {
                bool failed = FALSE;
-               attr_t turnon = AttrOf(newscr->_line[i].text[j]) & ~rattr;
+               NCURSES_CH_T *thisline = newscr->_line[i].text;
+               attr_t thisattr = AttrOf(thisline[j]) & SP->_xmc_triggers;
+               attr_t turnon = thisattr & ~rattr;
 
                /* is an attribute turned on here? */
                if (turnon == 0) {
-                   rattr = AttrOf(newscr->_line[i].text[j]);
+                   rattr = thisattr;
                    continue;
                }
 
@@ -669,23 +712,33 @@ doupdate(void)
                TR(TRACE_ATTRS, ("...to %s", _traceattr(turnon)));
 
                /*
-                * If the attribute change location is a blank with a
-                * "safe" attribute, undo the attribute turnon.  This may
-                * ensure there's enough room to set the attribute before
-                * the first non-blank in the run.
+                * If the attribute change location is a blank with a "safe"
+                * attribute, undo the attribute turnon.  This may ensure
+                * there's enough room to set the attribute before the first
+                * non-blank in the run.
                 */
-#define SAFE(a)        (!((a) & (attr_t)~NONBLANK_ATTR))
-               if (ISBLANK(newscr->_line[i].text[j]) && SAFE(turnon)) {
-                   RemAttr(newscr->_line[i].text[j], turnon);
+#define SAFE(a)        (!((a) & SP->_xmc_triggers))
+               if (ISBLANK(thisline[j]) && SAFE(turnon)) {
+                   RemAttr(thisline[j], turnon);
                    continue;
                }
 
                /* check that there's enough room at start of span */
                for (k = 1; k <= magic_cookie_glitch; k++) {
                    if (j - k < 0
-                       || !ISBLANK(newscr->_line[i].text[j - k])
-                       || !SAFE(AttrOf(newscr->_line[i].text[j - k])))
+                       || !ISBLANK(thisline[j - k])
+                       || !SAFE(AttrOf(thisline[j - k]))) {
                        failed = TRUE;
+                       TR(TRACE_ATTRS, ("No room at start in %d,%d%s%s",
+                                        i, j - k,
+                                        (ISBLANK(thisline[j - k])
+                                         ? ""
+                                         : ":nonblank"),
+                                        (SAFE(AttrOf(thisline[j - k]))
+                                         ? ""
+                                         : ":unsafe")));
+                       break;
+                   }
                }
                if (!failed) {
                    bool end_onscreen = FALSE;
@@ -694,7 +747,8 @@ doupdate(void)
                    /* find end of span, if it's onscreen */
                    for (m = i; m < screen_lines; m++) {
                        for (; n < screen_columns; n++) {
-                           if (AttrOf(newscr->_line[m].text[n]) == rattr) {
+                           attr_t testattr = AttrOf(newscr->_line[m].text[n]);
+                           if ((testattr & SP->_xmc_triggers) == rattr) {
                                end_onscreen = TRUE;
                                TR(TRACE_ATTRS,
                                   ("Range attributed with %s ends at (%d, %d)",
@@ -713,22 +767,34 @@ doupdate(void)
                        NCURSES_CH_T *lastline = newscr->_line[m].text;
 
                        /*
-                        * If there are safely-attributed blanks at the
-                        * end of the range, shorten the range.  This will
-                        * help ensure that there is enough room at end
-                        * of span.
+                        * If there are safely-attributed blanks at the end of
+                        * the range, shorten the range.  This will help ensure
+                        * that there is enough room at end of span.
                         */
                        while (n >= 0
                               && ISBLANK(lastline[n])
-                              && SAFE(AttrOf(lastline[n])))
+                              && SAFE(AttrOf(lastline[n]))) {
                            RemAttr(lastline[n--], turnon);
+                       }
 
                        /* check that there's enough room at end of span */
-                       for (k = 1; k <= magic_cookie_glitch; k++)
+                       for (k = 1; k <= magic_cookie_glitch; k++) {
                            if (n + k >= screen_columns
                                || !ISBLANK(lastline[n + k])
-                               || !SAFE(AttrOf(lastline[n + k])))
+                               || !SAFE(AttrOf(lastline[n + k]))) {
                                failed = TRUE;
+                               TR(TRACE_ATTRS,
+                                  ("No room at end in %d,%d%s%s",
+                                   i, j - k,
+                                   (ISBLANK(lastline[n + k])
+                                    ? ""
+                                    : ":nonblank"),
+                                   (SAFE(AttrOf(lastline[n + k]))
+                                    ? ""
+                                    : ":unsafe")));
+                               break;
+                           }
+                       }
                    }
                }
 
@@ -742,7 +808,8 @@ doupdate(void)
                    /* turn off new attributes over span */
                    for (p = i; p < screen_lines; p++) {
                        for (; q < screen_columns; q++) {
-                           if (AttrOf(newscr->_line[p].text[q]) == rattr)
+                           attr_t testattr = AttrOf(newscr->_line[p].text[q]);
+                           if ((testattr & SP->_xmc_triggers) == rattr)
                                goto foundend;
                            RemAttr(newscr->_line[p].text[q], turnon);
                        }
@@ -755,22 +822,23 @@ doupdate(void)
                        _traceattr(turnon), i, j));
 
                    /*
-                    * back up the start of range so there's room
-                    * for cookies before the first nonblank character
+                    * Back up the start of range so there's room for cookies
+                    * before the first nonblank character.
                     */
                    for (k = 1; k <= magic_cookie_glitch; k++)
-                       AddAttr(newscr->_line[i].text[j - k], turnon);
+                       AddAttr(thisline[j - k], turnon);
                }
 
-               rattr = AttrOf(newscr->_line[i].text[j]);
+               rattr = thisattr;
            }
        }
 
 #ifdef TRACE
        /* show altered highlights after magic-cookie check */
-       if (_nc_tracing & TRACE_UPDATE) {
+       if (USE_TRACEF(TRACE_UPDATE)) {
            _tracef("After magic-cookie check...");
            _tracedump("newscr", newscr);
+           _nc_unlock_global(tracef);
        }
 #endif /* TRACE */
     }
@@ -778,7 +846,6 @@ doupdate(void)
 
     nonempty = 0;
     if (curscr->_clear || newscr->_clear) {    /* force refresh ? */
-       TR(TRACE_UPDATE, ("clearing and updating from scratch"));
        ClrUpdate();
        curscr->_clear = FALSE; /* reset flag */
        newscr->_clear = FALSE; /* reset flag */
@@ -846,13 +913,18 @@ doupdate(void)
 
   cleanup:
     /*
-     * Keep the physical screen in normal mode in case we get other
-     * processes writing to the screen.
+     * We would like to keep the physical screen in normal mode in case we get
+     * other processes writing to the screen.  This goal cannot be met for
+     * magic cookies since it interferes with attributes that may propagate
+     * past the current position.
      */
-    UpdateAttrs(normal);
+#if USE_XMC_SUPPORT
+    if (magic_cookie_glitch != 0)
+#endif
+       UpdateAttrs(normal);
 
     _nc_flush();
-    curscr->_attrs = newscr->_attrs;
+    WINDOW_ATTRS(curscr) = WINDOW_ATTRS(newscr);
 
 #if USE_TRACE_TIMES
     (void) times(&after);
@@ -881,10 +953,10 @@ doupdate(void)
 #define BCE_ATTRS (A_NORMAL|A_COLOR)
 #define BCE_BKGD(win) (((win) == curscr ? stdscr : (win))->_nc_bkgd)
 
-static inline NCURSES_CH_T
+static NCURSES_INLINE NCURSES_CH_T
 ClrBlank(WINDOW *win)
 {
-    NCURSES_CH_T blank = NewChar(BLANK_TEXT);
+    NCURSES_CH_T blank = blankchar;
     if (back_color_erase)
        AddAttr(blank, (AttrOf(BCE_BKGD(win)) & BCE_ATTRS));
     return blank;
@@ -904,7 +976,7 @@ ClrUpdate(void)
     NCURSES_CH_T blank = ClrBlank(stdscr);
     int nonempty = min(screen_lines, newscr->_maxy + 1);
 
-    TR(TRACE_UPDATE, ("ClrUpdate() called"));
+    TR(TRACE_UPDATE, (T_CALLED("ClrUpdate")));
 
     ClearScreen(blank);
 
@@ -914,6 +986,8 @@ ClrUpdate(void)
 
     for (i = 0; i < nonempty; i++)
        TransformLine(i);
+
+    TR(TRACE_UPDATE, (T_RETURN("")));
 }
 
 /*
@@ -1031,14 +1105,8 @@ ClrBottom(int total)
 
 #if USE_XMC_SUPPORT
 #if USE_WIDEC_SUPPORT
-static inline bool
-check_xmc_transition(NCURSES_CH_T * a, NCURSES_CH_T * b)
-{
-    if (((a->attr ^ b->attr) & ~(a->attr) & SP->_xmc_triggers) != 0) {
-       return TRUE;
-    }
-    return FALSE;
-}
+#define check_xmc_transition(a, b) \
+    ((((a)->attr ^ (b)->attr) & ~((a)->attr) & SP->_xmc_triggers) != 0)
 #define xmc_turn_on(a,b) check_xmc_transition(&(a), &(b))
 #else
 #define xmc_turn_on(a,b) ((((a)^(b)) & ~(a) & SP->_xmc_triggers) != 0)
@@ -1433,7 +1501,7 @@ InsStr(NCURSES_CH_T * line, int count)
     /* The order must match that of InsCharCost. */
     if (parm_ich) {
        TPUTS_TRACE("parm_ich");
-       tputs(tparm(parm_ich, count), count, _nc_outch);
+       tputs(TPARM_1(parm_ich, count), count, _nc_outch);
        while (count) {
            PutAttrChar(CHREF(*line));
            line++;
@@ -1481,12 +1549,14 @@ DelChar(int count)
 {
     int n;
 
-    TR(TRACE_UPDATE, ("DelChar(%d) called, position = (%d,%d)", count,
-                     newscr->_cury, newscr->_curx));
+    TR(TRACE_UPDATE, ("DelChar(%d) called, position = (%ld,%ld)",
+                     count,
+                     (long) newscr->_cury,
+                     (long) newscr->_curx));
 
     if (parm_dch) {
        TPUTS_TRACE("parm_dch");
-       tputs(tparm(parm_dch, count), count, _nc_outch);
+       tputs(TPARM_1(parm_dch, count), count, _nc_outch);
     } else {
        for (n = 0; n < count; n++) {
            TPUTS_TRACE("delete_character");
@@ -1547,12 +1617,12 @@ scroll_csr_forward(int n, int top, int bot, int miny, int maxy, NCURSES_CH_T bla
        GoTo(bot, 0);
        UpdateAttrs(blank);
        TPUTS_TRACE("parm_index");
-       tputs(tparm(parm_index, n, 0), n, _nc_outch);
+       tputs(TPARM_2(parm_index, n, 0), n, _nc_outch);
     } else if (parm_delete_line && bot == maxy) {
        GoTo(top, 0);
        UpdateAttrs(blank);
        TPUTS_TRACE("parm_delete_line");
-       tputs(tparm(parm_delete_line, n, 0), n, _nc_outch);
+       tputs(TPARM_2(parm_delete_line, n, 0), n, _nc_outch);
     } else if (scroll_forward && top == miny && bot == maxy) {
        GoTo(bot, 0);
        UpdateAttrs(blank);
@@ -1605,12 +1675,12 @@ scroll_csr_backward(int n, int top, int bot, int miny, int maxy,
        GoTo(top, 0);
        UpdateAttrs(blank);
        TPUTS_TRACE("parm_rindex");
-       tputs(tparm(parm_rindex, n, 0), n, _nc_outch);
+       tputs(TPARM_2(parm_rindex, n, 0), n, _nc_outch);
     } else if (parm_insert_line && bot == maxy) {
        GoTo(top, 0);
        UpdateAttrs(blank);
        TPUTS_TRACE("parm_insert_line");
-       tputs(tparm(parm_insert_line, n, 0), n, _nc_outch);
+       tputs(TPARM_2(parm_insert_line, n, 0), n, _nc_outch);
     } else if (scroll_reverse && top == miny && bot == maxy) {
        GoTo(top, 0);
        UpdateAttrs(blank);
@@ -1658,7 +1728,7 @@ scroll_idl(int n, int del, int ins, NCURSES_CH_T blank)
        putp(delete_line);
     } else if (parm_delete_line) {
        TPUTS_TRACE("parm_delete_line");
-       tputs(tparm(parm_delete_line, n, 0), n, _nc_outch);
+       tputs(TPARM_2(parm_delete_line, n, 0), n, _nc_outch);
     } else {                   /* if (delete_line) */
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("delete_line");
@@ -1673,7 +1743,7 @@ scroll_idl(int n, int del, int ins, NCURSES_CH_T blank)
        putp(insert_line);
     } else if (parm_insert_line) {
        TPUTS_TRACE("parm_insert_line");
-       tputs(tparm(parm_insert_line, n, 0), n, _nc_outch);
+       tputs(TPARM_2(parm_insert_line, n, 0), n, _nc_outch);
     } else {                   /* if (insert_line) */
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("insert_line");
@@ -1728,7 +1798,7 @@ _nc_scrolln(int n, int top, int bot, int maxy)
                putp(save_cursor);
            }
            TPUTS_TRACE("change_scroll_region");
-           putp(tparm(change_scroll_region, top, bot));
+           putp(TPARM_2(change_scroll_region, top, bot));
            if (cursor_saved) {
                TPUTS_TRACE("restore_cursor");
                putp(restore_cursor);
@@ -1739,7 +1809,7 @@ _nc_scrolln(int n, int top, int bot, int maxy)
            res = scroll_csr_forward(n, top, bot, top, bot, blank);
 
            TPUTS_TRACE("change_scroll_region");
-           putp(tparm(change_scroll_region, 0, maxy));
+           putp(TPARM_2(change_scroll_region, 0, maxy));
            SP->_cursrow = SP->_curscol = -1;
        }
 
@@ -1751,7 +1821,7 @@ _nc_scrolln(int n, int top, int bot, int maxy)
         */
        if (res != ERR
            && (non_dest_scroll_region || (memory_below && bot == maxy))) {
-           NCURSES_CH_T blank2 = NewChar(BLANK_TEXT);
+           static const NCURSES_CH_T blank2 = NewChar(BLANK_TEXT);
            if (bot == maxy && clr_eos) {
                GoTo(bot - n + 1, 0);
                ClrToEOS(blank2);
@@ -1774,7 +1844,7 @@ _nc_scrolln(int n, int top, int bot, int maxy)
                putp(save_cursor);
            }
            TPUTS_TRACE("change_scroll_region");
-           putp(tparm(change_scroll_region, top, bot));
+           putp(TPARM_2(change_scroll_region, top, bot));
            if (cursor_saved) {
                TPUTS_TRACE("restore_cursor");
                putp(restore_cursor);
@@ -1785,7 +1855,7 @@ _nc_scrolln(int n, int top, int bot, int maxy)
            res = scroll_csr_backward(-n, top, bot, top, bot, blank);
 
            TPUTS_TRACE("change_scroll_region");
-           putp(tparm(change_scroll_region, 0, maxy));
+           putp(TPARM_2(change_scroll_region, 0, maxy));
            SP->_cursrow = SP->_curscol = -1;
        }
 
@@ -1797,7 +1867,7 @@ _nc_scrolln(int n, int top, int bot, int maxy)
         */
        if (res != ERR
            && (non_dest_scroll_region || (memory_above && top == 0))) {
-           NCURSES_CH_T blank2 = NewChar(BLANK_TEXT);
+           static const NCURSES_CH_T blank2 = NewChar(BLANK_TEXT);
            for (i = 0; i < -n; i++) {
                GoTo(i + top, 0);
                ClrToEOL(blank2, FALSE);
@@ -1872,7 +1942,7 @@ _nc_screen_wrap(void)
 #if NCURSES_EXT_FUNCS
     if (SP->_coloron
        && !SP->_default_color) {
-       NCURSES_CH_T blank = NewChar(BLANK_TEXT);
+       static const NCURSES_CH_T blank = NewChar(BLANK_TEXT);
        SP->_default_color = TRUE;
        _nc_do_color(-1, 0, FALSE, _nc_outch);
        SP->_default_color = FALSE;