whatnow: call external anno(1) instead of internal annotate().
[mmh] / uip / whatnow.c
index b809ebe..ae0c30b 100644 (file)
@@ -144,7 +144,7 @@ main(int argc, char **argv)
        ** Get the initial current working directory.
        */
 
-       if (getcwd(cwd, sizeof (cwd)) == NULL) {
+       if (!getcwd(cwd, sizeof (cwd))) {
                adios("getcwd", "could not get working directory");
        }
 
@@ -191,7 +191,7 @@ main(int argc, char **argv)
                        drft = cp;
        }
 
-       if ((drft == NULL && (drft = getenv("mhdraft")) == NULL) || *drft == 0)
+       if ((!drft && !(drft = getenv("mhdraft"))) || !*drft)
                drft = getcpy(m_draft(seq_cur));
 
        msgnam = (cp = getenv("mhaltmsg")) && *cp ? getcpy(cp) : NULL;
@@ -199,7 +199,7 @@ main(int argc, char **argv)
        if ((cp = getenv("mhuse")) && *cp)
                use = atoi(cp);
 
-       if (ed == NULL && ((ed = getenv("mheditor")) == NULL || *ed == 0)) {
+       if (!ed && (!(ed = getenv("mheditor")) || !*ed)) {
                ed = NULL;
                nedit++;
        }
@@ -220,7 +220,7 @@ main(int argc, char **argv)
                        /* display the msg being replied to or distributed */
                        if (msgnam) {
                                snprintf(buf, sizeof buf, "%s '%s'",
-                                               lproc, msgnam);
+                                               listproc, msgnam);
                                system(buf);
                        } else {
                                advise(NULL, "no alternate message to display");
@@ -238,7 +238,7 @@ main(int argc, char **argv)
 
                case LISTSW:
                        /* display the draft file */
-                       snprintf(buf, sizeof buf, "%s '%s'", lproc, drft);
+                       snprintf(buf, sizeof buf, "%s '%s'", listproc, drft);
                        system(buf);
                        break;
 
@@ -286,11 +286,11 @@ main(int argc, char **argv)
                        } else {
                                writesomecmd(buf, BUFSIZ, "cd", "pwd", argp);
                        }
-                       if ((f = popen_in_dir(cwd, buf, "r")) != (FILE *)0) {
+                       if ((f = popen_in_dir(cwd, buf, "r"))) {
                                fgets(cwd, sizeof (cwd), f);
 
-                               if (strchr(cwd, '\n') != NULL)
-                                               *strchr(cwd, '\n') = '\0';
+                               if (strchr(cwd, '\n'))
+                                       *strchr(cwd, '\n') = '\0';
 
                                pclose(f);
                        } else {
@@ -328,7 +328,7 @@ main(int argc, char **argv)
                        l = NULL;
                        n = 0;
 
-                       while (*++argp != NULL) {
+                       while (*++argp) {
                                if (strcmp(*argp, "-l") == 0)
                                        l = "/";
 
@@ -349,7 +349,16 @@ main(int argc, char **argv)
                                advise(NULL, "usage is alist [-ln].");
 
                        else
-                               annolist(drft, attach_hdr, l, n);
+                               snprintf(buf, sizeof buf,
+                                               "anno -list -comp '%s' "
+                                               "%s%s '%s'",
+                                               attach_hdr,
+                                               n ? "-number all" : "",
+                                               l ? "-text IGNORE " : "",
+                                               drft);
+                               if (system(buf) != 0) {
+                                       advise(NULL, "Could not list attachment headers.");
+                               }
 
                        break;
 
@@ -380,21 +389,26 @@ main(int argc, char **argv)
                        ** working directory to relative path names.
                        ** Add the attachment annotation to the draft.
                        */
-                       if ((f = popen_in_dir(cwd, buf, "r")) != (FILE *)0) {
-                               while (fgets(shell, sizeof (shell), f)
-                                               != NULL) {
+                       if ((f = popen_in_dir(cwd, buf, "r"))) {
+                               char buf[BUFSIZ];
+
+                               while (fgets(shell, sizeof(shell), f)) {
                                        *(strchr(shell, '\n')) = '\0';
 
                                        if (*shell == '/')
-                                               annotate(drft, attach_hdr,
-                                                               shell, 0,
-                                                               -2, 1, 0);
+                                               sprintf(file, "%s", shell);
                                        else {
                                                sprintf(file, "%s/%s", cwd,
                                                                shell);
-                                               annotate(drft, attach_hdr,
-                                                               file, 0,
-                                                               -2, 1, 0);
+                                       }
+                                       snprintf(buf, sizeof buf,
+                                                       "anno -nodate -append "
+                                                       "-comp '%s' -text '%s'"
+                                                       " '%s'",
+                                                       attach_hdr, file,
+                                                       drft);
+                                       if (system(buf) != 0) {
+                                               advise(NULL, "Could not add attachment header.");
                                        }
                                }
 
@@ -413,7 +427,6 @@ main(int argc, char **argv)
                        ** names and numbers aren't allowed, so this
                        ** catches a -n anywhere in the argument list.
                        */
-
                        for (n = 0, arguments = argp + 1;
                                        *arguments != NULL;
                                        arguments++) {
@@ -430,21 +443,23 @@ main(int argc, char **argv)
                        ** just processed after processing each one so
                        ** that the numbering stays correct.
                        */
-
                        if (n == 1) {
-                               for (arguments = argp + 1;
-                                               *arguments != NULL;
+                               for (arguments=argp+1; *arguments;
                                                arguments++) {
                                        if (strcmp(*arguments, "-n") == 0)
                                                continue;
 
                                        if (**arguments != '\0') {
+                                               char buf[BUFSIZ];
+
                                                n = atoi(*arguments);
-                                               annotate(drft, attach_hdr,
-                                                               NULL, 0,
-                                                               n, 1, 0);
+                                               snprintf(buf, sizeof buf, "anno -delete -comp '%s' -number '%d' '%s'", attach_hdr, n, drft);
+                                               if (system(buf) != 0) {
+                                                       advise(NULL, "Could not delete attachment header.");
+                                               }
 
-                                               for (argp = arguments + 1; *argp != NULL; argp++) {
+                                               for (argp=arguments+1; *argp;
+                                                               argp++) {
                                                        if (atoi(*argp) > n) {
                                                                if (atoi(*argp) == 1)
                                                                        *argp = "";
@@ -454,7 +469,9 @@ main(int argc, char **argv)
                                                }
                                        }
                                }
+                               break;
                        }
+                       /* else */
 
                        /*
                        ** The arguments are interpreted as file names.
@@ -469,12 +486,17 @@ main(int argc, char **argv)
                        ** with a space in it.
                        */
                        writelscmd(buf, sizeof(buf), argp);
-                       if ((f = popen_in_dir(cwd, buf, "r")) != (FILE *)0) {
-                               while (fgets(shell, sizeof (shell), f)
-                                               != NULL) {
+                       if ((f = popen_in_dir(cwd, buf, "r"))) {
+                               while (fgets(shell, sizeof (shell), f)) {
                                        *(strchr(shell, '\n')) = '\0';
-                                       annotate(drft, attach_hdr, shell,
-                                                       0, 0, 1, 0);
+                                       snprintf(buf, sizeof buf,
+                                                       "anno -delete -comp "
+                                                       "'%s' -text '%s' '%s'",
+                                                       attach_hdr, shell,
+                                                       drft);
+                                       if (system(buf) != 0) {
+                                               advise(NULL, "Could not delete attachment header.");
+                                       }
                                }
                                pclose(f);
                        } else {
@@ -525,7 +547,7 @@ writesomecmd(char *buf, int bufsz, char *cmd, char *trailcmd, char **argp)
 
        cp = buf + ln;
 
-       while (*++argp != NULL) {
+       while (*++argp) {
                ln = strlen(*argp);
                /* +1 for leading space */
                if (ln + trailln + 1 > bufsz - (cp-buf))
@@ -614,27 +636,26 @@ editfile(char **ed, char **arg, char *file, int use, struct msgs *mp,
        if (reedit) {
                if (!*ed) {  /* no explicit editor */
                        *ed = edsave;  /* so use the previous one */
-                       if ((cp = mhbasename(*ed)) == NULL)
+                       if (!(cp = mhbasename(*ed)))
                                cp = *ed;
 
                        /* unless we've specified it via "editor-next" */
                        cp = concat(cp, "-next", NULL);
-                       if ((cp = context_find(cp)) != NULL)
+                       if ((cp = context_find(cp)))
                                *ed = cp;
                }
-       } else {
+       } else if (!*ed) {
                /* set initial editor */
-               if (*ed == NULL && (*ed = context_find("editor")) == NULL)
-                       *ed = defaulteditor;
+               *ed = defaulteditor;
        }
 
        if (altmsg) {
-               if (mp == NULL || *altmsg == '/' || cwd == NULL)
+               if (!mp || *altmsg == '/' || !cwd)
                        strncpy(altpath, altmsg, sizeof(altpath));
                else
                        snprintf(altpath, sizeof(altpath), "%s/%s",
                                        mp->foldpath, altmsg);
-               if (cwd == NULL)
+               if (!cwd)
                        strncpy(linkpath, altmsglink, sizeof(linkpath));
                else
                        snprintf(linkpath, sizeof(linkpath), "%s/%s",
@@ -791,9 +812,9 @@ sendfile(char **arg, char *file, int pushsw)
                vec[vecp++] = file;
                vec[vecp] = NULL;
 
-               execvp(sendproc, vec);
+               execvp("send", vec);
                fprintf(stderr, "unable to exec ");
-               perror(sendproc);
+               perror("send");
                _exit(-1);
 
        default: