X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=init.c;h=079ecc7a0261e990a07f2aa04263000e93b9500c;hp=1c870b5745dc4200c3bc348b165a3f2b1224834b;hb=5b5c457f669423f32f1e96ed5ff784e07b4a81da;hpb=57c0b219e1b899b31397f355dd7b84e8c03a5094 diff --git a/init.c b/init.c index 1c870b5..079ecc7 100644 --- a/init.c +++ b/init.c @@ -32,6 +32,8 @@ #include "lib/intl.h" #include "lib/str.h" #include "lib/rx.h" +#include "lib/list.h" +#include "lib/debug.h" #include #include @@ -41,7 +43,36 @@ #include #include +/* for synonym warning reports: synonym found during parsing */ +typedef struct { + char* f; /* file */ + int l; /* line */ + int n; /* new name (index) */ + int o; /* old name (index) */ +} syn_t; + +/* for synonym warning reports: list of synonyms found */ list2_t* Synonyms; +/* for synonym warning reports: current rc file */ +char* CurRCFile = NULL; +/* for synonym warning reports: current rc line */ +int CurRCLine = 0; + +/* for synonym warning reports: adds synonym to end of list */ +static void syn_add (int n, int o) { + syn_t* tmp = safe_malloc (sizeof (syn_t)); + tmp->f = safe_strdup (CurRCFile); + tmp->l = CurRCLine; + tmp->n = n; + tmp->o = o; + list_push_back (&Synonyms, tmp); +} + +/* for synonym warning reports: free single item (for list_del()) */ +static void syn_del (void** p) { + FREE(&(*(syn_t**) p)->f); + FREE(p); +} void toggle_quadoption (int opt) { @@ -93,9 +124,9 @@ int mutt_option_index (char *s) int i; for (i = 0; MuttVars[i].option; i++) - if (safe_strcmp (s, MuttVars[i].option) == 0) { + if (mutt_strcmp (s, MuttVars[i].option) == 0) { if (MuttVars[i].type == DT_SYN) - list_push_back (&Synonyms, &MuttVars[i]); + syn_add (mutt_option_index ((char *) MuttVars[i].data), i); return (MuttVars[i].type == DT_SYN ? mutt_option_index ((char *) MuttVars[i].data) : i); } @@ -201,14 +232,12 @@ int mutt_extract_token (BUFFER * dest, BUFFER * tok, int flags) } } while (pc && *pc != '`'); if (!pc) { - dprint (1, (debugfile, "mutt_get_token: mismatched backtics\n")); + debug_print (1, ("mismatched backtics\n")); return (-1); } cmd = str_substrdup (tok->dptr, pc); if ((pid = mutt_create_filter (cmd, NULL, &fp, NULL)) < 0) { - dprint (1, - (debugfile, "mutt_get_token: unable to fork command: %s", - cmd)); + debug_print (1, ("unable to fork command: %s\n", cmd)); FREE (&cmd); return (-1); } @@ -231,8 +260,8 @@ int mutt_extract_token (BUFFER * dest, BUFFER * tok, int flags) FREE (&expn.data); } else if (expn.data) { - expnlen = safe_strlen (expn.data); - tok->dsize = expnlen + safe_strlen (tok->dptr) + 1; + expnlen = mutt_strlen (expn.data); + tok->dsize = expnlen + mutt_strlen (tok->dptr) + 1; ptr = safe_malloc (tok->dsize); memcpy (ptr, expn.data, expnlen); strcpy (ptr + expnlen, tok->dptr); /* __STRCPY_CHECKED__ */ @@ -400,7 +429,7 @@ static int remove_from_spam_list (SPAM_LIST ** list, const char *pat) /* Being first is a special case. */ spam = *list; - if (spam->rx && !safe_strcmp (spam->rx->pattern, pat)) { + if (spam->rx && !mutt_strcmp (spam->rx->pattern, pat)) { *list = spam->next; rx_free (&spam->rx); FREE(&spam->template); @@ -410,7 +439,7 @@ static int remove_from_spam_list (SPAM_LIST ** list, const char *pat) prev = spam; for (spam = prev->next; spam;) { - if (!safe_strcmp (spam->rx->pattern, pat)) { + if (!mutt_strcmp (spam->rx->pattern, pat)) { prev->next = spam->next; rx_free (&spam->rx); FREE(spam->template); @@ -430,7 +459,7 @@ static void remove_from_list (LIST ** l, const char *str) { LIST *p, *last = NULL; - if (safe_strcmp ("*", str) == 0) + if (mutt_strcmp ("*", str) == 0) mutt_free_list (l); /* ``unCMD *'' means delete all current entries */ else { p = *l; @@ -456,8 +485,8 @@ static int remove_from_rx_list (list2_t** l, const char *str) { int i = 0; - if (safe_strcmp ("*", str) == 0) { - list_del (l, rx_free); + if (mutt_strcmp ("*", str) == 0) { + list_del (l, (list_del_t*) rx_free); return (0); } else { @@ -489,8 +518,8 @@ static int parse_ifdef (BUFFER * tmp, BUFFER * s, unsigned long data, continue; for (j = 0; b[j].name; j++) - if (!ascii_strncasecmp (tmp->data, b[j].name, safe_strlen (tmp->data)) - && (safe_strlen (b[j].name) == safe_strlen (tmp->data))) { + if (!ascii_strncasecmp (tmp->data, b[j].name, mutt_strlen (tmp->data)) + && (mutt_strlen (b[j].name) == mutt_strlen (tmp->data))) { res = 1; break; } @@ -500,13 +529,13 @@ static int parse_ifdef (BUFFER * tmp, BUFFER * s, unsigned long data, char *p = NULL; i = 0; - j = safe_strlen (tmp->data); + j = mutt_strlen (tmp->data); /* need at least input of 'feature_X' */ if (j >= 7) { p = tmp->data + 7; j -= 7; while (Features[i].name) { - if (safe_strlen (Features[i].name) == j && + if (mutt_strlen (Features[i].name) == j && ascii_strncasecmp (Features[i].name, p, j)) { res = 1; break; @@ -612,7 +641,7 @@ static int parse_unalternates (BUFFER * buf, BUFFER * s, unsigned long data, mutt_extract_token (buf, s, 0); remove_from_rx_list (&Alternates, buf->data); - if (safe_strcmp (buf->data, "*") && + if (mutt_strcmp (buf->data, "*") && add_to_rx_list (&UnAlternates, buf->data, REG_ICASE, err) != 0) return -1; @@ -668,9 +697,9 @@ static int parse_spam_list (BUFFER * buf, BUFFER * s, unsigned long data, /* nospam only ever has one parameter. */ /* "*" is a special case. */ - if (!safe_strcmp (buf->data, "*")) { + if (!mutt_strcmp (buf->data, "*")) { mutt_free_spam_list (&SpamList); - list_del (&NoSpamList, rx_free); + list_del (&NoSpamList, (list_del_t*) rx_free); return 0; } @@ -698,7 +727,7 @@ static int parse_unlist (BUFFER * buf, BUFFER * s, unsigned long data, /* * Check for deletion of entire list */ - if (safe_strcmp (buf->data, "*") == 0) { + if (mutt_strcmp (buf->data, "*") == 0) { mutt_free_list ((LIST **) data); break; } @@ -732,7 +761,7 @@ static int parse_unlists (BUFFER * buf, BUFFER * s, unsigned long data, remove_from_rx_list (&SubscribedLists, buf->data); remove_from_rx_list (&MailLists, buf->data); - if (safe_strcmp (buf->data, "*") && + if (mutt_strcmp (buf->data, "*") && add_to_rx_list (&UnMailLists, buf->data, REG_ICASE, err) != 0) return -1; } @@ -766,7 +795,7 @@ static int parse_unsubscribe (BUFFER * buf, BUFFER * s, unsigned long data, mutt_extract_token (buf, s, 0); remove_from_rx_list (&SubscribedLists, buf->data); - if (safe_strcmp (buf->data, "*") && + if (mutt_strcmp (buf->data, "*") && add_to_rx_list (&UnSubscribedLists, buf->data, REG_ICASE, err) != 0) return -1; } @@ -783,7 +812,7 @@ static int parse_unalias (BUFFER * buf, BUFFER * s, unsigned long data, do { mutt_extract_token (buf, s, 0); - if (safe_strcmp ("*", buf->data) == 0) { + if (mutt_strcmp ("*", buf->data) == 0) { if (CurrentMenu == MENU_ALIAS) { for (tmp = Aliases; tmp; tmp = tmp->next) tmp->del = 1; @@ -831,7 +860,7 @@ static int parse_alias (BUFFER * buf, BUFFER * s, unsigned long data, mutt_extract_token (buf, s, 0); - dprint (2, (debugfile, "parse_alias: First token is '%s'.\n", buf->data)); + debug_print (2, ("first token is '%s'.\n", buf->data)); /* check to see if an alias with this name already exists */ for (; tmp; tmp = tmp->next) { @@ -858,7 +887,7 @@ static int parse_alias (BUFFER * buf, BUFFER * s, unsigned long data, mutt_extract_token (buf, s, M_TOKEN_QUOTE | M_TOKEN_SPACE | M_TOKEN_SEMICOLON); - dprint (2, (debugfile, "parse_alias: Second token is '%s'.\n", buf->data)); + debug_print (2, ("second token is '%s'.\n", buf->data)); tmp->addr = mutt_parse_adrlist (tmp->addr, buf->data); if (last) last->next = tmp; @@ -870,14 +899,14 @@ static int parse_alias (BUFFER * buf, BUFFER * s, unsigned long data, return -1; } #ifdef DEBUG - if (debuglevel >= 2) { + if (DebugLevel >= 2) { ADDRESS *a; for (a = tmp->addr; a; a = a->next) { if (!a->group) - dprint (2, (debugfile, "parse_alias: %s\n", a->mailbox)); + debug_print (2, ("%s\n", a->mailbox)); else - dprint (2, (debugfile, "parse_alias: Group %s\n", a->mailbox)); + debug_print (2, ("group %s\n", a->mailbox)); } } #endif @@ -894,13 +923,13 @@ parse_unmy_hdr (BUFFER * buf, BUFFER * s, unsigned long data, BUFFER * err) do { mutt_extract_token (buf, s, 0); - if (safe_strcmp ("*", buf->data) == 0) + if (mutt_strcmp ("*", buf->data) == 0) mutt_free_list (&UserHeader); else { tmp = UserHeader; last = NULL; - l = safe_strlen (buf->data); + l = mutt_strlen (buf->data); if (buf->data[l - 1] == ':') l--; @@ -1078,9 +1107,9 @@ static void mutt_restore_default (struct option_t *p) char *s = (char *) p->init; pp->rx = safe_calloc (1, sizeof (regex_t)); - if (safe_strcmp (p->option, "mask") != 0) + if (mutt_strcmp (p->option, "mask") != 0) flags |= mutt_which_case ((const char *) p->init); - if (safe_strcmp (p->option, "mask") == 0 && *s == '!') { + if (mutt_strcmp (p->option, "mask") == 0 && *s == '!') { s++; pp->not = 1; } @@ -1147,7 +1176,7 @@ static int parse_set (BUFFER * tmp, BUFFER * s, unsigned long data, mutt_extract_token (tmp, s, M_TOKEN_EQUAL); if ((idx = mutt_option_index (tmp->data)) == -1 && - !(reset && !safe_strcmp ("all", tmp->data))) { + !(reset && !mutt_strcmp ("all", tmp->data))) { snprintf (err->data, err->dsize, _("%s: unknown variable"), tmp->data); return (-1); } @@ -1164,7 +1193,7 @@ static int parse_set (BUFFER * tmp, BUFFER * s, unsigned long data, return (-1); } - if (!safe_strcmp ("all", tmp->data)) { + if (!mutt_strcmp ("all", tmp->data)) { for (idx = 0; MuttVars[idx].option; idx++) mutt_restore_default (&MuttVars[idx]); return 0; @@ -1248,7 +1277,7 @@ static int parse_set (BUFFER * tmp, BUFFER * s, unsigned long data, } else if (DTYPE (MuttVars[idx].type) == DT_STR) { *((char **) MuttVars[idx].data) = safe_strdup (tmp->data); - if (safe_strcmp (MuttVars[idx].option, "charset") == 0) + if (mutt_strcmp (MuttVars[idx].option, "charset") == 0) mutt_set_charset (Charset); } else { @@ -1270,7 +1299,7 @@ static int parse_set (BUFFER * tmp, BUFFER * s, unsigned long data, } if (option (OPTATTACHMSG) - && !safe_strcmp (MuttVars[idx].option, "reply_regexp")) { + && !mutt_strcmp (MuttVars[idx].option, "reply_regexp")) { snprintf (err->data, err->dsize, "Operation not permitted when in attach-message mode."); r = -1; @@ -1282,15 +1311,15 @@ static int parse_set (BUFFER * tmp, BUFFER * s, unsigned long data, /* copy the value of the string */ mutt_extract_token (tmp, s, 0); - if (!ptr->pattern || safe_strcmp (ptr->pattern, tmp->data) != 0) { + if (!ptr->pattern || mutt_strcmp (ptr->pattern, tmp->data) != 0) { int not = 0; /* $mask is case-sensitive */ - if (safe_strcmp (MuttVars[idx].option, "mask") != 0) + if (mutt_strcmp (MuttVars[idx].option, "mask") != 0) flags |= mutt_which_case (tmp->data); p = tmp->data; - if (safe_strcmp (MuttVars[idx].option, "mask") == 0) { + if (mutt_strcmp (MuttVars[idx].option, "mask") == 0) { if (*p == '!') { not = 1; p++; @@ -1319,7 +1348,7 @@ static int parse_set (BUFFER * tmp, BUFFER * s, unsigned long data, /* $reply_regexp and $alterantes require special treatment */ if (Context && Context->msgcount && - safe_strcmp (MuttVars[idx].option, "reply_regexp") == 0) { + mutt_strcmp (MuttVars[idx].option, "reply_regexp") == 0) { regmatch_t pmatch[1]; int i; @@ -1395,12 +1424,12 @@ static int parse_set (BUFFER * tmp, BUFFER * s, unsigned long data, *ptr = (short) val; /* these ones need a sanity check */ - if (safe_strcmp (MuttVars[idx].option, "history") == 0) { + if (mutt_strcmp (MuttVars[idx].option, "history") == 0) { if (*ptr < 0) *ptr = 0; mutt_init_history (); } - else if (safe_strcmp (MuttVars[idx].option, "pager_index_lines") == 0) { + else if (mutt_strcmp (MuttVars[idx].option, "pager_index_lines") == 0) { if (*ptr < 0) *ptr = 0; } @@ -1527,7 +1556,9 @@ static int source_rc (const char *rcfile, BUFFER * err) size_t buflen; pid_t pid; - dprint (2, (debugfile, "Reading configuration file '%s'.\n", rcfile)); + debug_print (2, ("reading configuration file '%s'.\n", rcfile)); + str_replace (&CurRCFile, rcfile); + CurRCLine = 0; if ((f = mutt_open_read (rcfile, &pid)) == NULL) { snprintf (err->data, err->dsize, "%s: %s", rcfile, strerror (errno)); @@ -1536,6 +1567,7 @@ static int source_rc (const char *rcfile, BUFFER * err) memset (&token, 0, sizeof (token)); while ((linebuf = mutt_read_line (linebuf, &buflen, f, &line)) != NULL) { + CurRCLine++; conv = ConfigCharset && (*ConfigCharset) && Charset; if (conv) { currentline = safe_strdup (linebuf); @@ -1618,7 +1650,7 @@ int mutt_parse_rc_line ( /* const */ char *line, BUFFER * token, BUFFER * err) memset (&expn, 0, sizeof (expn)); expn.data = expn.dptr = line; - expn.dsize = safe_strlen (line); + expn.dsize = mutt_strlen (line); *err->data = 0; @@ -1632,7 +1664,7 @@ int mutt_parse_rc_line ( /* const */ char *line, BUFFER * token, BUFFER * err) } mutt_extract_token (token, &expn, 0); for (i = 0; Commands[i].name; i++) { - if (!safe_strcmp (token->data, Commands[i].name)) { + if (!mutt_strcmp (token->data, Commands[i].name)) { if (Commands[i].func (token, &expn, Commands[i].data, err) != 0) goto finish; break; @@ -1740,8 +1772,8 @@ int mutt_command_complete (char *buffer, size_t len, int pos, int numtabs) /* loop through all the possible prefixes (no, inv, ...) */ if (!safe_strncmp (buffer, "set", 3)) { for (num = 0; prefixes[num]; num++) { - if (!safe_strncmp (pt, prefixes[num], safe_strlen (prefixes[num]))) { - pt += safe_strlen (prefixes[num]); + if (!safe_strncmp (pt, prefixes[num], mutt_strlen (prefixes[num]))) { + pt += mutt_strlen (prefixes[num]); break; } } @@ -1851,7 +1883,7 @@ int mutt_var_value_complete (char *buffer, size_t len, int pos) strfcpy (var, pt, sizeof (var)); /* ignore the trailing '=' when comparing */ - var[safe_strlen (var) - 1] = 0; + var[mutt_strlen (var) - 1] = 0; if ((idx = mutt_option_index (var)) == -1) return 0; /* no such variable. */ else { @@ -1979,31 +2011,6 @@ int mutt_getvaluebyname (const char *name, const struct mapping_t *map) return (-1); } -#ifdef DEBUG -static void start_debug (void) -{ - time_t t; - int i; - char buf[_POSIX_PATH_MAX]; - char buf2[_POSIX_PATH_MAX]; - - /* rotate the old debug logs */ - for (i = 3; i >= 0; i--) { - snprintf (buf, sizeof (buf), "%s/.muttdebug%d", NONULL (Homedir), i); - snprintf (buf2, sizeof (buf2), "%s/.muttdebug%d", NONULL (Homedir), - i + 1); - rename (buf, buf2); - } - if ((debugfile = safe_fopen (buf, "w")) != NULL) { - t = time (0); - setbuf (debugfile, NULL); /* don't buffer the debugging output! */ - fprintf (debugfile, - "Mutt-ng %s started at %s.\nDebugging at level %d.\n\n", - MUTT_VERSION, asctime (localtime (&t)), debuglevel); - } -} -#endif - static int mutt_execute_commands (LIST * p) { BUFFER err, token; @@ -2074,11 +2081,7 @@ void mutt_init (int skip_sys_rc, LIST * commands) Shell = safe_strdup ((p = getenv ("SHELL")) ? p : "/bin/sh"); } -#ifdef DEBUG - /* Start up debugging mode if requested */ - if (debuglevel > 0) - start_debug (); -#endif + debug_start(Homedir); /* And about the host... */ uname (&utsname); @@ -2098,7 +2101,7 @@ void mutt_init (int skip_sys_rc, LIST * commands) else #endif /* DOMAIN */ if (*DOMAIN != '@') { - Fqdn = safe_malloc (safe_strlen (DOMAIN) + safe_strlen (Hostname) + 2); + Fqdn = safe_malloc (mutt_strlen (DOMAIN) + mutt_strlen (Hostname) + 2); sprintf (Fqdn, "%s.%s", NONULL (Hostname), DOMAIN); /* __SPRINTF_CHECKED__ */ } else @@ -2167,7 +2170,7 @@ void mutt_init (int skip_sys_rc, LIST * commands) memset (&buf, 0, sizeof (buf)); buf.data = buf.dptr = buffer; - buf.dsize = safe_strlen (buffer); + buf.dsize = mutt_strlen (buffer); memset (&token, 0, sizeof (token)); parse_my_hdr (&token, &buf, 0, &err); @@ -2292,13 +2295,17 @@ void mutt_init (int skip_sys_rc, LIST * commands) int i = 0; fprintf (stderr, _("Warning: the following synonym variables were found:\n")); for (i = 0; i < Synonyms->length; i++) - fprintf (stderr, "$%s ($%s should be used)\n", - ((struct option_t*) Synonyms->data[i])->option, - (char*) ((struct option_t*) Synonyms->data[i])->data); + fprintf (stderr, "$%s ($%s should be used) (%s:%d)\n", + MuttVars[((syn_t*) Synonyms->data[i])->o].option, + MuttVars[((syn_t*) Synonyms->data[i])->n].option, + NONULL(((syn_t*) Synonyms->data[i])->f), + ((syn_t*) Synonyms->data[i])->l); fprintf (stderr, _("Warning: Synonym variables are scheduled for removal.\n")); - list_del (&Synonyms, NULL); + list_del (&Synonyms, syn_del); need_pause = 1; } + /* this is not needed during runtime */ + FREE(&CurRCFile); if (need_pause && !option (OPTNOCURSES)) { if (mutt_any_key_to_continue (NULL) == -1) @@ -2319,3 +2326,50 @@ int mutt_get_hook_type (const char *name) return c->data; return 0; } + +static int opt_cmp (const void* a, const void* b) { + return (mutt_strcmp ((*(struct option_t**) a)->option, + (*(struct option_t**) b)->option)); +} + +/* dump out the value of all the variables we have */ +int mutt_dump_variables (void) { + int i; + + char errbuff[STRING]; + char command[STRING]; + list2_t* tmp = NULL; + + BUFFER err, token; + + memset (&err, 0, sizeof (err)); + memset (&token, 0, sizeof (token)); + + err.data = errbuff; + err.dsize = sizeof (errbuff); + + /* get all non-synonyms into list... */ + for (i = 0; MuttVars[i].option; i++) { + if (MuttVars[i].type == DT_SYN) + continue; + list_push_back (&tmp, &MuttVars[i]); + } + if (!list_empty(tmp)) { + /* ...and dump list sorted */ + qsort (tmp->data, tmp->length, sizeof (void*), opt_cmp); + for (i = 0; i < tmp->length; i++) { + snprintf (command, sizeof (command), "set ?%s\n", + ((struct option_t*) tmp->data[i])->option); + if (mutt_parse_rc_line (command, &token, &err) == -1) { + fprintf (stderr, "%s\n", err.data); + FREE (&token.data); + list_del (&tmp, NULL); + return 1; + } + printf("%s\n", err.data); + } + } + FREE (&token.data); + list_del (&tmp, NULL); + return 0; +}