X-Git-Url: http://git.marmaro.de/?a=blobdiff_plain;f=uip%2Fwhatnow.c;h=118afe8554a7a1e00d635fa9a1f2d9cc2633973e;hb=e6705626cc95da96d367bc147ed0419a56f7c37a;hp=e3a411adfd95d2d967142a0f9e5d2db888c2510d;hpb=30292c537fe4eab052f303f1d3179c0e2fd037d8;p=mmh

diff --git a/uip/whatnow.c b/uip/whatnow.c
index e3a411a..118afe8 100644
--- a/uip/whatnow.c
+++ b/uip/whatnow.c
@@ -31,9 +31,8 @@
 **        alist                   This option lists the attachments on the
 **                                draft.
 **
-**        detach files            This option removes attachments from the
-**        detach -n numbers       draft.  This can be done by file name or
-**                                by attachment number.
+**        detach numbers          This option removes attachments by
+**                                attachment number from the draft.
 */
 
 #include <h/mh.h>
@@ -42,11 +41,11 @@
 #include <h/mime.h>
 #include <h/utils.h>
 
-static struct swit whatnowswitches[] = {
+static struct swit switches[] = {
 #define EDITRSW  0
 	{ "editor editor", 0 },
 #define PRMPTSW  1
-	{ "prompt string", 4 },
+	{ "prompt string", 0 },
 #define VERSIONSW  2
 	{ "Version", 0 },
 #define HELPSW  3
@@ -81,7 +80,7 @@ static struct swit aleqs[] = {
 #define ATTACHCMDSW  10
 	{ "attach", 0 },
 #define DETACHCMDSW  11
-	{ "detach [-n]", 0 },
+	{ "detach", 0 },
 #define ALISTCMDSW  12
 	{ "alist", 0 },
 	{ NULL, 0 }
@@ -115,12 +114,8 @@ main(int argc, char **argv)
 	char file[MAXPATHLEN + 1];  /* file name buffer */
 	char shell[MAXPATHLEN + 1];  /* shell response buffer */
 	FILE *f;  /* read pointer for bgnd proc */
-	int n;  /* set on -n to detach command */
 
-#ifdef LOCALE
 	setlocale(LC_ALL, "");
-#endif
-
 	invo_name = mhbasename(argv[0]);
 
 	/* read user profile/context */
@@ -139,9 +134,9 @@ main(int argc, char **argv)
 
 	while ((cp = *argp++)) {
 		if (*cp == '-') {
-			switch (smatch(++cp, whatnowswitches)) {
+			switch (smatch(++cp, switches)) {
 			case AMBIGSW:
-				ambigsw(cp, whatnowswitches);
+				ambigsw(cp, switches);
 				done(1);
 			case UNKWNSW:
 				adios(NULL, "-%s unknown", cp);
@@ -150,7 +145,7 @@ main(int argc, char **argv)
 				snprintf(buf, sizeof(buf),
 						"%s [switches] [file]",
 						invo_name);
-				print_help(buf, whatnowswitches, 1);
+				print_help(buf, switches, 1);
 				done(1);
 			case VERSIONSW:
 				print_version(invo_name);
@@ -302,8 +297,7 @@ main(int argc, char **argv)
 			** List attachments on current draft.
 			*/
 			snprintf(buf, sizeof buf, "anno -list -comp '%s' "
-					"-number all -text IGNORE '%s'",
-					attach_hdr, drft);
+					"-number '%s'", attach_hdr, drft);
 			if (system(buf) != 0) {
 				advise(NULL, "Could not list attachment headers.");
 			}
@@ -370,86 +364,39 @@ main(int argc, char **argv)
 			/*
 			** Detach files from current draft.
 			**
-			** Scan the arguments for a -n.  Mixed file
-			** 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++) {
-				if (strcmp(*arguments, "-n") == 0) {
-						n = 1;
-						break;
-				}
-			}
-
-			/*
-			** A -n was found so interpret the arguments as
+			** Interpret the arguments as
 			** attachment numbers.  Decrement any remaining
 			** argument number that is greater than the one
 			** just processed after processing each one so
 			** that the numbering stays correct.
 			*/
-			if (n == 1) {
-				for (arguments=argp+1; *arguments;
-						arguments++) {
-					if (strcmp(*arguments, "-n") == 0)
-						continue;
-
-					if (**arguments != '\0') {
-						char buf[BUFSIZ];
-
-						n = atoi(*arguments);
-						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 (arguments=argp+1; *arguments; arguments++) {
+				char buf[BUFSIZ];
+				int n;
 
-						for (argp=arguments+1; *argp;
-								argp++) {
-							if (atoi(*argp) > n) {
-								if (atoi(*argp) == 1)
-									*argp = "";
-								else
-									sprintf(*argp, "%d", atoi(*argp) - 1);
-							}
-						}
-					}
+				if (**arguments == '\0') {
+					continue;
 				}
-				break;
-			}
-			/* else */
 
-			/*
-			** The arguments are interpreted as file names.
-			** Run them through the user's shell for wildcard
-			** expansion and other goodies.  Do this from
-			** the current working directory if the argument
-			** is not an absolute path name (does not begin
-			** with a /).
-			**
-			** We feed all the file names to the shell at
-			** once, otherwise you can't provide a file name
-			** with a space in it.
-			*/
-			writelscmd(buf, sizeof(buf), argp);
-			if ((f = popen_in_dir(cwd, buf, "r"))) {
-				while (fgets(shell, sizeof (shell), f)) {
-					*(strchr(shell, '\n')) = '\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.");
+				n = atoi(*arguments);
+				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; argp++) {
+					if (atoi(*argp) > n) {
+						if (atoi(*argp) == 1) {
+							*argp = "";
+						} else {
+							sprintf(*argp, "%d", atoi(*argp) - 1);
+						}
 					}
 				}
-				pclose(f);
-			} else {
-				advise("popen", "could not get file from shell");
 			}
-
 			break;
 
 		default: