]> ncurses.scripts.mit.edu Git - ncurses.git/blobdiff - ncurses/tinfo/alloc_ttype.c
ncurses 6.0 - patch 20160924
[ncurses.git] / ncurses / tinfo / alloc_ttype.c
index a1bf9b0733f022b835437119475e4304fcd20c55..d0821a3050d0e9d951cb02a1baaa2fc3f5388777 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1999 Free Software Foundation, Inc.                        *
+ * Copyright (c) 1999-2013,2016 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: Thomas E. Dickey <dickey@clark.net> 1999                        *
+ *  Author: Thomas E. Dickey <dickey@clark.net> 1999-on                     *
  ****************************************************************************/
 
-
 /*
  * align_ttype.c --  functions for TERMTYPE
  *
 #include <curses.priv.h>
 
 #include <tic.h>
-#include <term_entry.h>
 
-MODULE_ID("$Id: alloc_ttype.c,v 1.6 1999/03/01 22:10:44 tom Exp $")
+MODULE_ID("$Id: alloc_ttype.c,v 1.28 2016/05/28 23:22:52 tom Exp $")
 
 #if NCURSES_XNAMES
 /*
  * Merge the a/b lists into dst.  Both a/b are sorted (see _nc_extend_names()),
  * so we do not have to worry about order dependencies.
  */
-static int merge_names(char **dst, char **a, int na, char **b, int nb)
+static int
+merge_names(char **dst, char **a, int na, char **b, int nb)
 {
     int n = 0;
-    while (na && nb) {
+    while (na > 0 && nb > 0) {
        int cmp = strcmp(*a, *b);
        if (cmp < 0) {
            dst[n++] = *a++;
@@ -78,7 +77,8 @@ static int merge_names(char **dst, char **a, int na, char **b, int nb)
     return n;
 }
 
-static bool find_name(char **table, int length, char *name)
+static bool
+find_name(char **table, int length, char *name)
 {
     while (length-- > 0) {
        if (!strcmp(*table++, name)) {
@@ -90,61 +90,70 @@ static bool find_name(char **table, int length, char *name)
     return FALSE;
 }
 
-static void realign_data(TERMTYPE *to, char **ext_Names, int ext_Booleans, int ext_Numbers, int ext_Strings)
+#define EXTEND_NUM(num, ext) \
+       to->num = (unsigned short) (to->num + (ext - to->ext))
+
+static void
+realign_data(TERMTYPE *to, char **ext_Names,
+            int ext_Booleans,
+            int ext_Numbers,
+            int ext_Strings)
 {
     int n, m, base;
     int limit = (to->ext_Booleans + to->ext_Numbers + to->ext_Strings);
 
     if (to->ext_Booleans != ext_Booleans) {
-       to->num_Booleans += (ext_Booleans - to->ext_Booleans);
-       to->Booleans = typeRealloc(char, to->num_Booleans, to->Booleans);
-       for (n = to->ext_Booleans-1,
-            m = ext_Booleans-1,
-            base = to->num_Booleans - (m+1); m >= 0; m--) {
+       EXTEND_NUM(num_Booleans, ext_Booleans);
+       TYPE_REALLOC(NCURSES_SBOOL, to->num_Booleans, to->Booleans);
+       for (n = to->ext_Booleans - 1,
+            m = ext_Booleans - 1,
+            base = to->num_Booleans - (m + 1); m >= 0; m--) {
            if (find_name(to->ext_Names, limit, ext_Names[m])) {
                to->Booleans[base + m] = to->Booleans[base + n--];
            } else {
                to->Booleans[base + m] = FALSE;
            }
        }
-       to->ext_Booleans = ext_Booleans;
+       to->ext_Booleans = UShort(ext_Booleans);
     }
+
     if (to->ext_Numbers != ext_Numbers) {
-       to->num_Numbers += (ext_Numbers - to->ext_Numbers);
-       to->Numbers = typeRealloc(short, to->num_Numbers, to->Numbers);
-       for (n = to->ext_Numbers-1,
-            m = ext_Numbers-1,
-            base = to->num_Numbers - (m+1); m >= 0; m--) {
-           if (find_name(to->ext_Names, limit, ext_Names[m+ext_Booleans])) {
+       EXTEND_NUM(num_Numbers, ext_Numbers);
+       TYPE_REALLOC(short, to->num_Numbers, to->Numbers);
+       for (n = to->ext_Numbers - 1,
+            m = ext_Numbers - 1,
+            base = to->num_Numbers - (m + 1); m >= 0; m--) {
+           if (find_name(to->ext_Names, limit, ext_Names[m + ext_Booleans])) {
                to->Numbers[base + m] = to->Numbers[base + n--];
            } else {
                to->Numbers[base + m] = ABSENT_NUMERIC;
            }
        }
-       to->ext_Numbers  = ext_Numbers;
+       to->ext_Numbers = UShort(ext_Numbers);
     }
     if (to->ext_Strings != ext_Strings) {
-       to->num_Strings += (ext_Strings - to->ext_Strings);
-       to->Strings = typeRealloc(char*, to->num_Strings, to->Strings);
-       for (n = to->ext_Strings-1,
-            m = ext_Strings-1,
-            base = to->num_Strings - (m+1); m >= 0; m--) {
-           if (find_name(to->ext_Names, limit, ext_Names[m+ext_Booleans+ext_Numbers])) {
+       EXTEND_NUM(num_Strings, ext_Strings);
+       TYPE_REALLOC(char *, to->num_Strings, to->Strings);
+       for (n = to->ext_Strings - 1,
+            m = ext_Strings - 1,
+            base = to->num_Strings - (m + 1); m >= 0; m--) {
+           if (find_name(to->ext_Names, limit, ext_Names[m + ext_Booleans + ext_Numbers])) {
                to->Strings[base + m] = to->Strings[base + n--];
            } else {
                to->Strings[base + m] = ABSENT_STRING;
            }
        }
-       to->ext_Strings = ext_Strings;
+       to->ext_Strings = UShort(ext_Strings);
     }
 }
 
 /*
  * Returns the first index in ext_Names[] for the given token-type
  */
-static int _nc_first_ext_name(TERMTYPE *tp, int token_type)
+static unsigned
+_nc_first_ext_name(TERMTYPE *tp, int token_type)
 {
-    int first;
+    unsigned first;
 
     switch (token_type) {
     case BOOLEAN:
@@ -154,7 +163,7 @@ static int _nc_first_ext_name(TERMTYPE *tp, int token_type)
        first = tp->ext_Booleans;
        break;
     case STRING:
-       first = tp->ext_Booleans + tp->ext_Numbers;
+       first = (unsigned) (tp->ext_Booleans + tp->ext_Numbers);
        break;
     default:
        first = 0;
@@ -166,20 +175,21 @@ static int _nc_first_ext_name(TERMTYPE *tp, int token_type)
 /*
  * Returns the last index in ext_Names[] for the given token-type
  */
-static int _nc_last_ext_name(TERMTYPE *tp, int token_type)
+static unsigned
+_nc_last_ext_name(TERMTYPE *tp, int token_type)
 {
-    int last;
+    unsigned last;
 
     switch (token_type) {
     case BOOLEAN:
-       last  = tp->ext_Booleans;
+       last = tp->ext_Booleans;
        break;
     case NUMBER:
-       last  = tp->ext_Booleans + tp->ext_Numbers;
+       last = (unsigned) (tp->ext_Booleans + tp->ext_Numbers);
        break;
     default:
     case STRING:
-       last  = NUM_EXT_NAMES(tp);
+       last = NUM_EXT_NAMES(tp);
        break;
     }
     return last;
@@ -188,15 +198,16 @@ static int _nc_last_ext_name(TERMTYPE *tp, int token_type)
 /*
  * Lookup an entry from extended-names, returning -1 if not found
  */
-static int _nc_find_ext_name(TERMTYPE *tp, char *name, int token_type)
+static int
+_nc_find_ext_name(TERMTYPE *tp, char *name, int token_type)
 {
     unsigned j;
     unsigned first = _nc_first_ext_name(tp, token_type);
-    unsigned last  = _nc_last_ext_name(tp, token_type);
+    unsigned last = _nc_last_ext_name(tp, token_type);
 
     for (j = first; j < last; j++) {
        if (!strcmp(name, tp->ext_Names[j])) {
-           return j;
+           return (int) j;
        }
     }
     return -1;
@@ -206,7 +217,8 @@ static int _nc_find_ext_name(TERMTYPE *tp, char *name, int token_type)
  * Translate an index into ext_Names[] into the corresponding index into data
  * (e.g., Booleans[]).
  */
-static int _nc_ext_data_index(TERMTYPE *tp, int n, int token_type)
+static int
+_nc_ext_data_index(TERMTYPE *tp, int n, int token_type)
 {
     switch (token_type) {
     case BOOLEAN:
@@ -214,12 +226,12 @@ static int _nc_ext_data_index(TERMTYPE *tp, int n, int token_type)
        break;
     case NUMBER:
        n += (tp->num_Numbers - tp->ext_Numbers)
-          - (tp->ext_Booleans);
+           - (tp->ext_Booleans);
        break;
     default:
     case STRING:
        n += (tp->num_Strings - tp->ext_Strings)
-          - (tp->ext_Booleans + tp->ext_Numbers);
+           - (tp->ext_Booleans + tp->ext_Numbers);
     }
     return n;
 }
@@ -228,51 +240,56 @@ static int _nc_ext_data_index(TERMTYPE *tp, int n, int token_type)
  * Adjust tables to remove (not free) an extended name and its corresponding
  * data.
  */
-static void _nc_del_ext_name(TERMTYPE *tp, char *name, int token_type)
+static bool
+_nc_del_ext_name(TERMTYPE *tp, char *name, int token_type)
 {
-    int j;
-    int first, last;
+    int first;
 
     if ((first = _nc_find_ext_name(tp, name, token_type)) >= 0) {
-       last = NUM_EXT_NAMES(tp) - 1;
+       int j;
+       int last = (int) NUM_EXT_NAMES(tp) - 1;
+
        for (j = first; j < last; j++) {
-           tp->ext_Names[j] = tp->ext_Names[j+1];
+           tp->ext_Names[j] = tp->ext_Names[j + 1];
        }
        first = _nc_ext_data_index(tp, first, token_type);
        switch (token_type) {
        case BOOLEAN:
            last = tp->num_Booleans - 1;
            for (j = first; j < last; j++)
-               tp->Booleans[j] = tp->Booleans[j+1];
-           tp->ext_Booleans -= 1;
-           tp->num_Booleans -= 1;
+               tp->Booleans[j] = tp->Booleans[j + 1];
+           tp->ext_Booleans--;
+           tp->num_Booleans--;
            break;
        case NUMBER:
            last = tp->num_Numbers - 1;
            for (j = first; j < last; j++)
-               tp->Numbers[j] = tp->Numbers[j+1];
-           tp->ext_Numbers -= 1;
-           tp->num_Numbers -= 1;
+               tp->Numbers[j] = tp->Numbers[j + 1];
+           tp->ext_Numbers--;
+           tp->num_Numbers--;
            break;
        case STRING:
            last = tp->num_Strings - 1;
            for (j = first; j < last; j++)
-               tp->Strings[j] = tp->Strings[j+1];
-           tp->ext_Strings -= 1;
-           tp->num_Strings -= 1;
+               tp->Strings[j] = tp->Strings[j + 1];
+           tp->ext_Strings--;
+           tp->num_Strings--;
            break;
        }
+       return TRUE;
     }
+    return FALSE;
 }
 
 /*
  * Adjust tables to insert an extended name, making room for new data.  The
  * index into the corresponding data array is returned.
  */
-static int _nc_ins_ext_name(TERMTYPE *tp, char *name, int token_type)
+static int
+_nc_ins_ext_name(TERMTYPE *tp, char *name, int token_type)
 {
     unsigned first = _nc_first_ext_name(tp, token_type);
-    unsigned last  = _nc_last_ext_name(tp, token_type);
+    unsigned last = _nc_last_ext_name(tp, token_type);
     unsigned total = NUM_EXT_NAMES(tp) + 1;
     unsigned j, k;
 
@@ -280,42 +297,42 @@ static int _nc_ins_ext_name(TERMTYPE *tp, char *name, int token_type)
        int cmp = strcmp(name, tp->ext_Names[j]);
        if (cmp == 0)
            /* already present */
-           return _nc_ext_data_index(tp, j, token_type);
+           return _nc_ext_data_index(tp, (int) j, token_type);
        if (cmp < 0) {
            break;
        }
     }
 
-    tp->ext_Names = typeRealloc(char *, total, tp->ext_Names);
-    for (k = total-1; k > j; k--)
-       tp->ext_Names[k] = tp->ext_Names[k-1];
+    TYPE_REALLOC(char *, total, tp->ext_Names);
+    for (k = total - 1; k > j; k--)
+       tp->ext_Names[k] = tp->ext_Names[k - 1];
     tp->ext_Names[j] = name;
-    j = _nc_ext_data_index(tp, j, token_type);
+    j = (unsigned) _nc_ext_data_index(tp, (int) j, token_type);
 
     switch (token_type) {
     case BOOLEAN:
-       tp->ext_Booleans += 1;
-       tp->num_Booleans += 1;
-       tp->Booleans = typeRealloc(char, tp->num_Booleans, tp->Booleans);
-       for (k = tp->num_Booleans-1; k > j; k--)
-           tp->Booleans[k] = tp->Booleans[k-1];
+       tp->ext_Booleans++;
+       tp->num_Booleans++;
+       TYPE_REALLOC(NCURSES_SBOOL, tp->num_Booleans, tp->Booleans);
+       for (k = (unsigned) (tp->num_Booleans - 1); k > j; k--)
+           tp->Booleans[k] = tp->Booleans[k - 1];
        break;
     case NUMBER:
-       tp->ext_Numbers += 1;
-       tp->num_Numbers += 1;
-       tp->Numbers = typeRealloc(short, tp->num_Numbers, tp->Numbers);
-       for (k = tp->num_Numbers-1; k > j; k--)
-           tp->Numbers[k] = tp->Numbers[k-1];
+       tp->ext_Numbers++;
+       tp->num_Numbers++;
+       TYPE_REALLOC(short, tp->num_Numbers, tp->Numbers);
+       for (k = (unsigned) (tp->num_Numbers - 1); k > j; k--)
+           tp->Numbers[k] = tp->Numbers[k - 1];
        break;
     case STRING:
-       tp->ext_Strings += 1;
-       tp->num_Strings += 1;
-       tp->Strings = typeRealloc(char *, tp->num_Strings, tp->Strings);
-       for (k = tp->num_Strings-1; k > j; k--)
-           tp->Strings[k] = tp->Strings[k-1];
+       tp->ext_Strings++;
+       tp->num_Strings++;
+       TYPE_REALLOC(char *, tp->num_Strings, tp->Strings);
+       for (k = (unsigned) (tp->num_Strings - 1); k > j; k--)
+           tp->Strings[k] = tp->Strings[k - 1];
        break;
     }
-    return j;
+    return (int) j;
 }
 
 /*
@@ -323,25 +340,44 @@ static int _nc_ins_ext_name(TERMTYPE *tp, char *name, int token_type)
  * as a boolean or number.  We'll get this as a special case when we get a
  * cancellation of a name that is inherited from another entry.
  */
-static void adjust_cancels(TERMTYPE *to, TERMTYPE *from)
+static void
+adjust_cancels(TERMTYPE *to, TERMTYPE *from)
 {
     int first = to->ext_Booleans + to->ext_Numbers;
-    int last  = first + to->ext_Strings;
+    int last = first + to->ext_Strings;
     int j, k;
 
-    for (j = first; j < last; ) {
+    for (j = first; j < last;) {
        char *name = to->ext_Names[j];
-       unsigned j_str = to->num_Strings - first - to->ext_Strings;
+       int j_str = to->num_Strings - first - to->ext_Strings;
 
        if (to->Strings[j + j_str] == CANCELLED_STRING) {
-           if ((k = _nc_find_ext_name(from, to->ext_Names[j], BOOLEAN)) >= 0) {
-               _nc_del_ext_name(to, name, STRING);
-               k = _nc_ins_ext_name(to, name, BOOLEAN);
-               to->Booleans[k] = FALSE;
-           } else if ((k = _nc_find_ext_name(from, to->ext_Names[j], NUMBER)) >= 0) {
-               _nc_del_ext_name(to, name, STRING);
-               k = _nc_ins_ext_name(to, name, NUMBER);
-               to->Numbers[k] = CANCELLED_NUMERIC;
+           if (_nc_find_ext_name(from, to->ext_Names[j], BOOLEAN) >= 0) {
+               if (_nc_del_ext_name(to, name, STRING)
+                   || _nc_del_ext_name(to, name, NUMBER)) {
+                   k = _nc_ins_ext_name(to, name, BOOLEAN);
+                   to->Booleans[k] = FALSE;
+               } else {
+                   j++;
+               }
+           } else if (_nc_find_ext_name(from, to->ext_Names[j], NUMBER) >= 0) {
+               if (_nc_del_ext_name(to, name, STRING)
+                   || _nc_del_ext_name(to, name, BOOLEAN)) {
+                   k = _nc_ins_ext_name(to, name, NUMBER);
+                   to->Numbers[k] = CANCELLED_NUMERIC;
+               } else {
+                   j++;
+               }
+           } else if (_nc_find_ext_name(from, to->ext_Names[j], STRING) >= 0) {
+               if (_nc_del_ext_name(to, name, NUMBER)
+                   || _nc_del_ext_name(to, name, BOOLEAN)) {
+                   k = _nc_ins_ext_name(to, name, STRING);
+                   to->Strings[k] = CANCELLED_STRING;
+               } else {
+                   j++;
+               }
+           } else {
+               j++;
            }
        } else {
            j++;
@@ -349,22 +385,27 @@ static void adjust_cancels(TERMTYPE *to, TERMTYPE *from)
     }
 }
 
-void _nc_align_termtype(TERMTYPE *to, TERMTYPE *from)
+NCURSES_EXPORT(void)
+_nc_align_termtype(TERMTYPE *to, TERMTYPE *from)
 {
-    int na = NUM_EXT_NAMES(to);
-    int nb = NUM_EXT_NAMES(from);
-    int n;
-    bool same;
+    int na = (int) NUM_EXT_NAMES(to);
+    int nb = (int) NUM_EXT_NAMES(from);
     char **ext_Names;
-    int ext_Booleans, ext_Numbers, ext_Strings;
 
-    DEBUG(2, ("align_termtype to(%d:%s), from(%d:%s)", na, to->term_names, nb, from->term_names));
+    DEBUG(2, ("align_termtype to(%d:%s), from(%d:%s)", na, to->term_names,
+             nb, from->term_names));
 
     if (na != 0 || nb != 0) {
+       int ext_Booleans, ext_Numbers, ext_Strings;
+       bool used_ext_Names = FALSE;
+
        if ((na == nb)          /* check if the arrays are equivalent */
-        && (to->ext_Booleans == from->ext_Booleans)
-        && (to->ext_Numbers  == from->ext_Numbers)
-        && (to->ext_Strings  == from->ext_Strings)) {
+           &&(to->ext_Booleans == from->ext_Booleans)
+           && (to->ext_Numbers == from->ext_Numbers)
+           && (to->ext_Strings == from->ext_Strings)) {
+           int n;
+           bool same;
+
            for (n = 0, same = TRUE; n < na; n++) {
                if (strcmp(to->ext_Names[n], from->ext_Names[n])) {
                    same = FALSE;
@@ -380,7 +421,7 @@ void _nc_align_termtype(TERMTYPE *to, TERMTYPE *from)
         * into it, updating to's counts for booleans, etc.  Fortunately we do
         * this only for the terminfo compiler (tic) and comparer (infocmp).
         */
-       ext_Names = typeMalloc(char *, na+nb);
+       TYPE_MALLOC(char *, (size_t)(na + nb), ext_Names);
 
        if (to->ext_Strings && (from->ext_Booleans + from->ext_Numbers))
            adjust_cancels(to, from);
@@ -389,63 +430,74 @@ void _nc_align_termtype(TERMTYPE *to, TERMTYPE *from)
            adjust_cancels(from, to);
 
        ext_Booleans = merge_names(ext_Names,
-                                   to->ext_Names,
-                                   to->ext_Booleans,
-                                   from->ext_Names,
-                                   from->ext_Booleans);
-       ext_Numbers  = merge_names(ext_Names + ext_Booleans,
-                                   to->ext_Names
-                                   + to->ext_Booleans,
-                                   to->ext_Numbers,
-                                   from->ext_Names
-                                   + from->ext_Booleans,
-                                   from->ext_Numbers);
-       ext_Strings  = merge_names(ext_Names + ext_Numbers + ext_Booleans,
-                                   to->ext_Names
-                                   + to->ext_Booleans
-                                   + to->ext_Numbers,
-                                   to->ext_Strings,
-                                   from->ext_Names
-                                   + from->ext_Booleans
-                                   + from->ext_Numbers,
-                                   from->ext_Strings);
+                                  to->ext_Names,
+                                  to->ext_Booleans,
+                                  from->ext_Names,
+                                  from->ext_Booleans);
+       ext_Numbers = merge_names(ext_Names + ext_Booleans,
+                                 to->ext_Names
+                                 + to->ext_Booleans,
+                                 to->ext_Numbers,
+                                 from->ext_Names
+                                 + from->ext_Booleans,
+                                 from->ext_Numbers);
+       ext_Strings = merge_names(ext_Names + ext_Numbers + ext_Booleans,
+                                 to->ext_Names
+                                 + to->ext_Booleans
+                                 + to->ext_Numbers,
+                                 to->ext_Strings,
+                                 from->ext_Names
+                                 + from->ext_Booleans
+                                 + from->ext_Numbers,
+                                 from->ext_Strings);
        /*
         * Now we must reallocate the Booleans, etc., to allow the data to be
         * overlaid.
         */
        if (na != (ext_Booleans + ext_Numbers + ext_Strings)) {
            realign_data(to, ext_Names, ext_Booleans, ext_Numbers, ext_Strings);
-           free(to->ext_Names);
+           FreeIfNeeded(to->ext_Names);
            to->ext_Names = ext_Names;
-           DEBUG(2, ("realigned %d extended names for '%s' (to)", NUM_EXT_NAMES(to), to->term_names));
+           DEBUG(2, ("realigned %d extended names for '%s' (to)",
+                     NUM_EXT_NAMES(to), to->term_names));
+           used_ext_Names = TRUE;
        }
        if (nb != (ext_Booleans + ext_Numbers + ext_Strings)) {
            nb = (ext_Booleans + ext_Numbers + ext_Strings);
            realign_data(from, ext_Names, ext_Booleans, ext_Numbers, ext_Strings);
-           from->ext_Names = typeRealloc(char *, nb, from->ext_Names);
-           memcpy(from->ext_Names, ext_Names, sizeof(char *) * nb);
-           DEBUG(2, ("realigned %d extended names for '%s' (from)", NUM_EXT_NAMES(from), from->term_names));
+           TYPE_REALLOC(char *, (size_t) nb, from->ext_Names);
+           memcpy(from->ext_Names, ext_Names, sizeof(char *) * (size_t) nb);
+           DEBUG(2, ("realigned %d extended names for '%s' (from)",
+                     NUM_EXT_NAMES(from), from->term_names));
        }
+       if (!used_ext_Names)
+           free(ext_Names);
     }
 }
 #endif
 
-void _nc_copy_termtype(TERMTYPE *dst, TERMTYPE *src)
+NCURSES_EXPORT(void)
+_nc_copy_termtype(TERMTYPE *dst, const TERMTYPE *src)
 {
-    int i;
+#if NCURSES_XNAMES
+    unsigned i;
+#endif
 
-    *dst = *src;       /* ...to copy the sizes and string-tables */
-    dst->Booleans = typeMalloc(char,   NUM_BOOLEANS(dst));
-    dst->Numbers  = typeMalloc(short,  NUM_NUMBERS(dst));
-    dst->Strings  = typeMalloc(char *, NUM_STRINGS(dst));
+    *dst = *src;               /* ...to copy the sizes and string-tables */
 
-    /* FIXME: use memcpy for these and similar loops */
-    for_each_boolean(i,dst)
-       dst->Booleans[i] = src->Booleans[i];
-    for_each_number(i,dst)
-       dst->Numbers[i] = src->Numbers[i];
-    for_each_string(i,dst)
-       dst->Strings[i] = src->Strings[i];
+    TYPE_MALLOC(NCURSES_SBOOL, NUM_BOOLEANS(dst), dst->Booleans);
+    TYPE_MALLOC(short, NUM_NUMBERS(dst), dst->Numbers);
+    TYPE_MALLOC(char *, NUM_STRINGS(dst), dst->Strings);
+
+    memcpy(dst->Booleans,
+          src->Booleans,
+          NUM_BOOLEANS(dst) * sizeof(dst->Booleans[0]));
+    memcpy(dst->Numbers,
+          src->Numbers,
+          NUM_NUMBERS(dst) * sizeof(dst->Numbers[0]));
+    memcpy(dst->Strings,
+          src->Strings,
+          NUM_STRINGS(dst) * sizeof(dst->Strings[0]));
 
     /* FIXME: we probably should also copy str_table and ext_str_table,
      * but tic and infocmp are not written to exploit that (yet).
@@ -453,9 +505,10 @@ void _nc_copy_termtype(TERMTYPE *dst, TERMTYPE *src)
 
 #if NCURSES_XNAMES
     if ((i = NUM_EXT_NAMES(src)) != 0) {
-       dst->ext_Names = typeMalloc(char *, i);
+       TYPE_MALLOC(char *, i, dst->ext_Names);
        memcpy(dst->ext_Names, src->ext_Names, i * sizeof(char *));
+    } else {
+       dst->ext_Names = 0;
     }
 #endif
-
 }