X-Git-Url: http://git.marmaro.de/?a=blobdiff_plain;f=uip%2Frmf.c;h=f2460042503dc320953400f8cca287447f7525fd;hb=43c31a90ba57b93a5504c39a28b9ef55a9b6e801;hp=096a9bf279b73bd0711aac9f909cb8b6dded00c1;hpb=30292c537fe4eab052f303f1d3179c0e2fd037d8;p=mmh

diff --git a/uip/rmf.c b/uip/rmf.c
index 096a9bf..f246004 100644
--- a/uip/rmf.c
+++ b/uip/rmf.c
@@ -7,12 +7,16 @@
 */
 
 #include <h/mh.h>
+#include <unistd.h>
+#include <dirent.h>
+#include <locale.h>
+#include <sysexits.h>
 
 static struct swit switches[] = {
 #define INTRSW  0
 	{ "interactive", 0 },
 #define NINTRSW  1
-	{ "nointeractive", 0 },
+	{ "nointeractive", 2 },
 #define VERSIONSW  2
 	{ "Version", 0 },
 #define HELPSW  3
@@ -34,9 +38,7 @@ main(int argc, char **argv)
 	char *cp, *folder = NULL, newfolder[BUFSIZ];
 	char buf[BUFSIZ], **argp, **arguments;
 
-#ifdef LOCALE
 	setlocale(LC_ALL, "");
-#endif
 	invo_name = mhbasename(argv[0]);
 
 	/* read user profile/context */
@@ -50,17 +52,17 @@ main(int argc, char **argv)
 			switch (smatch(++cp, switches)) {
 			case AMBIGSW:
 				ambigsw(cp, switches);
-				done(1);
+				exit(EX_USAGE);
 			case UNKWNSW:
-				adios(NULL, "-%s unknown", cp);
+				adios(EX_USAGE, NULL, "-%s unknown", cp);
 
 			case HELPSW:
 				snprintf(buf, sizeof(buf), "%s [+folder] [switches]", invo_name);
 				print_help(buf, switches, 1);
-				done(1);
+				exit(argc == 2 ? EX_OK : EX_USAGE);
 			case VERSIONSW:
 				print_version(invo_name);
-				done(1);
+				exit(argc == 2 ? EX_OK : EX_USAGE);
 
 			case INTRSW:
 				interactive = 1;
@@ -72,11 +74,11 @@ main(int argc, char **argv)
 		}
 		if (*cp == '+' || *cp == '@') {
 			if (folder)
-				adios(NULL, "only one folder at a time!");
+				adios(EX_USAGE, NULL, "only one folder at a time!");
 			else
 				folder = getcpy(expandfol(cp));
 		} else {
-			adios(NULL, "usage: %s [+folder] [switches]",
+			adios(EX_USAGE, NULL, "usage: %s [+folder] [switches]",
 					invo_name);
 		}
 	}
@@ -86,7 +88,7 @@ main(int argc, char **argv)
 		defolder++;
 	}
 	if (strcmp(toabsdir(folder), pwd()) == 0)
-		adios(NULL, "You can't remove the current working directory");
+		adios(EX_USAGE, NULL, "You can't remove the current working directory");
 
 	if (interactive == -1)
 		interactive = defolder;
@@ -107,7 +109,7 @@ main(int argc, char **argv)
 	if (interactive) {
 		cp = concat("Remove folder \"", folder, "\"? ", NULL);
 		if (!getanswer(cp))
-			done(0);
+			exit(EX_OK);
 		free(cp);
 	}
 
@@ -120,14 +122,13 @@ main(int argc, char **argv)
 		}
 	}
 	context_save();  /* save the context file */
-	done(0);
-	return 1;
+	return 0;
 }
 
 static int
 rmf(char *folder)
 {
-	int i, j, others;
+	int i, others;
 	register char *maildir;
 	char cur[BUFSIZ];
 	register struct dirent *dp;
@@ -152,7 +153,7 @@ rmf(char *folder)
 	}
 
 	if ((dd = opendir(".")) == NULL)
-		adios(NULL, "unable to read folder +%s", folder);
+		adios(EX_IOERR, NULL, "unable to read folder +%s", folder);
 	others = 0;
 
 	/*
@@ -161,7 +162,6 @@ rmf(char *folder)
 
 	ext_hook("del-hook", maildir, NULL);
 
-	j = strlen(backup_prefix);
 	while ((dp = readdir(dd))) {
 		switch (dp->d_name[0]) {
 		case '.':
@@ -175,8 +175,6 @@ rmf(char *folder)
 		default:
 			if (m_atoi(dp->d_name))
 				break;
-			if (strncmp(dp->d_name, backup_prefix, j) == 0)
-				break;
 
 			admonish(NULL, "file \"%s/%s\" not deleted",
 					folder, dp->d_name);