projects
/
mmh
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
remove dublicated code in uip/spost
[mmh]
/
uip
/
mark.c
diff --git
a/uip/mark.c
b/uip/mark.c
index
d4a3729
..
a6ac480
100644
(file)
--- a/
uip/mark.c
+++ b/
uip/mark.c
@@
-10,6
+10,9
@@
#include <h/mh.h>
#include <h/utils.h>
#include <h/mh.h>
#include <h/utils.h>
+#include <unistd.h>
+#include <locale.h>
+#include <sysexits.h>
static struct swit switches[] = {
#define ADDSW 0
static struct swit switches[] = {
#define ADDSW 0
@@
-73,17
+76,17
@@
main(int argc, char **argv)
switch (smatch(++cp, switches)) {
case AMBIGSW:
ambigsw(cp, switches);
switch (smatch(++cp, switches)) {
case AMBIGSW:
ambigsw(cp, switches);
- done(1);
+ exit(EX_USAGE);
case UNKWNSW:
case UNKWNSW:
- adios(NULL, "-%s unknown\n", cp);
+ adios(EX_USAGE, NULL, "-%s unknown\n", cp);
case HELPSW:
snprintf(buf, sizeof(buf), "%s [+folder] [msgs] [switches]", invo_name);
print_help(buf, switches, 1);
case HELPSW:
snprintf(buf, sizeof(buf), "%s [+folder] [msgs] [switches]", invo_name);
print_help(buf, switches, 1);
- done(1);
+ exit(argc == 2 ? EX_OK : EX_USAGE);
case VERSIONSW:
print_version(invo_name);
case VERSIONSW:
print_version(invo_name);
- done(1);
+ exit(argc == 2 ? EX_OK : EX_USAGE);
case ADDSW:
addsw++;
case ADDSW:
addsw++;
@@
-99,13
+102,15
@@
main(int argc, char **argv)
continue;
case SEQSW:
continue;
case SEQSW:
- if (!(cp = *argp++) || *cp == '-')
- adios(NULL, "missing argument to %s",
+ if (!(cp = *argp++) || *cp == '-') {
+ adios(EX_USAGE, NULL, "missing argument to %s",
argp[-2]);
argp[-2]);
+ }
/* check if too many sequences specified */
/* check if too many sequences specified */
- if (seqp >= NUMATTRS)
- adios(NULL, "too many sequences (more than %d) specified", NUMATTRS);
+ if (seqp >= NUMATTRS) {
+ adios(EX_USAGE, NULL, "too many sequences (more than %d) specified", NUMATTRS);
+ }
seqs[seqp++] = cp;
continue;
seqs[seqp++] = cp;
continue;
@@
-129,12
+134,14
@@
main(int argc, char **argv)
}
}
if (*cp == '+' || *cp == '@') {
}
}
if (*cp == '+' || *cp == '@') {
- if (folder)
- adios(NULL, "only one folder at a time!");
- else
+ if (folder) {
+ adios(EX_USAGE, NULL, "only one folder at a time!");
+ } else {
folder = getcpy(expandfol(cp));
folder = getcpy(expandfol(cp));
- } else
+ }
+ } else {
app_msgarg(&msgs, cp);
app_msgarg(&msgs, cp);
+ }
}
/*
}
/*
@@
-155,51
+162,62
@@
main(int argc, char **argv)
folder = getcurfol();
maildir = toabsdir(folder);
folder = getcurfol();
maildir = toabsdir(folder);
- if (chdir(maildir) == NOTOK)
- adios(maildir, "unable to change directory to");
+ if (chdir(maildir) == NOTOK) {
+ adios(EX_OSERR, maildir, "unable to change directory to");
+ }
/* read folder and create message structure */
/* read folder and create message structure */
- if (!(mp = folder_read(folder)))
- adios(NULL, "unable to read folder %s", folder);
+ if (!(mp = folder_read(folder))) {
+ adios(EX_IOERR, NULL, "unable to read folder %s", folder);
+ }
/* print some general debugging info */
if (debugsw)
print_debug(mp);
/* check for empty folder */
/* print some general debugging info */
if (debugsw)
print_debug(mp);
/* check for empty folder */
- if (mp->nummsg == 0)
- adios(NULL, "no messages in %s", folder);
+ if (mp->nummsg == 0) {
+ adios(EX_DATAERR, NULL, "no messages in %s", folder);
+ }
/* parse all the message ranges/sequences and set SELECTED */
/* parse all the message ranges/sequences and set SELECTED */
- for (msgnum = 0; msgnum < msgs.size; msgnum++)
- if (!m_convert(mp, msgs.msgs[msgnum]))
- done(1);
+ for (msgnum = 0; msgnum < msgs.size; msgnum++) {
+ if (!m_convert(mp, msgs.msgs[msgnum])) {
+ exit(EX_USAGE);
+ }
+ }
- if (publicsw == 1 && is_readonly(mp))
- adios(NULL, "folder %s is read-only, so -public not allowed",
+ if (publicsw == 1 && is_readonly(mp)) {
+ adios(EX_NOPERM, NULL, "folder %s is read-only, so -public not allowed",
folder);
folder);
+ }
/*
** Make sure at least one sequence has been
** specified if we are adding or deleting.
*/
/*
** Make sure at least one sequence has been
** specified if we are adding or deleting.
*/
- if (seqp == 0 && (addsw || deletesw))
- adios(NULL, "-%s requires at least one -sequence argument",
+ if (seqp == 0 && (addsw || deletesw)) {
+ adios(EX_USAGE, NULL, "-%s requires at least one -sequence argument",
addsw ? "add" : "delete");
addsw ? "add" : "delete");
+ }
seqs[seqp] = NULL;
/* Adding messages to sequences */
if (addsw) {
seqs[seqp] = NULL;
/* Adding messages to sequences */
if (addsw) {
- for (seqp = 0; seqs[seqp]; seqp++)
- if (!seq_addsel(mp, seqs[seqp], publicsw, zerosw))
- done(1);
+ for (seqp = 0; seqs[seqp]; seqp++) {
+ if (!seq_addsel(mp, seqs[seqp], publicsw, zerosw)) {
+ exit(EX_SOFTWARE);
+ }
+ }
}
/* Deleting messages from sequences */
if (deletesw) {
}
/* Deleting messages from sequences */
if (deletesw) {
- for (seqp = 0; seqs[seqp]; seqp++)
- if (!seq_delsel(mp, seqs[seqp], publicsw, zerosw))
- done(1);
+ for (seqp = 0; seqs[seqp]; seqp++) {
+ if (!seq_delsel(mp, seqs[seqp], publicsw, zerosw)) {
+ exit(EX_SOFTWARE);
+ }
+ }
}
/* Listing messages in sequences */
}
/* Listing messages in sequences */
@@
-222,8
+240,7
@@
main(int argc, char **argv)
context_replace(curfolder, folder); /* update current folder */
context_save(); /* save the context file */
folder_free(mp); /* free folder/message structure */
context_replace(curfolder, folder); /* update current folder */
context_save(); /* save the context file */
folder_free(mp); /* free folder/message structure */
- done(0);
- return 1;
+ return EX_OK;
}
}