]> ncurses.scripts.mit.edu Git - ncurses.git/blobdiff - ncurses/tty/tty_update.c
ncurses 5.5
[ncurses.git] / ncurses / tty / tty_update.c
index c792402dcf112f14ada4ba117090932167e5272b..408ebe3be7d05c8fa60f5dd0ad37d64b03f1e86c 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1998-2001,2002 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2004,2005 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            *
 /****************************************************************************
  *  Author: Zeyd M. Ben-Halim <zmbenhal@netcom.com> 1992,1995               *
  *     and: Eric S. Raymond <esr@snark.thyrsus.com>                         *
+ *     and: Thomas E. Dickey                        1996-on                 *
  ****************************************************************************/
 
 /*-----------------------------------------------------------------
  *
  *     lib_doupdate.c
  *
- *     The routine doupdate() and its dependents.  Also _nc_outstr(),
- *     so all physical output is concentrated here (except _nc_outch()
    in lib_tputs.c).
+ *     The routine doupdate() and its dependents.
+ *     All physical output is concentrated here (except _nc_outch()
 *    in lib_tputs.c).
  *
  *-----------------------------------------------------------------*/
 
 #endif
 #endif
 
+#include <ctype.h>
 #include <term.h>
 
-MODULE_ID("$Id: tty_update.c,v 1.177 2002/09/28 20:41:55 tom Exp $")
+MODULE_ID("$Id: tty_update.c,v 1.220 2005/08/13 17:12:32 tom Exp $")
 
 /*
  * This define controls the line-breakout optimization.  Every once in a
@@ -87,6 +89,8 @@ MODULE_ID("$Id: tty_update.c,v 1.177 2002/09/28 20:41:55 tom Exp $")
 
 #define FILL_BCE() (SP->_coloron && !SP->_default_color && !back_color_erase)
 
+static const NCURSES_CH_T normal = NewChar2(BLANK_TEXT, BLANK_ATTR);
+
 /*
  * Enable checking to see if doupdate and friends are tracking the true
  * cursor position correctly.  NOTE: this is a debugging hack which will
@@ -165,61 +169,107 @@ position_check(int expected_y, int expected_x, char *legend)
 static inline void
 GoTo(int const row, int const col)
 {
-    attr_t oldattr = SP->_current_attr;
-
     TR(TRACE_MOVE, ("GoTo(%d, %d) from (%d, %d)",
                    row, col, SP->_cursrow, SP->_curscol));
 
     position_check(SP->_cursrow, SP->_curscol, "GoTo");
 
-    /*
-     * Force restore even if msgr is on when we're in an alternate
-     * character set -- these have a strong tendency to screw up the
-     * CR & LF used for local character motions!
-     */
-    if ((oldattr & A_ALTCHARSET)
-       || (oldattr && !move_standout_mode)) {
-       TR(TRACE_CHARPUT, ("turning off (%#lx) %s before move",
-                          oldattr, _traceattr(oldattr)));
-       vidattr(A_NORMAL);
-    }
-
     mvcur(SP->_cursrow, SP->_curscol, row, col);
-    SP->_cursrow = row;
-    SP->_curscol = col;
     position_check(SP->_cursrow, SP->_curscol, "GoTo2");
 }
 
 static inline void
 PutAttrChar(CARG_CH_T ch)
 {
+    int chlen = 1;
+    NCURSES_CH_T my_ch;
     PUTC_DATA;
     NCURSES_CH_T tilde;
+    NCURSES_CH_T attr = CHDEREF(ch);
+
+    TR(TRACE_CHARPUT, ("PutAttrChar(%s) at (%d, %d)",
+                      _tracech_t(ch),
+                      SP->_cursrow, SP->_curscol));
+#if USE_WIDEC_SUPPORT
+    /*
+     * If this is not a valid character, there is nothing more to do.
+     */
+    if (isWidecExt(CHDEREF(ch))) {
+       TR(TRACE_CHARPUT, ("...skip"));
+       return;
+    }
+    /*
+     * Determine the number of character cells which the 'ch' value will use
+     * on the screen.  It should be at least one.
+     */
+    if ((chlen = wcwidth(CharOf(CHDEREF(ch)))) <= 0) {
+       static NCURSES_CH_T blank = NewChar(BLANK_TEXT);
+
+       if (is8bits(CharOf(CHDEREF(ch)))
+           && (isprint(CharOf(CHDEREF(ch)))
+               || (SP->_legacy_coding && CharOf(CHDEREF(ch)) >= 160))) {
+           ;
+       } else {
+           ch = CHREF(blank);
+           TR(TRACE_CHARPUT, ("forced to blank"));
+       }
+       chlen = 1;
+    }
+#endif
 
+    if ((AttrOf(attr) & A_ALTCHARSET)
+       && SP->_acs_map != 0
+       && CharOfD(ch) < ACS_LEN) {
+       my_ch = CHDEREF(ch);    /* work around const param */
+#if USE_WIDEC_SUPPORT
+       /*
+        * This is crude & ugly, but works most of the time.  It checks if the
+        * acs_chars string specified that we have a mapping for this
+        * character, and uses the wide-character mapping when we expect the
+        * normal one to be broken (by mis-design ;-).
+        */
+       if (SP->_screen_acs_fix
+           && SP->_screen_acs_map[CharOf(my_ch)]) {
+           RemAttr(attr, A_ALTCHARSET);
+           my_ch = _nc_wacs[CharOf(my_ch)];
+       }
+#endif
+       /*
+        * If we (still) have alternate character set, it is the normal 8bit
+        * flavor.  The _screen_acs_map[] array tells if the character was
+        * really in acs_chars, needed because of the way wide/normal line
+        * drawing flavors are integrated.
+        */
+       if (AttrOf(attr) & A_ALTCHARSET) {
+           int j = CharOfD(ch);
+           chtype temp = UChar(SP->_acs_map[j]);
+
+           if (!(SP->_screen_acs_map[j]))
+               RemAttr(attr, A_ALTCHARSET);
+           if (temp != 0)
+               SetChar(my_ch, temp, AttrOf(attr));
+       }
+       ch = CHREF(my_ch);
+    }
     if (tilde_glitch && (CharOfD(ch) == L('~'))) {
-       SetChar(tilde, L('`'), AttrOfD(ch));
+       SetChar(tilde, L('`'), AttrOf(attr));
        ch = CHREF(tilde);
     }
 
-    TR(TRACE_CHARPUT, ("PutAttrChar(%s) at (%d, %d)",
-                      _tracech_t(ch),
-                      SP->_cursrow, SP->_curscol));
-    UpdateAttrs(AttrOfD(ch));
+    UpdateAttrs(attr);
 #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->_outch != 0) {
-       SP->_outch((int) ch);
+       SP->_outch(UChar(ch));
     } else
 #endif
     {
        PUTC(CHDEREF(ch), SP->_ofp);    /* macro's fastest... */
-#ifdef TRACE
-       _nc_outchars++;
-#endif /* TRACE */
+       TRACE_OUTCHARS(1);
     }
-    SP->_curscol++;
+    SP->_curscol += chlen;
     if (char_padding) {
        TPUTS_TRACE("char_padding");
        putp(char_padding);
@@ -368,13 +418,15 @@ can_clear_with(ARG_CH_T ch)
 {
     if (!back_color_erase && SP->_coloron) {
 #if NCURSES_EXT_FUNCS
+       int pair;
+
        if (!SP->_default_color)
            return FALSE;
        if (SP->_default_fg != C_MASK || SP->_default_bg != C_MASK)
            return FALSE;
-       if (AttrOfD(ch) & A_COLOR) {
+       if ((pair = GetPair(CHDEREF(ch))) != 0) {
            short fg, bg;
-           pair_content(PAIR_NUMBER(AttrOfD(ch)), &fg, &bg);
+           pair_content(pair, &fg, &bg);
            if (fg != C_MASK || bg != C_MASK)
                return FALSE;
        }
@@ -435,7 +487,7 @@ EmitRange(const NCURSES_CH_T * ntext, int num)
            if (erase_chars
                && runcount > SP->_ech_cost + SP->_cup_ch_cost
                && can_clear_with(CHREF(ntext0))) {
-               UpdateAttrs(AttrOf(ntext0));
+               UpdateAttrs(ntext0);
                putp(tparm(erase_chars, runcount));
 
                /*
@@ -455,8 +507,9 @@ EmitRange(const NCURSES_CH_T * ntext, int num)
                if (wrap_possible)
                    rep_count--;
 
-               UpdateAttrs(AttrOf(ntext0));
-               putp(tparm(repeat_char, CharOf(ntext0), rep_count));
+               UpdateAttrs(ntext0);
+               tputs(tparm(repeat_char, CharOf(ntext0), rep_count),
+                     rep_count, _nc_outch);
                SP->_curscol += rep_count;
 
                if (wrap_possible)
@@ -490,27 +543,32 @@ PutRange(const NCURSES_CH_T * otext,
         int row,
         int first, int last)
 {
-    int j, run;
+    int i, j, same;
 
     TR(TRACE_CHARPUT, ("PutRange(%p, %p, %d, %d, %d)",
                       otext, ntext, row, first, last));
 
     if (otext != ntext
        && (last - first + 1) > SP->_inline_cost) {
-       for (j = first, run = 0; j <= last; j++) {
-           if (!run && isnac(otext[j]))
+       for (j = first, same = 0; j <= last; j++) {
+           if (!same && isWidecExt(otext[j]))
                continue;
            if (CharEq(otext[j], ntext[j])) {
-               run++;
+               same++;
            } else {
-               if (run > SP->_inline_cost) {
-                   int before_run = (j - run);
-                   EmitRange(ntext + first, before_run - first);
+               if (same > SP->_inline_cost) {
+                   EmitRange(ntext + first, j - same - first);
                    GoTo(row, first = j);
                }
-               run = 0;
+               same = 0;
            }
        }
+       i = EmitRange(ntext + first, j - same - first);
+       /*
+        * Always return 1 for the next GoTo() after a PutRange() if we found
+        * identical characters at end of interval
+        */
+       return (same == 0 ? i : 1);
     }
     return EmitRange(ntext + first, last - first + 1);
 }
@@ -791,7 +849,7 @@ doupdate(void)
      * Keep the physical screen in normal mode in case we get other
      * processes writing to the screen.
      */
-    UpdateAttrs(A_NORMAL);
+    UpdateAttrs(normal);
 
     _nc_flush();
     curscr->_attrs = newscr->_attrs;
@@ -886,14 +944,14 @@ ClrToEOL(NCURSES_CH_T blank, bool needclear)
     }
 
     if (needclear) {
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("clr_eol");
-       if (SP->_el_cost > (screen_columns - SP->_curscol)) {
+       if (clr_eol && SP->_el_cost <= (screen_columns - SP->_curscol)) {
+           putp(clr_eol);
+       } else {
            int count = (screen_columns - SP->_curscol);
            while (count-- > 0)
                PutChar(CHREF(blank));
-       } else {
-           putp(clr_eol);
        }
     }
 }
@@ -912,7 +970,7 @@ ClrToEOS(NCURSES_CH_T blank)
     row = SP->_cursrow;
     col = SP->_curscol;
 
-    UpdateAttrs(AttrOf(blank));
+    UpdateAttrs(blank);
     TPUTS_TRACE("clr_eos");
     tputs(clr_eos, screen_lines - row, _nc_outch);
 
@@ -939,7 +997,7 @@ ClrBottom(int total)
     int col;
     int top = total;
     int last = min(screen_columns, newscr->_maxx + 1);
-    NCURSES_CH_T blank = ClrBlank(stdscr);
+    NCURSES_CH_T blank = newscr->_line[total - 1].text[last - 1];
     bool ok;
 
     if (clr_eos && can_clear_with(CHREF(blank))) {
@@ -959,17 +1017,16 @@ ClrBottom(int total)
        }
 
        /* don't use clr_eos for just one line if clr_eol available */
-       if (top < total - 1 || (top < total && !clr_eol && !clr_bol)) {
+       if (top < total) {
            GoTo(top, 0);
            ClrToEOS(blank);
-           total = top;
            if (SP->oldhash && SP->newhash) {
                for (row = top; row < screen_lines; row++)
                    SP->oldhash[row] = SP->newhash[row];
            }
        }
     }
-    return total;
+    return top;
 }
 
 #if USE_XMC_SUPPORT
@@ -1018,14 +1075,12 @@ TransformLine(int const lineno)
     int n;
     bool attrchanged = FALSE;
 
-    TR(TRACE_UPDATE, ("TransformLine(%d) called", lineno));
+    TR(TRACE_UPDATE, (T_CALLED("TransformLine(%d)"), lineno));
 
     /* copy new hash value to old one */
     if (SP->oldhash && SP->newhash)
        SP->oldhash[lineno] = SP->newhash[lineno];
 
-#define ColorOf(n) (AttrOf(n) & A_COLOR)
-#define unColor(n) (AttrOf(n) & ALL_BUT_COLOR)
     /*
      * If we have colors, there is the possibility of having two color pairs
      * that display as the same colors.  For instance, Lynx does this.  Check
@@ -1033,24 +1088,19 @@ TransformLine(int const lineno)
      * they are equivalent.
      */
     if (SP->_coloron) {
-       attr_t oldColor;
-       attr_t newColor;
        int oldPair;
        int newPair;
 
        for (n = 0; n < screen_columns; n++) {
            if (!CharEq(newLine[n], oldLine[n])) {
-               oldColor = ColorOf(oldLine[n]);
-               newColor = ColorOf(newLine[n]);
-               if (oldColor != newColor
+               oldPair = GetPair(oldLine[n]);
+               newPair = GetPair(newLine[n]);
+               if (oldPair != newPair
                    && unColor(oldLine[n]) == unColor(newLine[n])) {
-                   oldPair = PAIR_NUMBER(oldColor);
-                   newPair = PAIR_NUMBER(newColor);
                    if (oldPair < COLOR_PAIRS
                        && newPair < COLOR_PAIRS
                        && SP->_color_pairs[oldPair] == SP->_color_pairs[newPair]) {
-                       RemAttr(oldLine[n], A_COLOR);
-                       AddAttr(oldLine[n], ColorOf(newLine[n]));
+                       SetPair(oldLine[n], GetPair(newLine[n]));
                    }
                }
            }
@@ -1060,8 +1110,10 @@ TransformLine(int const lineno)
     if (ceol_standout_glitch && clr_eol) {
        firstChar = 0;
        while (firstChar < screen_columns) {
-           if (AttrOf(newLine[firstChar]) != AttrOf(oldLine[firstChar]))
+           if (!SameAttrOf(newLine[firstChar], oldLine[firstChar])) {
                attrchanged = TRUE;
+               break;
+           }
            firstChar++;
        }
     }
@@ -1124,17 +1176,9 @@ TransformLine(int const lineno)
     } else {
        NCURSES_CH_T blank;
 
-       /* find the first differing character */
-       while (firstChar < screen_columns &&
-              CharEq(newLine[firstChar], oldLine[firstChar]))
-           firstChar++;
-
-       /* if there wasn't one, we're done */
-       if (firstChar >= screen_columns)
-           return;
-
        /* it may be cheap to clear leading whitespace with clr_bol */
-       if (clr_bol && (blank = newLine[0], can_clear_with(CHREF(blank)))) {
+       blank = newLine[0];
+       if (clr_bol && can_clear_with(CHREF(blank))) {
            int oFirstChar, nFirstChar;
 
            for (oFirstChar = 0; oFirstChar < screen_columns; oFirstChar++)
@@ -1144,25 +1188,44 @@ TransformLine(int const lineno)
                if (!CharEq(newLine[nFirstChar], blank))
                    break;
 
-           if (nFirstChar > oFirstChar + SP->_el1_cost) {
-               if (nFirstChar >= screen_columns && SP->_el_cost <= SP->_el1_cost) {
-                   GoTo(lineno, 0);
-                   UpdateAttrs(AttrOf(blank));
-                   TPUTS_TRACE("clr_eol");
-                   putp(clr_eol);
-               } else {
-                   GoTo(lineno, nFirstChar - 1);
-                   UpdateAttrs(AttrOf(blank));
-                   TPUTS_TRACE("clr_bol");
-                   putp(clr_bol);
-               }
-
-               while (firstChar < nFirstChar)
-                   oldLine[firstChar++] = blank;
+           if (nFirstChar == oFirstChar) {
+               firstChar = nFirstChar;
+               /* find the first differing character */
+               while (firstChar < screen_columns
+                      && CharEq(newLine[firstChar], oldLine[firstChar]))
+                   firstChar++;
+           } else if (oFirstChar > nFirstChar) {
+               firstChar = nFirstChar;
+           } else {            /* oFirstChar < nFirstChar */
+               firstChar = oFirstChar;
+               if (SP->_el1_cost < nFirstChar - oFirstChar) {
+                   if (nFirstChar >= screen_columns
+                       && SP->_el_cost <= SP->_el1_cost) {
+                       GoTo(lineno, 0);
+                       UpdateAttrs(blank);
+                       TPUTS_TRACE("clr_eol");
+                       putp(clr_eol);
+                   } else {
+                       GoTo(lineno, nFirstChar - 1);
+                       UpdateAttrs(blank);
+                       TPUTS_TRACE("clr_bol");
+                       putp(clr_bol);
+                   }
 
-               if (firstChar >= screen_columns)
-                   return;
+                   while (firstChar < nFirstChar)
+                       oldLine[firstChar++] = blank;
+               }
            }
+       } else {
+           /* find the first differing character */
+           while (firstChar < screen_columns
+                  && CharEq(newLine[firstChar], oldLine[firstChar]))
+               firstChar++;
+       }
+       /* if there wasn't one, we're done */
+       if (firstChar >= screen_columns) {
+           TR(TRACE_UPDATE, (T_RETURN("")));
+           return;
        }
 
        blank = newLine[screen_columns - 1];
@@ -1182,6 +1245,7 @@ TransformLine(int const lineno)
                       newLine + firstChar,
                       (nLastChar - firstChar + 1) * sizeof(NCURSES_CH_T));
            }
+           TR(TRACE_UPDATE, (T_RETURN("")));
            return;
        }
 
@@ -1221,7 +1285,7 @@ TransformLine(int const lineno)
            /* can be -1 if no characters differ */
            while (CharEq(newLine[nLastChar], oldLine[oLastChar])) {
                /* don't split a wide char */
-               if (isnac(newLine[nLastChar]) &&
+               if (isWidecExt(newLine[nLastChar]) &&
                    !CharEq(newLine[nLastChar - 1], oldLine[oLastChar - 1]))
                    break;
                nLastChar--;
@@ -1238,9 +1302,15 @@ TransformLine(int const lineno)
 
            if (oLastChar < nLastChar) {
                int m = max(nLastNonblank, oLastNonblank);
+#if USE_WIDEC_SUPPORT
+               while (isWidecExt(newLine[n + 1]) && n) {
+                   --n;
+                   --oLastChar;
+               }
+#endif
                GoTo(lineno, n + 1);
-               if (InsCharCost(nLastChar - oLastChar)
-                   > (m - n)) {
+               if ((nLastChar < nLastNonblank)
+                   || InsCharCost(nLastChar - oLastChar) > (m - n)) {
                    PutRange(oldLine, newLine, lineno, n + 1, m);
                } else {
                    InsStr(&newLine[n + 1], nLastChar - oLastChar);
@@ -1262,7 +1332,7 @@ TransformLine(int const lineno)
                     * setting the video attributes from
                     * the last character on the row.
                     */
-                   UpdateAttrs(AttrOf(blank));
+                   UpdateAttrs(blank);
                    DelChar(oLastChar - nLastChar);
                }
            }
@@ -1274,6 +1344,8 @@ TransformLine(int const lineno)
        memcpy(oldLine + firstChar,
               newLine + firstChar,
               (screen_columns - firstChar) * sizeof(NCURSES_CH_T));
+    TR(TRACE_UPDATE, (T_RETURN("")));
+    return;
 }
 
 /*
@@ -1294,7 +1366,7 @@ ClearScreen(NCURSES_CH_T blank)
 #if NCURSES_EXT_FUNCS
     if (SP->_coloron
        && !SP->_default_color) {
-       _nc_do_color((int) COLOR_PAIR(SP->_current_attr), 0, FALSE, _nc_outch);
+       _nc_do_color(GET_SCREEN_PAIR(SP), 0, FALSE, _nc_outch);
        if (!back_color_erase) {
            fast_clear = FALSE;
        }
@@ -1303,7 +1375,7 @@ ClearScreen(NCURSES_CH_T blank)
 
     if (fast_clear) {
        if (clear_screen) {
-           UpdateAttrs(AttrOf(blank));
+           UpdateAttrs(blank);
            TPUTS_TRACE("clear_screen");
            putp(clear_screen);
            SP->_cursrow = SP->_curscol = 0;
@@ -1312,13 +1384,13 @@ ClearScreen(NCURSES_CH_T blank)
            SP->_cursrow = SP->_curscol = -1;
            GoTo(0, 0);
 
-           UpdateAttrs(AttrOf(blank));
+           UpdateAttrs(blank);
            TPUTS_TRACE("clr_eos");
-           putp(clr_eos);
+           tputs(clr_eos, screen_lines, _nc_outch);
        } else if (clr_eol) {
            SP->_cursrow = SP->_curscol = -1;
 
-           UpdateAttrs(AttrOf(blank));
+           UpdateAttrs(blank);
            for (i = 0; i < screen_lines; i++) {
                GoTo(i, 0);
                TPUTS_TRACE("clr_eol");
@@ -1327,7 +1399,7 @@ ClearScreen(NCURSES_CH_T blank)
            GoTo(0, 0);
        }
     } else {
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        for (i = 0; i < screen_lines; i++) {
            GoTo(i, 0);
            for (j = 0; j < screen_columns; j++)
@@ -1423,19 +1495,6 @@ DelChar(int count)
     }
 }
 
-/*
-**     _nc_outstr(char *str)
-**
-**     Emit a string without waiting for update.
-*/
-
-NCURSES_EXPORT(void)
-_nc_outstr(const char *str)
-{
-    (void) putp(str);
-    _nc_flush();
-}
-
 /*
  * Physical-scrolling support
  *
@@ -1476,37 +1535,37 @@ scroll_csr_forward(int n, int top, int bot, int miny, int maxy, NCURSES_CH_T bla
 
     if (n == 1 && scroll_forward && top == miny && bot == maxy) {
        GoTo(bot, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("scroll_forward");
-       tputs(scroll_forward, 0, _nc_outch);
+       putp(scroll_forward);
     } else if (n == 1 && delete_line && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("delete_line");
-       tputs(delete_line, 0, _nc_outch);
+       putp(delete_line);
     } else if (parm_index && top == miny && bot == maxy) {
        GoTo(bot, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("parm_index");
        tputs(tparm(parm_index, n, 0), n, _nc_outch);
     } else if (parm_delete_line && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("parm_delete_line");
        tputs(tparm(parm_delete_line, n, 0), n, _nc_outch);
     } else if (scroll_forward && top == miny && bot == maxy) {
        GoTo(bot, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("scroll_forward");
-           tputs(scroll_forward, 0, _nc_outch);
+           putp(scroll_forward);
        }
     } else if (delete_line && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("delete_line");
-           tputs(delete_line, 0, _nc_outch);
+           putp(delete_line);
        }
     } else
        return ERR;
@@ -1534,37 +1593,37 @@ scroll_csr_backward(int n, int top, int bot, int miny, int maxy,
 
     if (n == 1 && scroll_reverse && top == miny && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("scroll_reverse");
-       tputs(scroll_reverse, 0, _nc_outch);
+       putp(scroll_reverse);
     } else if (n == 1 && insert_line && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("insert_line");
-       tputs(insert_line, 0, _nc_outch);
+       putp(insert_line);
     } else if (parm_rindex && top == miny && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("parm_rindex");
        tputs(tparm(parm_rindex, n, 0), n, _nc_outch);
     } else if (parm_insert_line && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        TPUTS_TRACE("parm_insert_line");
        tputs(tparm(parm_insert_line, n, 0), n, _nc_outch);
     } else if (scroll_reverse && top == miny && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("scroll_reverse");
-           tputs(scroll_reverse, 0, _nc_outch);
+           putp(scroll_reverse);
        }
     } else if (insert_line && bot == maxy) {
        GoTo(top, 0);
-       UpdateAttrs(AttrOf(blank));
+       UpdateAttrs(blank);
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("insert_line");
-           tputs(insert_line, 0, _nc_outch);
+           putp(insert_line);
        }
     } else
        return ERR;
@@ -1593,38 +1652,46 @@ scroll_idl(int n, int del, int ins, NCURSES_CH_T blank)
        return ERR;
 
     GoTo(del, 0);
-    UpdateAttrs(AttrOf(blank));
+    UpdateAttrs(blank);
     if (n == 1 && delete_line) {
        TPUTS_TRACE("delete_line");
-       tputs(delete_line, 0, _nc_outch);
+       putp(delete_line);
     } else if (parm_delete_line) {
        TPUTS_TRACE("parm_delete_line");
        tputs(tparm(parm_delete_line, n, 0), n, _nc_outch);
     } else {                   /* if (delete_line) */
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("delete_line");
-           tputs(delete_line, 0, _nc_outch);
+           putp(delete_line);
        }
     }
 
     GoTo(ins, 0);
-    UpdateAttrs(AttrOf(blank));
+    UpdateAttrs(blank);
     if (n == 1 && insert_line) {
        TPUTS_TRACE("insert_line");
-       tputs(insert_line, 0, _nc_outch);
+       putp(insert_line);
     } else if (parm_insert_line) {
        TPUTS_TRACE("parm_insert_line");
        tputs(tparm(parm_insert_line, n, 0), n, _nc_outch);
     } else {                   /* if (insert_line) */
        for (i = 0; i < n; i++) {
            TPUTS_TRACE("insert_line");
-           tputs(insert_line, 0, _nc_outch);
+           putp(insert_line);
        }
     }
 
     return OK;
 }
 
+/*
+ * Note:  some terminals require the cursor to be within the scrolling margins
+ * before setting them.  Generally, the cursor must be at the appropriate end
+ * of the scrolling margins when issuing an indexing operation (it is not
+ * apparent whether it must also be at the left margin; we do this just to be
+ * safe).  To make the related cursor movement a little faster, we use the
+ * save/restore cursor capabilities if the terminal has them.
+ */
 NCURSES_EXPORT(int)
 _nc_scrolln(int n, int top, int bot, int maxy)
 /* scroll region from top to bot by n lines */
@@ -1658,13 +1725,13 @@ _nc_scrolln(int n, int top, int bot, int maxy)
                && save_cursor && restore_cursor) {
                cursor_saved = TRUE;
                TPUTS_TRACE("save_cursor");
-               tputs(save_cursor, 0, _nc_outch);
+               putp(save_cursor);
            }
            TPUTS_TRACE("change_scroll_region");
-           tputs(tparm(change_scroll_region, top, bot), 0, _nc_outch);
+           putp(tparm(change_scroll_region, top, bot));
            if (cursor_saved) {
                TPUTS_TRACE("restore_cursor");
-               tputs(restore_cursor, 0, _nc_outch);
+               putp(restore_cursor);
            } else {
                SP->_cursrow = SP->_curscol = -1;
            }
@@ -1672,7 +1739,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");
-           tputs(tparm(change_scroll_region, 0, maxy), 0, _nc_outch);
+           putp(tparm(change_scroll_region, 0, maxy));
            SP->_cursrow = SP->_curscol = -1;
        }
 
@@ -1686,7 +1753,7 @@ _nc_scrolln(int n, int top, int bot, int maxy)
            && (non_dest_scroll_region || (memory_below && bot == maxy))) {
            NCURSES_CH_T blank2 = NewChar(BLANK_TEXT);
            if (bot == maxy && clr_eos) {
-               GoTo(bot - n, 0);
+               GoTo(bot - n + 1, 0);
                ClrToEOS(blank2);
            } else {
                for (i = 0; i < n; i++) {
@@ -1704,13 +1771,13 @@ _nc_scrolln(int n, int top, int bot, int maxy)
                && save_cursor && restore_cursor) {
                cursor_saved = TRUE;
                TPUTS_TRACE("save_cursor");
-               tputs(save_cursor, 0, _nc_outch);
+               putp(save_cursor);
            }
            TPUTS_TRACE("change_scroll_region");
-           tputs(tparm(change_scroll_region, top, bot), 0, _nc_outch);
+           putp(tparm(change_scroll_region, top, bot));
            if (cursor_saved) {
                TPUTS_TRACE("restore_cursor");
-               tputs(restore_cursor, 0, _nc_outch);
+               putp(restore_cursor);
            } else {
                SP->_cursrow = SP->_curscol = -1;
            }
@@ -1718,7 +1785,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");
-           tputs(tparm(change_scroll_region, 0, maxy), 0, _nc_outch);
+           putp(tparm(change_scroll_region, 0, maxy));
            SP->_cursrow = SP->_curscol = -1;
        }
 
@@ -1753,7 +1820,7 @@ NCURSES_EXPORT(void)
 _nc_screen_resume(void)
 {
     /* make sure terminal is in a sane known state */
-    SP->_current_attr = A_NORMAL;
+    SetAttr(SCREEN_ATTRS(SP), A_NORMAL);
     newscr->_clear = TRUE;
 
     /* reset color pairs and definitions */
@@ -1801,7 +1868,7 @@ _nc_screen_init(void)
 NCURSES_EXPORT(void)
 _nc_screen_wrap(void)
 {
-    UpdateAttrs(A_NORMAL);
+    UpdateAttrs(normal);
 #if NCURSES_EXT_FUNCS
     if (SP->_coloron
        && !SP->_default_color) {
@@ -1811,8 +1878,6 @@ _nc_screen_wrap(void)
        SP->_default_color = FALSE;
 
        mvcur(SP->_cursrow, SP->_curscol, screen_lines - 1, 0);
-       SP->_cursrow = screen_lines - 1;
-       SP->_curscol = 0;
 
        ClrToEOL(blank, TRUE);
     }
@@ -1826,7 +1891,7 @@ _nc_screen_wrap(void)
 NCURSES_EXPORT(void)
 _nc_do_xmc_glitch(attr_t previous)
 {
-    attr_t chg = XMC_CHANGES(previous ^ SP->_current_attr);
+    attr_t chg = XMC_CHANGES(previous ^ AttrOf(SCREEN_ATTRS(SP)));
 
     while (chg != 0) {
        if (chg & 1) {