]> diplodocus.org Git - nmh/commitdiff
Try to suppress warnings from changes (but why is this thing complaining
authorKen Hornstein <kenh@pobox.com>
Thu, 23 Jan 2014 19:40:44 +0000 (14:40 -0500)
committerKen Hornstein <kenh@pobox.com>
Thu, 23 Jan 2014 19:40:44 +0000 (14:40 -0500)
about drft being clobbered now??)

uip/sendsbr.c
uip/whatnowsbr.c

index fd80a0f00d84ad00675ca69eeb153b23a3da8612..30046a3b6f13a1ef30a0309d309e6b3f3b08bc11 100644 (file)
@@ -52,7 +52,7 @@ static int sendaux (char **, int, char *, char *, struct stat *);
  */
 
 int
  */
 
 int
-sendsbr (char **vec, int vecp, char *program, char *drft, struct stat *st,
+sendsbr (char **vec, int vecp, char *program, char *draft, struct stat *st,
          int rename_drft)
 {
     int status, i;
          int rename_drft)
 {
     int status, i;
@@ -60,6 +60,7 @@ sendsbr (char **vec, int vecp, char *program, char *drft, struct stat *st,
     char buffer[BUFSIZ], file[BUFSIZ];
     struct stat sts;
     char **buildvec, *buildprogram;
     char buffer[BUFSIZ], file[BUFSIZ];
     struct stat sts;
     char **buildvec, *buildprogram;
+    volatile char *drft = draft;
 
     /*
      * Run the mimebuildproc (which is by default mhbuild) on the message
 
     /*
      * Run the mimebuildproc (which is by default mhbuild) on the message
@@ -76,7 +77,7 @@ sendsbr (char **vec, int vecp, char *program, char *drft, struct stat *st,
        buildvec[i++] = "-auto";
        if (distfile)
            buildvec[i++] = "-dist";
        buildvec[i++] = "-auto";
        if (distfile)
            buildvec[i++] = "-dist";
-       buildvec[i++] = drft;
+       buildvec[i++] = (char *) drft;
        buildvec[i] = NULL;
        execvp(buildprogram, buildvec);
        fprintf(stderr, "unable to exec ");
        buildvec[i] = NULL;
        execvp(buildprogram, buildvec);
        fprintf(stderr, "unable to exec ");
@@ -98,11 +99,12 @@ sendsbr (char **vec, int vecp, char *program, char *drft, struct stat *st,
         * rename the draft file.  I'm not quite sure why.
         */
        if (pushsw && unique) {
         * rename the draft file.  I'm not quite sure why.
         */
        if (pushsw && unique) {
-            char *cp = m_mktemp2(drft, invo_name, NULL, NULL);
+            char *cp = m_mktemp2((char *) drft, invo_name, NULL, NULL);
             if (cp == NULL) {
                 adios ("sendsbr", "unable to create temporary file");
             }
             if (cp == NULL) {
                 adios ("sendsbr", "unable to create temporary file");
             }
-           if (rename (drft, strncpy(file, cp, sizeof(file))) == NOTOK)
+           if (rename ((char *) drft,
+                       strncpy(file, cp, sizeof(file))) == NOTOK)
                adios (file, "unable to rename %s to", drft);
            drft = file;
        }
                adios (file, "unable to rename %s to", drft);
            drft = file;
        }
@@ -111,17 +113,21 @@ sendsbr (char **vec, int vecp, char *program, char *drft, struct stat *st,
         * Check if we need to split the message into
         * multiple messages of type "message/partial".
         */
         * Check if we need to split the message into
         * multiple messages of type "message/partial".
         */
-       if (splitsw >= 0 && !distfile && stat (drft, &sts) != NOTOK
+       if (splitsw >= 0 && !distfile && stat ((char *) drft, &sts) != NOTOK
                && sts.st_size >= CPERMSG) {
                && sts.st_size >= CPERMSG) {
-           status = splitmsg (vec, vecp, program, drft, st, splitsw) ? NOTOK : OK;
+           status = splitmsg (vec, vecp, program, (char *) drft,
+                              st, splitsw) ? NOTOK : OK;
        } else {
        } else {
-           status = sendaux (vec, vecp, program, drft, st) ? NOTOK : OK;
+           status = sendaux (vec, vecp, program, (char *) drft,
+                             st) ? NOTOK : OK;
        }
 
        /* rename the original draft */
        if (rename_drft && status == OK &&
        }
 
        /* rename the original draft */
        if (rename_drft && status == OK &&
-               rename (drft, strncpy (buffer, m_backup (drft), sizeof(buffer))) == NOTOK)
-           advise (buffer, "unable to rename %s to", drft);
+               rename ((char *) drft,
+                       strncpy (buffer, m_backup ((char *) drft),
+                                sizeof(buffer))) == NOTOK)
+           advise (buffer, "unable to rename %s to", (char *) drft);
        break;
 
     default: 
        break;
 
     default: 
index 68339b5c5d0e4479b482f9368bedba7089884083..005095a8a9818e409d3f99663e209c3928e2d0a9 100644 (file)
@@ -104,7 +104,6 @@ static int editfile (char **, char **, char *, int, struct msgs *,
 static int sendfile (char **, char *, int);
 static void sendit (char *, char **, char *, int);
 static int buildfile (char **, char *);
 static int sendfile (char **, char *, int);
 static void sendit (char *, char **, char *, int);
 static int buildfile (char **, char *);
-static int check_draft (char *);
 static int whomfile (char **, char *);
 static int removefile (char *);
 static void writelscmd(char *, int, char *, char **);
 static int whomfile (char **, char *);
 static int removefile (char *);
 static void writelscmd(char *, int, char *, char **);
@@ -885,66 +884,6 @@ buildfile (char **argp, char *file)
 }
 
 
 }
 
 
-/*
- *  Check if draft is a mhbuild composition file
- */
-
-static int
-check_draft (char *msgnam)
-{
-    int        state;
-    char buf[BUFSIZ], name[NAMESZ];
-    FILE *fp;
-    m_getfld_state_t gstate = 0;
-
-    if ((fp = fopen (msgnam, "r")) == NULL)
-       return 0;
-    for (;;) {
-       int bufsz = sizeof buf;
-       switch (state = m_getfld (&gstate, name, buf, &bufsz, fp)) {
-           case FLD:
-           case FLDPLUS:
-               /*
-                * If draft already contains any of the
-                * Content-XXX fields, then assume it already
-                * been converted.
-                */
-               if (uprf (name, XXX_FIELD_PRF)) {
-                   fclose (fp);
-                   m_getfld_state_destroy (&gstate);
-                   return 0;
-               }
-               while (state == FLDPLUS) {
-                   bufsz = sizeof buf;
-                   state = m_getfld (&gstate, name, buf, &bufsz, fp);
-               }
-               break;
-
-           case BODY:
-               do {
-                   char *bp;
-
-                   for (bp = buf; *bp; bp++)
-                       if (*bp != ' ' && *bp != '\t' && *bp != '\n') {
-                           fclose (fp);
-                           m_getfld_state_destroy (&gstate);
-                           return 1;
-                       }
-
-                   bufsz = sizeof buf;
-                   state = m_getfld (&gstate, name, buf, &bufsz, fp);
-               } while (state == BODY);
-               /* and fall... */
-
-           default:
-               fclose (fp);
-               m_getfld_state_destroy (&gstate);
-               return 0;
-       }
-    }
-}
-
-
 #ifndef CYRUS_SASL
 # define SASLminc(a) (a)
 #else /* CYRUS_SASL */
 #ifndef CYRUS_SASL
 # define SASLminc(a) (a)
 #else /* CYRUS_SASL */