X-Git-Url: http://ncurses.scripts.mit.edu/?a=blobdiff_plain;f=progs%2Ftic.c;h=65b5fb75aea8d338dda6e2f1817b4a58d289311b;hb=6b99a559185b3b8fad80b56bc2070b08101c33d1;hp=9efb4b034d16bb08154cd81590eb589974da4b55;hpb=b60a2772d9f149d8e900c1d5a09a53a56a0837a8;p=ncurses.git diff --git a/progs/tic.c b/progs/tic.c index 9efb4b03..65b5fb75 100644 --- a/progs/tic.c +++ b/progs/tic.c @@ -48,7 +48,7 @@ #include #include -MODULE_ID("$Id: tic.c,v 1.266 2019/02/10 02:25:46 tom Exp $") +MODULE_ID("$Id: tic.c,v 1.272 2019/04/06 20:18:54 tom Exp $") #define STDIN_NAME "" @@ -1095,6 +1095,50 @@ main(int argc, char *argv[]) static void check_acs(TERMTYPE2 *tp) { + int vt100_smacs = 0; + int vt100_rmacs = 0; + int vt100_enacs = 0; + + /* + * ena_acs is not always necessary, but if it is present, the enter/exit + * capabilities should be. + */ + ANDMISSING(ena_acs, enter_alt_charset_mode); + ANDMISSING(ena_acs, exit_alt_charset_mode); + PAIRED(exit_alt_charset_mode, exit_alt_charset_mode); + + /* + * vt100-like is frequently used, but perhaps ena_acs is missing, etc. + */ + if (VALID_STRING(enter_alt_charset_mode)) { + vt100_smacs = (!strcmp("\033(0", enter_alt_charset_mode) + ? 2 + : (!strcmp("\016", enter_alt_charset_mode) + ? 1 + : 0)); + } + if (VALID_STRING(exit_alt_charset_mode)) { + vt100_rmacs = (!strcmp("\033(B", exit_alt_charset_mode) + ? 2 + : (!strcmp("\017", exit_alt_charset_mode) + ? 1 + : 0)); + } + if (VALID_STRING(ena_acs)) { + vt100_enacs = (!strcmp("\033(B\033)0", ena_acs) + ? 2 + : 0); + } + if (vt100_rmacs && vt100_smacs && (vt100_rmacs != vt100_smacs)) { + _nc_warning("rmacs/smacs are inconsistent"); + } + if ((vt100_rmacs == 2) && (vt100_smacs == 2) && vt100_enacs) { + _nc_warning("rmacs/smacs make enacs redundant"); + } + if ((vt100_rmacs == 1) && (vt100_smacs == 1) && !vt100_enacs) { + _nc_warning("VT100-style rmacs/smacs require enacs"); + } + if (VALID_STRING(acs_chars)) { const char *boxes = "lmkjtuvwqxn"; char mapped[256]; @@ -1622,6 +1666,7 @@ check_printer(TERMTYPE2 *tp) #endif } +#if NCURSES_XNAMES static bool uses_SGR_39_49(const char *value) { @@ -1635,7 +1680,6 @@ uses_SGR_39_49(const char *value) static void check_screen(TERMTYPE2 *tp) { -#if NCURSES_XNAMES if (_nc_user_definable) { int have_XT = tigetflag("XT"); int have_XM = tigetflag("XM"); @@ -1691,8 +1735,10 @@ check_screen(TERMTYPE2 *tp) } } } -#endif } +#else +#define check_screen(tp) /* nothing */ +#endif /* * Returns the expected number of parameters for the given capability. @@ -1788,59 +1834,21 @@ expected_params(const char *name) return result; } -typedef struct { - const char *name; - int n_type; - int n_parms; -} USERCAPS; - /* - * These are user-capabilities that happen to be used in ncurses' terminal + * Check for user-capabilities that happen to be used in ncurses' terminal * database. */ -static USERCAPS * +#if NCURSES_XNAMES +static struct user_table_entry const * lookup_user_capability(const char *name) { - /* *INDENT-OFF* */ -#define DATA(name,type,parms) { name, type, parms } - static USERCAPS table[] = { - DATA( "AX", BOOLEAN, 0 ), - DATA( "Cr", STRING, 0 ), - DATA( "Cs", STRING, 1 ), - DATA( "E0", STRING, 0 ), - DATA( "E3", STRING, 0 ), - DATA( "G0", BOOLEAN, 0 ), - DATA( "Ms", STRING, 2 ), - DATA( "RGB", BOOLEAN, 0 ), /* FIXME can be number or string */ - DATA( "S0", STRING, 1 ), - DATA( "Se", STRING, 0 ), - DATA( "Smulx", STRING, 1 ), - DATA( "Ss", STRING, 1 ), - DATA( "TS", STRING, 0 ), - DATA( "U8", NUMBER, 0 ), - DATA( "XM", STRING, 1 ), - DATA( "XT", BOOLEAN, 0 ), - DATA( "grbom", STRING, 0 ), - DATA( "gsbom", STRING, 0 ), - DATA( "rmxx", STRING, 0 ), - DATA( "smxx", STRING, 0 ), - DATA( "xm", STRING, 9 ), - }; -#undef DATA - /* *INDENT-ON* */ - - size_t n; - USERCAPS *result = 0; + struct user_table_entry const *result = 0; if (*name != 'k') { - for (n = 0; n < SIZEOF(table); ++n) { - if (!strcmp(name, table[n].name)) { - result = &table[n]; - break; - } - } + result = _nc_find_user_entry(name); } return result; } +#endif /* * If a given name is likely to be a user-capability, return the number of @@ -1862,12 +1870,15 @@ is_user_capability(const char *name) (name[1] >= '0' && name[1] <= '9') && name[2] == '\0') { result = (name[1] == '6') ? 2 : 0; - } else if (using_extensions) { - USERCAPS *p = lookup_user_capability(name); + } +#if NCURSES_XNAMES + else if (using_extensions) { + struct user_table_entry const *p = lookup_user_capability(name); if (p != 0) { - result = p->n_parms; + result = (int) p->ute_argc; } } +#endif return result; } @@ -1914,6 +1925,7 @@ check_params(TERMTYPE2 *tp, const char *name, char *value, int extended) s++; } +#if NCURSES_XNAMES if (extended) { int check = is_user_capability(name); if (check != actual) { @@ -1925,6 +1937,7 @@ check_params(TERMTYPE2 *tp, const char *name, char *value, int extended) } expected = actual; } +#endif if (params[0]) { _nc_warning("%s refers to parameter 0 (%%p0), which is not allowed", name); @@ -1953,13 +1966,16 @@ check_params(TERMTYPE2 *tp, const char *name, char *value, int extended) analyzed = popcount; } if (actual != analyzed && expected != analyzed) { +#if NCURSES_XNAMES int user_cap = is_user_capability(name); if ((user_cap == analyzed) && using_extensions) { ; /* ignore */ } else if (user_cap >= 0) { _nc_warning("tparm will use %d parameters for %s, expected %d", analyzed, name, user_cap); - } else { + } else +#endif + { _nc_warning("tparm analyzed %d parameters for %s, expected %d", analyzed, name, actual); } @@ -2801,6 +2817,7 @@ check_sgr_param(TERMTYPE2 *tp, int code, const char *name, char *value) } } +#if NCURSES_XNAMES static int standard_type(const char *name) { @@ -2834,20 +2851,12 @@ name_of_type(int type) static void check_user_capability_type(const char *name, int actual) { - USERCAPS *p = lookup_user_capability(name); - if (p != 0) { - if (p->n_type != actual) - _nc_warning("expected %s to be %s, but actually %s", - name, - name_of_type(p->n_type), - name_of_type(actual) - ); - } else { + if (lookup_user_capability(name) == 0) { int expected = standard_type(name); if (expected >= 0) { _nc_warning("expected %s to be %s, but actually %s", name, - name_of_type(p->n_type), + name_of_type(actual), name_of_type(expected) ); } else if (*name != 'k') { @@ -2857,6 +2866,7 @@ check_user_capability_type(const char *name, int actual) } } } +#endif /* other sanity-checks (things that we don't want in the normal * logic that reads a terminfo entry)