X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/5548c9aa00ddf0989b2a0471a16e9d2f6d434983..c02f66668d32980a3770e450061c5b96b9f31690:/sbr/readconfig.c

diff --git a/sbr/readconfig.c b/sbr/readconfig.c
index 1ddd648f..5b72113c 100644
--- a/sbr/readconfig.c
+++ b/sbr/readconfig.c
@@ -23,19 +23,16 @@ static struct procstr procs[] = {
     { "fileproc",      &fileproc },
     { "formatproc",    &formatproc },
     { "incproc",       &incproc },
-    { "installproc",   &installproc },
     { "lproc",         &lproc },
     { "mailproc",      &mailproc },
     { "mhlproc",       &mhlproc },
     { "moreproc",      &moreproc },
-    { "mshproc",       &mshproc },
     { "packproc",      &packproc },
     { "postproc",      &postproc },
     { "rmmproc",       &rmmproc },
     { "sendproc",      &sendproc },
     { "showmimeproc",  &showmimeproc },
     { "showproc",      &showproc },
-    { "vmhproc",       &vmhproc },
     { "whatnowproc",   &whatnowproc },
     { "whomproc",      &whomproc },
     { NULL,            NULL }
@@ -45,32 +42,34 @@ static struct node **opp = NULL;
 
 
 void
-readconfig (struct node **npp, FILE *ib, char *file, int ctx)
+readconfig (struct node **npp, FILE *ib, const char *file, int ctx)
 {
-    register int state;
-    register char *cp;
+    int state;
+    char *cp;
     char name[NAMESZ], field[BUFSIZ];
-    register struct node *np;
-    register struct procstr *ps;
+    struct node *np;
+    struct procstr *ps;
+    m_getfld_state_t gstate = 0;
 
     if (npp == NULL && (npp = opp) == NULL) {
 	admonish (NULL, "bug: readconfig called but pump not primed");
 	return;
     }
 
-    for (state = FLD;;) {
-	switch (state = m_getfld (state, name, field, sizeof(field), ib)) {
+    for (;;) {
+	int fieldsz = sizeof field;
+	switch (state = m_getfld (&gstate, name, field, &fieldsz, ib)) {
 	    case FLD:
 	    case FLDPLUS:
-	    case FLDEOF:
-		np = (struct node *) mh_xmalloc (sizeof(*np));
+		NEW(np);
 		*npp = np;
 		*(npp = &np->n_next) = NULL;
-		np->n_name = getcpy (name);
+		np->n_name = mh_xstrdup(name);
 		if (state == FLDPLUS) {
-		    cp = getcpy (field);
+		    cp = mh_xstrdup(field);
 		    while (state == FLDPLUS) {
-			state = m_getfld (state, name, field, sizeof(field), ib);
+			fieldsz = sizeof field;
+			state = m_getfld (&gstate, name, field, &fieldsz, ib);
 			cp = add (field, cp);
 		    }
 		    np->n_field = trimcpy (cp);
@@ -89,12 +88,9 @@ readconfig (struct node **npp, FILE *ib, char *file, int ctx)
 			*ps->procnaddr = np->n_field;
 			break;
 		    }
-		if (state == FLDEOF)
-		    break;
 		continue;
 
 	    case BODY:
-	    case BODYEOF:
 		adios (NULL, "no blank lines are permitted in %s", file);
 
 	    case FILEEOF:
@@ -105,6 +101,7 @@ readconfig (struct node **npp, FILE *ib, char *file, int ctx)
 	}
 	break;
     }
+    m_getfld_state_destroy (&gstate);
 
     /*
      * Special handling for the pager processes: lproc and moreproc.
@@ -133,7 +130,7 @@ readconfig (struct node **npp, FILE *ib, char *file, int ctx)
 	   mhn.defaults can be overridden without triggering
 	   warnings.
 
-	   Note that that mhn.defaults, $MHN, $MHBUILD, $MHSHOW, and
+	   Note that mhn.defaults, $MHN, $MHBUILD, $MHSHOW, and
 	   $MHSTORE all put their entries into just one list, m_defs,
 	   the same list that the profile uses. */
 
@@ -141,10 +138,10 @@ readconfig (struct node **npp, FILE *ib, char *file, int ctx)
 	for (np = m_defs; np; np = np->n_next) {
 	    /* Yes, this is O(N^2).  The profile should be small enough so
 	       that's not a performance problem. */
-	    if (strlen (np->n_name) > 0	 &&  strcmp ("#", np->n_name)) {
+	    if (*np->n_name && strcmp("#", np->n_name)) {
 		struct node *np2;
 		for (np2 = np->n_next; np2; np2 = np2->n_next) {
-		    if (! mh_strcasecmp (np->n_name, np2->n_name)) {
+		    if (! strcasecmp (np->n_name, np2->n_name)) {
 			admonish (NULL, "multiple \"%s\" profile components "
 					"in %s, ignoring \"%s\"",
 				  np->n_name, defpath, np2->n_field);
@@ -156,3 +153,19 @@ readconfig (struct node **npp, FILE *ib, char *file, int ctx)
 
     opp = npp;
 }
+
+
+void
+add_profile_entry (const char *key, const char *value) {
+    struct node *newnode;
+
+    /* This inserts the new node at the beginning of m_defs because
+       that doesn't require traversing it or checking to see if it's
+       empty. */
+    NEW(newnode);
+    newnode->n_name = getcpy (key);
+    newnode->n_field = getcpy (value);
+    newnode->n_context = 0;
+    newnode->n_next = m_defs;
+    m_defs = newnode;
+}