vector.c: Move interface to own file.

[nmh] / uip / folder.c

diff --git a/uip/folder.c b/uip/folder.c
index 4f36ed0e3aba3e913d8b5365722482f45552a650..946482938ff0fd3f4639105c3a8ef60aac3164b7 100644 (file)
--- a/uip/folder.c
+++ b/uip/folder.c
@@ -1,6 +1,4 @@
-
-/*
- * folder(s).c -- set/list the current message and/or folder
+/* folder.c -- set/list the current message and/or folder

  *             -- push/pop a folder onto/from the folder stack
  *             -- list the folder stack
  *
  *             -- push/pop a folder onto/from the folder stack
  *             -- list the folder stack
  *


@@ -9,9 +7,15 @@
  * complete copyright information.
  */
 
  * complete copyright information.
  */
 

-#include <h/mh.h>
-#include <h/crawl_folders.h>
-#include <h/utils.h>
+#include "h/mh.h"
+#include "sbr/path.h"
+#include "sbr/print_version.h"
+#include "sbr/print_help.h"
+#include "sbr/error.h"
+#include "h/crawl_folders.h"
+#include "h/done.h"
+#include "h/utils.h"
+#include "sbr/m_maildir.h"

 
 #define FOLDER_SWITCHES \
     X("all", 0, ALLSW) \
 
 #define FOLDER_SWITCHES \
     X("all", 0, ALLSW) \


@@ -47,14 +51,14 @@ DEFINE_SWITCH_ENUM(FOLDER);
 DEFINE_SWITCH_ARRAY(FOLDER, switches);
 #undef X
 
 DEFINE_SWITCH_ARRAY(FOLDER, switches);
 #undef X
 

-static int fshort   = 0;       /* output only folder names                 */
+static bool fshort;            /* output only folder names                 */

 static int fcreat   = 0;       /* should we ask to create new folders?     */
 static int fcreat   = 0;       /* should we ask to create new folders?     */

-static int fpack    = 0;       /* are we packing the folder?               */
-static int fverb    = 0;       /* print actions taken while packing folder */
+static bool fpack;             /* are we packing the folder?               */
+static bool fverb;             /* print actions taken while packing folder */

 static int fheader  = 0;       /* should we output a header?               */
 static int fheader  = 0;       /* should we output a header?               */

-static int frecurse = 0;       /* recurse through subfolders               */
+static bool frecurse;          /* recurse through subfolders               */

 static int ftotal   = 0;       /* should we output the totals?             */
 static int ftotal   = 0;       /* should we output the totals?             */

-static int all      = 0;       /* should we output all folders             */
+static bool all;               /* should we output all folders             */

 
 static int total_folders = 0;  /* total number of folders                  */
 
 
 static int total_folders = 0;  /* total number of folders                  */
 


@@ -96,29 +100,31 @@ static void readonly_folders (void);
  * Function for printing error message if folder does not exist with
  * -nocreate.
  */
  * Function for printing error message if folder does not exist with
  * -nocreate.
  */

-void
-nonexistant_folder (int status) {
+static void
+nonexistent_folder (int status)
+{

     NMH_UNUSED (status);
     NMH_UNUSED (status);

-    adios (NULL, "folder %s does not exist", folder);
+    die("folder %s does not exist", folder);

 }
 
 
 int
 main (int argc, char **argv)
 {
 }
 
 
 int
 main (int argc, char **argv)
 {

-    int printsw = 0, listsw = 0;
-    int pushsw = 0, popsw = 0;
+    bool printsw = false;
+    bool listsw = false;
+    bool pushsw = false;
+    bool popsw = false;

     char *cp, *dp, *msg = NULL, *argfolder = NULL;
     char **ap, **argp, buf[BUFSIZ], **arguments;
 
     char *cp, *dp, *msg = NULL, *argfolder = NULL;
     char **ap, **argp, buf[BUFSIZ], **arguments;
 

-    if (nmh_init(argv[0], 1)) { return 1; }
+    if (nmh_init(argv[0], true, true)) { return 1; }

 
     /*
      * If program was invoked with name ending
      * in `s', then add switch `-all'.
      */
 
     /*
      * If program was invoked with name ending
      * in `s', then add switch `-all'.
      */

-    if (argv[0][strlen (argv[0]) - 1] == 's')
-       all = 1;
+    all = has_suffix_c(argv[0], 's');

 
     arguments = getarguments (invo_name, argc, argv, 1);
     argp = arguments;
 
     arguments = getarguments (invo_name, argc, argv, 1);
     argp = arguments;


@@ -130,7 +136,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 [+folder] [msg] [switches]",
 
                case HELPSW: 
                    snprintf (buf, sizeof(buf), "%s [+folder] [msg] [switches]",


@@ -142,11 +148,11 @@ main (int argc, char **argv)
                    done (0);
 
                case ALLSW: 
                    done (0);
 
                case ALLSW: 

-                   all = 1;
+                   all = true;

                    continue;
 
                case NALLSW:
                    continue;
 
                case NALLSW:

-                   all = 0;
+                   all = false;

                    continue;
 
                case CREATSW: 
                    continue;
 
                case CREATSW: 


@@ -157,10 +163,10 @@ main (int argc, char **argv)
                    continue;
 
                case FASTSW: 
                    continue;
 
                case FASTSW: 

-                   fshort++;
+                   fshort = true;

                    continue;
                case NFASTSW: 
                    continue;
                case NFASTSW: 

-                   fshort = 0;
+                   fshort = false;

                    continue;
 
                case HDRSW: 
                    continue;
 
                case HDRSW: 


@@ -171,24 +177,24 @@ main (int argc, char **argv)
                    continue;
 
                case PACKSW: 
                    continue;
 
                case PACKSW: 

-                   fpack++;
+                   fpack = true;

                    continue;
                case NPACKSW: 
                    continue;
                case NPACKSW: 

-                   fpack = 0;
+                   fpack = false;

                    continue;
 
                case VERBSW:
                    continue;
 
                case VERBSW:

-                   fverb++;
+                   fverb = true;

                    continue;
                case NVERBSW:
                    continue;
                case NVERBSW:

-                   fverb = 0;
+                   fverb = false;

                    continue;
 
                case RECURSW: 
                    continue;
 
                case RECURSW: 

-                   frecurse++;
+                   frecurse = true;

                    continue;
                case NRECRSW: 
                    continue;
                case NRECRSW: 

-                   frecurse = 0;
+                   frecurse = false;

                    continue;
 
                case TOTALSW: 
                    continue;
 
                case TOTALSW: 


@@ -199,41 +205,39 @@ main (int argc, char **argv)
                    continue;
 
                case PRNTSW: 
                    continue;
 
                case PRNTSW: 

-                   printsw = 1;
+                   printsw = true;

                    continue;
                case NPRNTSW: 
                    continue;
                case NPRNTSW: 

-                   printsw = 0;
+                   printsw = false;

                    continue;
 
                case LISTSW: 
                    continue;
 
                case LISTSW: 

-                   listsw = 1;
+                   listsw = true;

                    continue;
                case NLISTSW: 
                    continue;
                case NLISTSW: 

-                   listsw = 0;
+                   listsw = false;

                    continue;
 
                case PUSHSW: 
                    continue;
 
                case PUSHSW: 

-                   pushsw = 1;
-                   listsw = 1;
-                   popsw  = 0;
+                   pushsw = true;
+                   listsw = true;
+                   popsw  = false;

                    continue;
                case POPSW: 
                    continue;
                case POPSW: 

-                   popsw  = 1;
-                   listsw = 1;
-                   pushsw = 0;
+                   popsw  = true;
+                   listsw = true;
+                   pushsw = false;

                    continue;
            }
        }
        if (*cp == '+' || *cp == '@') {
            if (argfolder)
                    continue;
            }
        }
        if (*cp == '+' || *cp == '@') {
            if (argfolder)

-               adios (NULL, "only one folder at a time!");
-           else
-               argfolder = pluspath (cp);
+               die("only one folder at a time!");
+            argfolder = pluspath (cp);

        } else {
            if (msg)
        } else {
            if (msg)

-               adios (NULL, "only one (current) message at a time!");
-           else
-               msg = cp;
+               die("only one (current) message at a time!");
+            msg = cp;

        }
     }
 
        }
     }
 


@@ -245,8 +249,8 @@ main (int argc, char **argv)
      * If we aren't working with the folder stack
      * (-push, -pop, -list) then the default is to print.
      */
      * If we aren't working with the folder stack
      * (-push, -pop, -list) then the default is to print.
      */

-    if (pushsw == 0 && popsw == 0 && listsw == 0)
-       printsw++;
+    if (!pushsw && !popsw && !listsw)
+       printsw = true;

 
     /* Pushing a folder onto the folder stack */
     if (pushsw) {
 
     /* Pushing a folder onto the folder stack */
     if (pushsw) {


@@ -254,13 +258,13 @@ main (int argc, char **argv)
            /* If no folder is given, the current folder and */
            /* the top of the folder stack are swapped.      */
            if ((cp = context_find (stack))) {
            /* If no folder is given, the current folder and */
            /* the top of the folder stack are swapped.      */
            if ((cp = context_find (stack))) {

-               dp = getcpy (cp);
+               dp = mh_xstrdup(cp);

                ap = brkstring (dp, " ", "\n");
                argfolder = getcpy(*ap++);
            } else {
                ap = brkstring (dp, " ", "\n");
                argfolder = getcpy(*ap++);
            } else {

-               adios (NULL, "no other folder");
+               die("no other folder");

            }
            }

-           for (cp = getcpy (getfolder (1)); *ap; ap++)
+           for (cp = mh_xstrdup(getfolder(1)); *ap; ap++)

                cp = add (*ap, add (" ", cp));
            free (dp);
            context_replace (stack, cp);        /* update folder stack */
                cp = add (*ap, add (" ", cp));
            free (dp);
            context_replace (stack, cp);        /* update folder stack */


@@ -269,20 +273,20 @@ main (int argc, char **argv)
            context_replace (stack,
                    (cp = context_find (stack))
                    ? concat (getfolder (1), " ", cp, NULL)
            context_replace (stack,
                    (cp = context_find (stack))
                    ? concat (getfolder (1), " ", cp, NULL)

-                   : getcpy (getfolder (1)));
+                   : mh_xstrdup(getfolder(1)));

        }
     }
 
     /* Popping a folder off of the folder stack */
     if (popsw) {
        if (argfolder)
        }
     }
 
     /* Popping a folder off of the folder stack */
     if (popsw) {
        if (argfolder)

-           adios (NULL, "sorry, no folders allowed with -pop");
+           die("sorry, no folders allowed with -pop");

        if ((cp = context_find (stack))) {
        if ((cp = context_find (stack))) {

-           dp = getcpy (cp);
+           dp = mh_xstrdup(cp);

            ap = brkstring (dp, " ", "\n");
            argfolder = getcpy(*ap++);
        } else {
            ap = brkstring (dp, " ", "\n");
            argfolder = getcpy(*ap++);
        } else {

-           adios (NULL, "folder stack empty");
+           die("folder stack empty");

        }
        if (*ap) {
            /* if there's anything left in the stack */
        }
        if (*ap) {
            /* if there's anything left in the stack */


@@ -306,14 +310,14 @@ main (int argc, char **argv)
 
     /* Listing the folder stack */
     if (listsw) {
 
     /* Listing the folder stack */
     if (listsw) {

-       printf ("%s", argfolder ? argfolder : getfolder (1));
+       fputs(argfolder ? argfolder : getfolder (1), stdout);

        if ((cp = context_find (stack))) {
        if ((cp = context_find (stack))) {

-           dp = getcpy (cp);
+           dp = mh_xstrdup(cp);

            for (ap = brkstring (dp, " ", "\n"); *ap; ap++)
                printf (" %s", *ap);
            free (dp);
        }
            for (ap = brkstring (dp, " ", "\n"); *ap; ap++)
                printf (" %s", *ap);
            free (dp);
        }

-       printf ("\n");
+       putchar('\n');

 
        if (!printsw)
            done (0);
 
        if (!printsw)
            done (0);


@@ -335,9 +339,10 @@ main (int argc, char **argv)
         */
        if (!argfolder) {
            if (msg)
         */
        if (!argfolder) {
            if (msg)

-               admonish (NULL, "no folder given for message %s", msg);
+               inform("no folder given for message %s, continuing...", msg);

            readonly_folders (); /* do any readonly folders */
            readonly_folders (); /* do any readonly folders */

-           strncpy (folder, (cp = context_find (pfolder)) ? cp : "", sizeof(folder));
+           cp = context_find(pfolder);
+           strncpy (folder, FENDNULL(cp), sizeof(folder));

            crawl_folders (".", get_folder_info_callback, NULL);
        } else {
            strncpy (folder, argfolder, sizeof(folder));
            crawl_folders (".", get_folder_info_callback, NULL);
        } else {
            strncpy (folder, argfolder, sizeof(folder));


@@ -359,7 +364,7 @@ main (int argc, char **argv)
         * Check if folder exists.  If not, then see if
         * we should create it, or just exit.
         */
         * Check if folder exists.  If not, then see if
         * we should create it, or just exit.
         */

-        create_folder (m_maildir (folder), fcreat, nonexistant_folder);
+        create_folder (m_maildir (folder), fcreat, nonexistent_folder);

 
        if (get_folder_info (folder, msg) && argfolder) {
            /* update current folder */
 
        if (get_folder_info (folder, msg) && argfolder) {
            /* update current folder */


@@ -378,7 +383,7 @@ main (int argc, char **argv)
 }
 
 static int
 }
 
 static int

-get_folder_info_body (char *fold, char *msg, boolean *crawl_children)
+get_folder_info_body (char *fold, char *msg, bool *crawl_children)

 {
     int        i, retval = 1;
     struct msgs *mp = NULL;
 {
     int        i, retval = 1;
     struct msgs *mp = NULL;


@@ -407,8 +412,8 @@ get_folder_info_body (char *fold, char *msg, boolean *crawl_children)
         * create message structure and get folder info
         */
        if (!(mp = folder_read (fold, fpack))) {
         * create message structure and get folder info
         */
        if (!(mp = folder_read (fold, fpack))) {

-           admonish (NULL, "unable to read folder %s", fold);
-           *crawl_children = FALSE;
+           inform("unable to read folder %s, continuing...", fold);
+           *crawl_children = false;

            return 0;
        }
 
            return 0;
        }
 


@@ -418,7 +423,7 @@ get_folder_info_body (char *fold, char *msg, boolean *crawl_children)
 
        if (fpack) {
            if (folder_pack (&mp, fverb) == -1) {
 
        if (fpack) {
            if (folder_pack (&mp, fverb) == -1) {

-               *crawl_children = FALSE; /* to please clang static analyzer */
+               *crawl_children = false; /* to please clang static analyzer */

                done (1);
            }
            seq_save (mp);              /* synchronize the sequences */
                done (1);
            }
            seq_save (mp);              /* synchronize the sequences */


@@ -442,10 +447,10 @@ get_folder_info_body (char *fold, char *msg, boolean *crawl_children)
     return retval;
 }
 
     return retval;
 }
 

-static boolean
+static bool

 get_folder_info_callback (char *fold, void *baton)
 {
 get_folder_info_callback (char *fold, void *baton)
 {

-    boolean crawl_children;
+    bool crawl_children;

     NMH_UNUSED (baton);
 
     get_folder_info_body (fold, NULL, &crawl_children);
     NMH_UNUSED (baton);
 
     get_folder_info_body (fold, NULL, &crawl_children);


@@ -456,7 +461,7 @@ get_folder_info_callback (char *fold, void *baton)
 static int
 get_folder_info (char *fold, char *msg)
 {
 static int
 get_folder_info (char *fold, char *msg)
 {

-    boolean crawl_children;
+    bool crawl_children;

     int retval;
 
     retval = get_folder_info_body (fold, msg, &crawl_children);
     int retval;
 
     retval = get_folder_info_body (fold, msg, &crawl_children);


@@ -475,7 +480,9 @@ get_folder_info (char *fold, char *msg)
 static void
 print_folders (void)
 {
 static void
 print_folders (void)
 {

-    int i, len, hasempty = 0, curprinted;
+    int i, len;
+    bool hasempty = false;
+    bool curprinted;

     int maxlen = 0, maxnummsg = 0, maxlowmsg = 0;
     int maxhghmsg = 0, maxcurmsg = 0, total_msgs = 0;
     int nummsgdigits, lowmsgdigits;
     int maxlen = 0, maxnummsg = 0, maxlowmsg = 0;
     int maxhghmsg = 0, maxcurmsg = 0, total_msgs = 0;
     int nummsgdigits, lowmsgdigits;


@@ -519,7 +526,7 @@ print_folders (void)
 
        /* check for empty folders */
        if (fi[i].nummsg == 0)
 
        /* check for empty folders */
        if (fi[i].nummsg == 0)

-           hasempty = 1;
+           hasempty = true;

     }
     nummsgdigits = num_digits (maxnummsg);
     lowmsgdigits = num_digits (maxlowmsg);
     }
     nummsgdigits = num_digits (maxnummsg);
     lowmsgdigits = num_digits (maxlowmsg);


@@ -546,7 +553,7 @@ print_folders (void)
     if (all || fshort || ftotal < 1) {
        for (i = 0; i < total_folders; i++) {
            if (fshort) {
     if (all || fshort || ftotal < 1) {
        for (i = 0; i < total_folders; i++) {
            if (fshort) {

-               printf ("%s\n", fi[i].name);
+               puts(fi[i].name);

                continue;
            }
 
                continue;
            }
 


@@ -563,26 +570,26 @@ print_folders (void)
 
            printf ("%-*s ", maxlen+1, tmpname);
 
 
            printf ("%-*s ", maxlen+1, tmpname);
 

-           curprinted = 0; /* remember if we print cur */
+           curprinted = false; /* remember if we print cur */

            if (fi[i].nummsg == 0) {
                printf ("has %*s messages%*s",
                        nummsgdigits, "no",
                        fi[i].others ? lowmsgdigits + hghmsgdigits + 5 : 0, "");
            } else {
            if (fi[i].nummsg == 0) {
                printf ("has %*s messages%*s",
                        nummsgdigits, "no",
                        fi[i].others ? lowmsgdigits + hghmsgdigits + 5 : 0, "");
            } else {

-               printf ("has %*d message%s  (%*d-%*d)",
+               printf ("has %*d message%1s  (%*d-%*d)",

                        nummsgdigits, fi[i].nummsg,
                        nummsgdigits, fi[i].nummsg,

-                       (fi[i].nummsg == 1) ? " " : "s",
+                       PLURALS(fi[i].nummsg),

                        lowmsgdigits, fi[i].lowmsg,
                        hghmsgdigits, fi[i].hghmsg);
                if (fi[i].curmsg >= fi[i].lowmsg && fi[i].curmsg <= fi[i].hghmsg) {
                        lowmsgdigits, fi[i].lowmsg,
                        hghmsgdigits, fi[i].hghmsg);
                if (fi[i].curmsg >= fi[i].lowmsg && fi[i].curmsg <= fi[i].hghmsg) {

-                   curprinted = 1;
+                   curprinted = true;

                    printf ("; cur=%*d", curmsgdigits, fi[i].curmsg);
                }
            }
 
            if (fi[i].others)
                printf (";%*s (others)", curprinted ? 0 : curmsgdigits + 6, "");
                    printf ("; cur=%*d", curmsgdigits, fi[i].curmsg);
                }
            }
 
            if (fi[i].others)
                printf (";%*s (others)", curprinted ? 0 : curmsgdigits + 6, "");

-           printf (".\n");
+           puts(".");

        }
     }
 
        }
     }
 


@@ -591,17 +598,17 @@ print_folders (void)
      */
     if (ftotal > 0 || (all && !fshort && ftotal >= 0)) {
        if (all)
      */
     if (ftotal > 0 || (all && !fshort && ftotal >= 0)) {
        if (all)

-           printf ("\n");
-       printf ("TOTAL = %d message%c in %d folder%s.\n",
-               total_msgs, total_msgs != 1 ? 's' : ' ',
-               total_folders, total_folders != 1 ? "s" : "");
+           putchar('\n');
+       printf ("TOTAL = %d message%s in %d folder%s.\n",
+               total_msgs, PLURALS(total_msgs),
+               total_folders, PLURALS(total_folders));

     }
 
     fflush (stdout);
 }
 
 /*
     }
 
     fflush (stdout);
 }
 
 /*

- * Set the current message and sychronize sequences
+ * Set the current message and synchronize sequences

  */
 
 static int
  */
 
 static int


@@ -612,7 +619,7 @@ sfold (struct msgs *mp, char *msg)
        return 0;
 
     if (mp->numsel > 1) {
        return 0;
 
     if (mp->numsel > 1) {

-       admonish (NULL, "only one message at a time!");
+       inform("only one message at a time!, continuing...");

        return 0;
     }
     seq_setprev (mp);          /* set the previous-sequence     */
        return 0;
     }
     seq_setprev (mp);          /* set the previous-sequence     */


@@ -633,7 +640,7 @@ readonly_folders (void)
 {
     int        atrlen;
     char atrcur[BUFSIZ];
 {
     int        atrlen;
     char atrcur[BUFSIZ];

-    register struct node *np;
+    struct node *np;

 
     snprintf (atrcur, sizeof(atrcur), "atr-%s-", current);
     atrlen = strlen (atrcur);
 
     snprintf (atrcur, sizeof(atrcur), "atr-%s-", current);
     atrlen = strlen (atrcur);