ncurses 6.2 - patch 20200531

[ncurses.git] / ncurses / tinfo / comp_expand.c

diff --git a/ncurses/tinfo/comp_expand.c b/ncurses/tinfo/comp_expand.c
index 840240088949acb7d9a46ef2c257ba41ea720716..02e38e6e3c7a440a3715b8d0010f0209dd852659 100644 (file)
--- a/ncurses/tinfo/comp_expand.c
+++ b/ncurses/tinfo/comp_expand.c
@@ -1,5 +1,6 @@
 /****************************************************************************
- * Copyright (c) 1998-2010,2011 Free Software Foundation, Inc.              *
+ * Copyright 2020 Thomas E. Dickey                                          *
+ * Copyright 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            *
@@ -27,7 +28,7 @@
  ****************************************************************************/
 
 /****************************************************************************
- *  Author: Thomas E. Dickey <dickey@clark.net> 1998                        *
+ *  Author: Thomas E. Dickey                    1998                        *
  ****************************************************************************/
 
 #include <curses.priv.h>
@@ -35,7 +36,13 @@
 #include <ctype.h>
 #include <tic.h>
 
-MODULE_ID("$Id: comp_expand.c,v 1.22 2011/05/21 18:55:07 tom Exp $")
+MODULE_ID("$Id: comp_expand.c,v 1.32 2020/02/02 23:34:34 tom Exp $")
+
+#if 0
+#define DEBUG_THIS(p) DEBUG(9, p)
+#else
+#define DEBUG_THIS(p)          /* nothing */
+#endif
 
 static int
 trailing_spaces(const char *src)
@@ -46,9 +53,10 @@ trailing_spaces(const char *src)
 }
 
 /* this deals with differences over whether 0x7f and 0x80..0x9f are controls */
-#define REALCTL(s) (UChar(*(s)) < 127 && iscntrl(UChar(*(s))))
 #define REALPRINT(s) (UChar(*(s)) < 127 && isprint(UChar(*(s))))
 
+#define P_LIMIT(p)   (length - (size_t)(p))
+
 NCURSES_EXPORT(char *)
 _nc_tic_expand(const char *srcp, bool tic_format, int numbers)
 {
@@ -57,9 +65,13 @@ _nc_tic_expand(const char *srcp, bool tic_format, int numbers)
 
     int bufp;
     const char *str = VALID_STRING(srcp) ? srcp : "\0\0";
-    bool islong = (strlen(str) > 3);
     size_t need = (2 + strlen(str)) * 4;
     int ch;
+    int octals = 0;
+    struct {
+       int ch;
+       int offset;
+    } fixups[MAX_TC_FIXUPS];
 
     if (srcp == 0) {
 #if NO_LEAKS
@@ -75,6 +87,7 @@ _nc_tic_expand(const char *srcp, bool tic_format, int numbers)
              return 0;
     }
 
+    DEBUG_THIS(("_nc_tic_expand %s", _nc_visbuf(srcp)));
     bufp = 0;
     while ((ch = UChar(*str)) != 0) {
        if (ch == '%' && REALPRINT(str + 1)) {
@@ -90,7 +103,8 @@ _nc_tic_expand(const char *srcp, bool tic_format, int numbers)
                    && str[1] != '\\'
                    && REALPRINT(str + 1)
                    && str[2] == S_QUOTE) {
-                   sprintf(buffer + bufp, "{%d}", str[1]);
+                   _nc_SPRINTF(buffer + bufp, _nc_SLIMIT(P_LIMIT(bufp))
+                               "{%d}", str[1]);
                    bufp += (int) strlen(buffer + bufp);
                    str += 2;
                } else {
@@ -130,6 +144,8 @@ _nc_tic_expand(const char *srcp, bool tic_format, int numbers)
                }
                break;
            default:
+               if (*str == ',')        /* minitel1 uses this */
+                   buffer[bufp++] = '\\';
                buffer[bufp++] = *str;
                break;
            }
@@ -155,32 +171,28 @@ _nc_tic_expand(const char *srcp, bool tic_format, int numbers)
                       && !(ch == '!' && !tic_format)
                       && ch != '^'))
            buffer[bufp++] = (char) ch;
-#if 0                          /* FIXME: this would be more readable (in fact the whole 'islong' logic should be removed) */
-       else if (ch == '\b') {
-           buffer[bufp++] = '\\';
-           buffer[bufp++] = 'b';
-       } else if (ch == '\f') {
-           buffer[bufp++] = '\\';
-           buffer[bufp++] = 'f';
-       } else if (ch == '\t' && islong) {
-           buffer[bufp++] = '\\';
-           buffer[bufp++] = 't';
-       }
-#endif
-       else if (ch == '\r' && (islong || (strlen(srcp) > 2 && str[1] == '\0'))) {
+       else if (ch == '\r') {
            buffer[bufp++] = '\\';
            buffer[bufp++] = 'r';
-       } else if (ch == '\n' && islong) {
+       } else if (ch == '\n') {
            buffer[bufp++] = '\\';
            buffer[bufp++] = 'n';
        }
 #define UnCtl(c) ((c) + '@')
-       else if (REALCTL(str) && ch != '\\'
-                && (!islong || isdigit(UChar(str[1])))) {
-           (void) sprintf(&buffer[bufp], "^%c", UnCtl(ch));
+       else if (UChar(ch) < 32
+                && isdigit(UChar(str[1]))) {
+           _nc_SPRINTF(&buffer[bufp], _nc_SLIMIT(P_LIMIT(bufp))
+                       "^%c", UnCtl(ch));
            bufp += 2;
        } else {
-           (void) sprintf(&buffer[bufp], "\\%03o", ch);
+           _nc_SPRINTF(&buffer[bufp], _nc_SLIMIT(P_LIMIT(bufp))
+                       "\\%03o", ch);
+           if ((octals < MAX_TC_FIXUPS) &&
+               ((tic_format && (ch == 127)) || ch < 32)) {
+               fixups[octals].ch = UChar(ch);
+               fixups[octals].offset = bufp;
+               ++octals;
+           }
            bufp += 4;
        }
 
@@ -188,5 +200,26 @@ _nc_tic_expand(const char *srcp, bool tic_format, int numbers)
     }
 
     buffer[bufp] = '\0';
+
+    /*
+     * If most of a short string is ASCII control characters, reformat the
+     * string to show those in up-arrow format.  For longer strings, it's
+     * more likely that the characters are just binary coding.
+     *
+     * If we're formatting termcap, just use the shorter format (up-arrows).
+     */
+    if (octals != 0 && (!tic_format || (bufp - (4 * octals)) < MIN_TC_FIXUPS)) {
+       while (--octals >= 0) {
+           char *p = buffer + fixups[octals].offset;
+           *p++ = '^';
+           *p++ = (char) ((fixups[octals].ch == 127)
+                          ? '?'
+                          : (fixups[octals].ch + (int) '@'));
+           while ((p[0] = p[2]) != 0) {
+               ++p;
+           }
+       }
+    }
+    DEBUG_THIS(("... %s", _nc_visbuf(buffer)));
     return (buffer);
 }