]> diplodocus.org Git - nmh/blobdiff - uip/mhbuild.c
projects / nmh / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
fdcompare.c: Move interface to own file.
[nmh] / uip / mhbuild.c
diff --git a/uip/mhbuild.c b/uip/mhbuild.c
index fe6eae7eee918170d776d646002817be882cc72d..fe21b15bc2926fcafc16977119a5890a9b6562e6 100644 (file)
--- a/uip/mhbuild.c
+++ b/uip/mhbuild.c
@@ -5,15 +5,27 @@
  * complete copyright information.
  */
 
  * complete copyright information.
  */
 
-#include <h/mh.h>
+#include "h/mh.h"
+#include "sbr/context_find.h"
+#include "sbr/readconfig.h"
+#include "sbr/ambigsw.h"
+#include "sbr/path.h"
+#include "sbr/print_version.h"
+#include "sbr/print_help.h"
+#include "sbr/error.h"
 #include <fcntl.h>
 #include <fcntl.h>
-#include <h/md5.h>
-#include <h/mts.h>
-#include <h/tws.h>
-#include <h/mime.h>
-#include <h/mhparse.h>
-#include <h/mhcachesbr.h>
-#include <h/utils.h>
+#include "h/md5.h"
+#include "h/mts.h"
+#include "h/tws.h"
+#include "h/mime.h"
+#include "h/mhparse.h"
+#include "h/mhcachesbr.h"
+#include "h/done.h"
+#include "h/utils.h"
+#include "sbr/m_maildir.h"
+#include "sbr/m_mktemp.h"
+#include "mhfree.h"
+#include "mhoutsbr.h"
 
 #define MHBUILD_SWITCHES \
     X("auto", 0, AUTOSW) \
 
 #define MHBUILD_SWITCHES \
     X("auto", 0, AUTOSW) \
@@ -68,38 +80,29 @@ DEFINE_SWITCH_ENUM(MIMEENCODING);
 DEFINE_SWITCH_ARRAY(MIMEENCODING, encodingswitches);
 #undef X
 
 DEFINE_SWITCH_ARRAY(MIMEENCODING, encodingswitches);
 #undef X
 
-/* mhcachesbr.c */
-extern int rcachesw;
-extern int wcachesw;
-extern char *cache_public;
-extern char *cache_private;
-
 int debugsw = 0;
 
 int debugsw = 0;
 
-int listsw   = 0;
-int rfc934sw = 0;
-int contentidsw = 1;
+bool listsw;
+bool rfc934sw;
+bool contentidsw = true;
 
 /*
  * Temporary files
  */
 static char infile[BUFSIZ];
 
 /*
  * Temporary files
  */
 static char infile[BUFSIZ];
-static int unlink_infile  = 0;
-
 static char outfile[BUFSIZ];
 static char outfile[BUFSIZ];
-static int unlink_outfile = 0;
-
-static void unlink_done (int) NORETURN;
-
-/* mhbuildsbr.c */
-int output_message_fp (CT, FILE *, char*);
 
 
 int
 main (int argc, char **argv)
 {
 
 
 int
 main (int argc, char **argv)
 {
-    int sizesw = 1, headsw = 1, directives = 1, autobuild = 0, dist = 0;
-    int verbosw = 0, dispo = 0;
+    bool sizesw = true;
+    bool headsw = true;
+    bool directives = true;
+    bool autobuild = false;
+    bool dist = false;
+    bool verbosw = false;
+    bool dispo = false;
     size_t maxunencoded = MAXTEXTPERLN;
     int *icachesw;
     char *cp, buf[BUFSIZ];
     size_t maxunencoded = MAXTEXTPERLN;
     int *icachesw;
     char *cp, buf[BUFSIZ];
@@ -111,9 +114,7 @@ main (int argc, char **argv)
     int header_encoding = CE_UNKNOWN;
     size_t n;
 
     int header_encoding = CE_UNKNOWN;
     size_t n;
 
-    if (nmh_init(argv[0], 2)) { return 1; }
-
-    done=unlink_done;
+    if (nmh_init(argv[0], true, false)) { return 1; }
 
     arguments = getarguments (invo_name, argc, argv, 1);
     argp = arguments;
 
     arguments = getarguments (invo_name, argc, argv, 1);
     argp = arguments;
@@ -121,11 +122,10 @@ main (int argc, char **argv)
     while ((cp = *argp++)) {
        if (cp[0] == '-' && cp[1] == '\0') {
            if (compfile)
     while ((cp = *argp++)) {
        if (cp[0] == '-' && cp[1] == '\0') {
            if (compfile)
-               adios (NULL, "cannot specify both standard input and a file");
-           else
-               compfile = cp;
-           listsw = 0;         /* turn off -list if using standard in/out */
-           verbosw = 0;        /* turn off -verbose listings */
+               die("cannot specify both standard input and a file");
+            compfile = cp;
+           listsw = false;     /* turn off -list if using standard in/out */
+           verbosw = false;    /* turn off -verbose listings */
            break;
        }
        if (*cp == '-') {
            break;
        }
        if (*cp == '-') {
@@ -134,7 +134,7 @@ main (int argc, char **argv)
                ambigsw (cp, switches);
                done (1);
            case UNKWNSW: 
                ambigsw (cp, switches);
                done (1);
            case UNKWNSW: 
-               adios (NULL, "-%s unknown", cp);
+               die("-%s unknown", cp);
 
            case HELPSW: 
                snprintf (buf, sizeof(buf), "%s [switches] file", invo_name);
 
            case HELPSW: 
                snprintf (buf, sizeof(buf), "%s [switches] file", invo_name);
@@ -146,16 +146,16 @@ main (int argc, char **argv)
 
            case AUTOSW:
                /* -auto implies -nodirectives */
 
            case AUTOSW:
                /* -auto implies -nodirectives */
-               autobuild = 1;
-               directives = 0;
+               autobuild = true;
+               directives = false;
                continue;
            case NAUTOSW:
                /*
                 * We're turning directives back on since this is likely here
                 * to override a profile entry
                 */
                continue;
            case NAUTOSW:
                /*
                 * We're turning directives back on since this is likely here
                 * to override a profile entry
                 */
-               autobuild = 0;
-               directives = 1;
+               autobuild = false;
+               directives = true;
                continue;
 
            case RCACHESW:
                continue;
 
            case RCACHESW:
@@ -165,13 +165,13 @@ main (int argc, char **argv)
                icachesw = &wcachesw;
            do_cache: ;
                if (!(cp = *argp++) || *cp == '-')
                icachesw = &wcachesw;
            do_cache: ;
                if (!(cp = *argp++) || *cp == '-')
-                   adios (NULL, "missing argument to %s", argp[-2]);
-               switch (*icachesw = smatch (cp, caches)) {
+                   die("missing argument to %s", argp[-2]);
+               switch (*icachesw = smatch (cp, cache_policy)) {
                case AMBIGSW:
                case AMBIGSW:
-                   ambigsw (cp, caches);
+                   ambigsw (cp, cache_policy);
                    done (1);
                case UNKWNSW:
                    done (1);
                case UNKWNSW:
-                   adios (NULL, "%s unknown", cp);
+                   die("%s unknown", cp);
                default:
                    break;
                }
                default:
                    break;
                }
@@ -185,58 +185,58 @@ main (int argc, char **argv)
                continue;
 
            case HEADSW:
                continue;
 
            case HEADSW:
-               headsw++;
+               headsw = true;
                continue;
            case NHEADSW:
                continue;
            case NHEADSW:
-               headsw = 0;
+               headsw = false;
                continue;
 
            case DIRECTIVES:
                continue;
 
            case DIRECTIVES:
-               directives = 1;
+               directives = true;
                continue;
            case NDIRECTIVES:
                continue;
            case NDIRECTIVES:
-               directives = 0;
+               directives = false;
                continue;
 
            case LISTSW:
                continue;
 
            case LISTSW:
-               listsw++;
+               listsw = true;
                continue;
            case NLISTSW:
                continue;
            case NLISTSW:
-               listsw = 0;
+               listsw = false;
                continue;
 
            case RFC934SW:
                continue;
 
            case RFC934SW:
-               rfc934sw++;
+               rfc934sw = true;
                continue;
            case NRFC934SW:
                continue;
            case NRFC934SW:
-               rfc934sw = 0;
+               rfc934sw = false;
                continue;
 
            case SIZESW:
                continue;
 
            case SIZESW:
-               sizesw++;
+               sizesw = true;
                continue;
            case NSIZESW:
                continue;
            case NSIZESW:
-               sizesw = 0;
+               sizesw = false;
                continue;
 
            case CONTENTIDSW:
                continue;
 
            case CONTENTIDSW:
-               contentidsw = 1;
+               contentidsw = true;
                continue;
            case NCONTENTIDSW:
                continue;
            case NCONTENTIDSW:
-               contentidsw = 0;
+               contentidsw = false;
                continue;
 
            case HEADERENCSW: {
                int encoding;
 
                if (!(cp = *argp++) || *cp == '-')
                continue;
 
            case HEADERENCSW: {
                int encoding;
 
                if (!(cp = *argp++) || *cp == '-')
-                   adios (NULL, "missing argument to %s", argp[-2]);
+                   die("missing argument to %s", argp[-2]);
                switch (encoding = smatch (cp, encodingswitches)) {
                case AMBIGSW:
                    ambigsw (cp, encodingswitches);
                    done (1);
                case UNKWNSW:
                switch (encoding = smatch (cp, encodingswitches)) {
                case AMBIGSW:
                    ambigsw (cp, encodingswitches);
                    done (1);
                case UNKWNSW:
-                   adios (NULL, "%s unknown encoding algorithm", cp);
+                   die("%s unknown encoding algorithm", cp);
                case BASE64SW:
                    header_encoding = CE_BASE64;
                    break;
                case BASE64SW:
                    header_encoding = CE_BASE64;
                    break;
@@ -247,7 +247,7 @@ main (int argc, char **argv)
                    header_encoding = CE_8BIT;
                    break;
                default:
                    header_encoding = CE_8BIT;
                    break;
                default:
-                   adios (NULL, "Internal error: algorithm %s", cp);
+                   die("Internal error: algorithm %s", cp);
                }
                continue;
            }
                }
                continue;
            }
@@ -258,37 +258,36 @@ main (int argc, char **argv)
 
            case MAXUNENCSW:
                if (!(cp = *argp++) || *cp == '-')
 
            case MAXUNENCSW:
                if (!(cp = *argp++) || *cp == '-')
-                   adios (NULL, "missing argument to %s", argp[-2]);
+                   die("missing argument to %s", argp[-2]);
                if ((maxunencoded = atoi(cp)) < 1)
                if ((maxunencoded = atoi(cp)) < 1)
-                   adios (NULL, "Invalid argument for %s: %s", argp[-2], cp);
+                   die("Invalid argument for %s: %s", argp[-2], cp);
                if (maxunencoded > 998)
                if (maxunencoded > 998)
-                   adios (NULL, "limit of -maxunencoded is 998");
+                   die("limit of -maxunencoded is 998");
                continue;
 
            case VERBSW: 
                continue;
 
            case VERBSW: 
-               verbosw++;
+               verbosw = true;
                continue;
            case NVERBSW: 
                continue;
            case NVERBSW: 
-               verbosw = 0;
+               verbosw = false;
                continue;
            case DISPOSW:
                continue;
            case DISPOSW:
-               dispo = 1;
+               dispo = true;
                continue;
            case NDISPOSW:
                continue;
            case NDISPOSW:
-               dispo = 0;
+               dispo = false;
                continue;
            case DEBUGSW:
                debugsw = 1;
                continue;
            case DISTSW:
                continue;
            case DEBUGSW:
                debugsw = 1;
                continue;
            case DISTSW:
-               dist = 1;
+               dist = true;
                continue;
            }
        }
        if (compfile)
                continue;
            }
        }
        if (compfile)
-           adios (NULL, "only one composition file allowed");
-       else
-           compfile = cp;
+           die("only one composition file allowed");
+        compfile = cp;
     }
 
     /*
     }
 
     /*
@@ -296,7 +295,7 @@ main (int argc, char **argv)
      */
     if ((cp = getenv ("MHBUILD"))) {
        if ((fp = fopen (cp, "r"))) {
      */
     if ((cp = getenv ("MHBUILD"))) {
        if ((fp = fopen (cp, "r"))) {
-           readconfig ((struct node **) 0, fp, cp, 0);
+           readconfig(NULL, fp, cp, 0);
            fclose (fp);
        } else {
            admonish ("", "unable to read $MHBUILD profile (%s)", cp);
            fclose (fp);
        } else {
            admonish ("", "unable to read $MHBUILD profile (%s)", cp);
@@ -307,7 +306,7 @@ main (int argc, char **argv)
      * Read the standard profile setup
      */
     if ((fp = fopen (cp = etcpath ("mhn.defaults"), "r"))) {
      * Read the standard profile setup
      */
     if ((fp = fopen (cp = etcpath ("mhn.defaults"), "r"))) {
-       readconfig ((struct node **) 0, fp, cp, 0);
+       readconfig(NULL, fp, cp, 0);
        fclose (fp);
     }
 
        fclose (fp);
     }
 
@@ -318,32 +317,29 @@ main (int argc, char **argv)
     /* Check for private cache location */
     if (!(cache_private = context_find (nmhprivcache)))
        cache_private = ".cache";
     /* Check for private cache location */
     if (!(cache_private = context_find (nmhprivcache)))
        cache_private = ".cache";
-    cache_private = getcpy (m_maildir (cache_private));
+    cache_private = mh_xstrdup(m_maildir(cache_private));
 
     if (!context_find ("path"))
        free (path ("./", TFOLDER));
 
     /* Check if we have a file to process */
     if (!compfile)
 
     if (!context_find ("path"))
        free (path ("./", TFOLDER));
 
     /* Check if we have a file to process */
     if (!compfile)
-       adios (NULL, "need to specify a %s composition file", invo_name);
+       die("need to specify a %s composition file", invo_name);
 
     /*
      * Process the composition file from standard input.
      */
     if (compfile[0] == '-' && compfile[1] == '\0') {
        if ((cp = m_mktemp2(NULL, invo_name, NULL, &fp)) == NULL) {
 
     /*
      * Process the composition file from standard input.
      */
     if (compfile[0] == '-' && compfile[1] == '\0') {
        if ((cp = m_mktemp2(NULL, invo_name, NULL, &fp)) == NULL) {
-           adios(NULL, "unable to create temporary file in %s",
+           die("unable to create temporary file in %s",
                  get_temp_dir());
        }
                  get_temp_dir());
        }
-
-       /* save a copy of the name for later removal */
        strncpy (infile, cp, sizeof(infile));
        strncpy (infile, cp, sizeof(infile));
-       unlink_infile = 1;
 
        /* copy standard input to temporary file */
        while ((n = fread(buffer, 1, sizeof(buffer), stdin)) > 0) {
            if (fwrite(buffer, 1, n, fp) != n) {
 
        /* copy standard input to temporary file */
        while ((n = fread(buffer, 1, sizeof(buffer), stdin)) > 0) {
            if (fwrite(buffer, 1, n, fp) != n) {
-               adios (NULL, "error copying to temporary file");
+               die("error copying to temporary file");
            }
        }
        fclose (fp);
            }
        }
        fclose (fp);
@@ -359,11 +355,11 @@ main (int argc, char **argv)
 
        if (!ct) {
            if (! (fp = fopen(infile, "r"))) {
 
        if (!ct) {
            if (! (fp = fopen(infile, "r"))) {
-               adios(NULL, "Unable to open %s for reading", infile);
+               die("Unable to open %s for reading", infile);
            }
            while ((n = fread(buffer, 1, sizeof(buffer), fp)) > 0) {
                if (fwrite(buffer, 1, n, stdout) != n) {
            }
            while ((n = fread(buffer, 1, sizeof(buffer), fp)) > 0) {
                if (fwrite(buffer, 1, n, stdout) != n) {
-                   adios(NULL, "error copying %s to stdout", infile);
+                   die("error copying %s to stdout", infile);
                }
            }
        } else {
                }
            }
        } else {
@@ -396,10 +392,9 @@ main (int argc, char **argv)
 
     /* output MIME message to this temporary file */
     if ((cp = m_mktemp2(compfile, invo_name, NULL, &fp_out)) == NULL) {
 
     /* output MIME message to this temporary file */
     if ((cp = m_mktemp2(compfile, invo_name, NULL, &fp_out)) == NULL) {
-       adios(NULL, "unable to create temporary file in %s", get_temp_dir());
+       die("unable to create temporary file");
     }
     strncpy(outfile, cp, sizeof(outfile));
     }
     strncpy(outfile, cp, sizeof(outfile));
-    unlink_outfile = 1;
 
     /* output the message */
     output_message_fp (ct, fp_out, outfile);
 
     /* output the message */
     output_message_fp (ct, fp_out, outfile);
@@ -423,25 +418,12 @@ main (int argc, char **argv)
        rename (buffer, compfile);
        done (1);
     }
        rename (buffer, compfile);
        done (1);
     }
-    unlink_outfile = 0;
+    /* Remove from atexit(3) list of files to unlink. */
+    if (!(m_unlink(outfile) == -1 && errno == ENOENT)) {
+        adios(outfile, "file exists after rename:");
+    }
 
     free_content (ct);
     done (0);
     return 1;
 }
 
     free_content (ct);
     done (0);
     return 1;
 }
-
-
-static void NORETURN
-unlink_done (int status)
-{
-    /*
-     * Check if we need to remove stray
-     * temporary files.
-     */
-    if (unlink_infile)
-       (void) m_unlink (infile);
-    if (unlink_outfile)
-       (void) m_unlink (outfile);
-
-    exit (status);
-}
Fork of https://git.savannah.nongnu.org/cgit/nmh.git