X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/5f7208794aead37a0bd63831b698f81054676844..a5c6f670c3636f018a65738645bf712e99d27698:/uip/folder.c diff --git a/uip/folder.c b/uip/folder.c index ef28a9b4..625d68ad 100644 --- 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 * @@ -12,6 +10,7 @@ #include #include #include +#include "sbr/m_maildir.h" #define FOLDER_SWITCHES \ X("all", 0, ALLSW) \ @@ -33,7 +32,7 @@ X("list", 0, LISTSW) \ X("nolist", 0, NLISTSW) \ X("print", 0, PRNTSW) \ - X("noprint", -4, NPRNTSW) \ + X("noprint", 0, NPRNTSW) \ X("push", 0, PUSHSW) \ X("pop", 0, POPSW) \ X("version", 0, VERSIONSW) \ @@ -96,8 +95,9 @@ static void readonly_folders (void); * Function for printing error message if folder does not exist with * -nocreate. */ +static void -nonexistant_folder (int status) { +nonexistent_folder (int status) { NMH_UNUSED (status); adios (NULL, "folder %s does not exist", folder); } @@ -117,8 +117,7 @@ main (int argc, char **argv) * 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; @@ -227,13 +226,11 @@ main (int argc, char **argv) if (*cp == '+' || *cp == '@') { if (argfolder) adios (NULL, "only one folder at a time!"); - else - argfolder = pluspath (cp); + argfolder = pluspath (cp); } else { if (msg) adios (NULL, "only one (current) message at a time!"); - else - msg = cp; + msg = cp; } } @@ -254,7 +251,7 @@ 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))) { - dp = getcpy (cp); + dp = mh_xstrdup(cp); ap = brkstring (dp, " ", "\n"); argfolder = getcpy(*ap++); } else { @@ -278,7 +275,7 @@ main (int argc, char **argv) if (argfolder) adios (NULL, "sorry, no folders allowed with -pop"); if ((cp = context_find (stack))) { - dp = getcpy (cp); + dp = mh_xstrdup(cp); ap = brkstring (dp, " ", "\n"); argfolder = getcpy(*ap++); } else { @@ -306,14 +303,14 @@ main (int argc, char **argv) /* Listing the folder stack */ if (listsw) { - printf ("%s", argfolder ? argfolder : getfolder (1)); + fputs(argfolder ? argfolder : getfolder (1), stdout); if ((cp = context_find (stack))) { - dp = getcpy (cp); + dp = mh_xstrdup(cp); for (ap = brkstring (dp, " ", "\n"); *ap; ap++) printf (" %s", *ap); free (dp); } - printf ("\n"); + putchar('\n'); if (!printsw) done (0); @@ -335,9 +332,10 @@ main (int argc, char **argv) */ 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 */ - 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)); @@ -359,7 +357,7 @@ main (int argc, char **argv) * 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 */ @@ -378,7 +376,7 @@ main (int argc, char **argv) } 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; @@ -407,8 +405,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))) { - admonish (NULL, "unable to read folder %s", fold); - *crawl_children = FALSE; + inform("unable to read folder %s, continuing...", fold); + *crawl_children = false; return 0; } @@ -418,7 +416,7 @@ get_folder_info_body (char *fold, char *msg, boolean *crawl_children) 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 */ @@ -442,10 +440,10 @@ get_folder_info_body (char *fold, char *msg, boolean *crawl_children) return retval; } -static boolean +static bool 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); @@ -456,7 +454,7 @@ get_folder_info_callback (char *fold, void *baton) 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); @@ -546,7 +544,7 @@ print_folders (void) 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; } @@ -569,9 +567,9 @@ print_folders (void) 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, - (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) { @@ -582,7 +580,7 @@ print_folders (void) if (fi[i].others) printf (";%*s (others)", curprinted ? 0 : curmsgdigits + 6, ""); - printf (".\n"); + puts("."); } } @@ -591,17 +589,17 @@ print_folders (void) */ 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); } /* - * Set the current message and sychronize sequences + * Set the current message and synchronize sequences */ static int @@ -612,7 +610,7 @@ sfold (struct msgs *mp, char *msg) 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 */ @@ -633,7 +631,7 @@ readonly_folders (void) { int atrlen; char atrcur[BUFSIZ]; - register struct node *np; + struct node *np; snprintf (atrcur, sizeof(atrcur), "atr-%s-", current); atrlen = strlen (atrcur);