X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/57b69f8f9204cde25e33e310d22f76fd42963fb8..04ee0c3f6ad7e4177cc43a0c57f046013a4fbac9:/uip/mhparam.c

diff --git a/uip/mhparam.c b/uip/mhparam.c
index edbefd78..6db9b3ad 100644
--- a/uip/mhparam.c
+++ b/uip/mhparam.c
@@ -1,6 +1,4 @@
-
-/*
- * mhparam.c -- print mh_profile values
+/* mhparam.c -- print mh_profile values
  *
  * Originally contributed by
  * Jeffrey C Honig <Jeffrey_C_Honig@cornell.edu>
@@ -10,8 +8,13 @@
  * complete copyright information.
  */
 
-#include <h/mh.h>
-#include <h/mts.h>
+#include "h/mh.h"
+#include "sbr/print_version.h"
+#include "sbr/print_help.h"
+#include "sbr/error.h"
+#include "h/mts.h"
+#include "h/done.h"
+#include "h/utils.h"
 
 #define MHPARAM_SWITCHES \
     X("components", 0, COMPSW) \
@@ -38,7 +41,7 @@ static char *sbackup = BACKUP_PREFIX;
 
 static char *datalocking = "fcntl";
 static char *localmbox = "";
-static int localmbox_primed = 0;
+static bool localmbox_primed;
 
 extern char *spoollocking;
 
@@ -115,7 +118,6 @@ static struct proc procs [] = {
      { "whomproc",         &whomproc },
      { "bindir",           &mhbindir },
      { "libexecdir",       &mhlibexecdir },
-     { "libdir",           &mhlibexecdir },
      { "etcdir",           &mhetcdir },
      { "docdir",           &mhdocdir },
      { "localmbox",	   &localmbox },
@@ -133,19 +135,21 @@ static struct proc procs [] = {
 /*
  * static prototypes
  */
-static char *p_find(char *);
+static char *p_find(char *) PURE;
 
 
 int
 main(int argc, char **argv)
 {
-    int i, compp = 0, missed = 0;
-    int all = 0, debug = 0;
+    int i, compp = 0;
+    bool missed;
+    bool all = false;
+    bool debug = false;
     int components = -1;
     char *cp, buf[BUFSIZ], **argp;
     char **arguments, *comps[MAXARGS];
 
-    if (nmh_init(argv[0], 2)) { return 1; }
+    if (nmh_init(argv[0], true, false)) { return 1; }
 
     arguments = getarguments (invo_name, argc, argv, 1);
     argp = arguments;
@@ -157,7 +161,7 @@ main(int argc, char **argv)
 		    ambigsw (cp, switches);
 		    done (1);
 		case UNKWNSW: 
-		    adios (NULL, "-%s unknown", cp);
+		    die("-%s unknown", cp);
 
 		case HELPSW: 
 		    snprintf (buf, sizeof(buf), "%s [profile-components] [switches]",
@@ -176,18 +180,18 @@ main(int argc, char **argv)
 		    break;
 
 		case ALLSW:
-		    all = 1;
+		    all = true;
 		    break;
 
 		case DEBUGSW:
-		    debug = 1;
+		    debug = true;
 		    break;
 	    }
 	} else {
 	    comps[compp++] = cp;
 	    if (strcmp("localmbox", cp) == 0 && ! localmbox_primed) {
 		localmbox = getlocalmbox();
-		localmbox_primed = 1;
+		localmbox_primed = true;
 	    }
 	}
     }
@@ -196,10 +200,10 @@ main(int argc, char **argv)
         struct node *np;
 
 	if (compp)
-	    advise(NULL, "profile-components ignored with -all");
+	    inform("profile-components ignored with -all");
 
 	if (components >= 0)
-	    advise(NULL, "-%scomponents ignored with -all",
+	    inform("-%scomponents ignored with -all",
 		   components ? "" : "no");
 
 	/* Print all entries in context/profile list.  That does not
@@ -221,7 +225,7 @@ main(int argc, char **argv)
 	/* Also set localmbox here */
 	if (! localmbox_primed) {
 	    localmbox = getlocalmbox();
-	    localmbox_primed = 1;
+	    localmbox_primed = true;
 	}
 
 	/*
@@ -230,10 +234,11 @@ main(int argc, char **argv)
 	 * value (as determined after context is read).
          */
 	for (ps = procs; ps->p_name; ps++)
-	    printf ("%s: %s\n", ps->p_name, *ps->p_field ? *ps->p_field : "");
+	    printf ("%s: %s\n", ps->p_name, FENDNULL(*ps->p_field));
 
     }
 
+    missed = false;
     if (! all) {
         if (components < 0)
 	    components = compp > 1;
@@ -254,13 +259,12 @@ main(int argc, char **argv)
 		    printf("%s: ", comps[i]);
 
 		puts(value);
-	    } else if (missed < 120)
-	        missed++;
+	    } else
+                missed = true;
 	}
     }
 
-    done (missed);
-    return 1;
+    done(missed);
 }
 
 
@@ -271,7 +275,7 @@ p_find(char *str)
 
     for (ps = procs; ps->p_name; ps++)
 	if (!strcasecmp (ps->p_name, str))
-	    return (*ps->p_field);
+	    return *ps->p_field;
 
     return NULL;
 }