X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=uip%2Fwhatnow.c;h=08e3be84738f005fb22716e0e41aaf22f035b150;hp=efdf02d4b7176195df17488f47f8398e70205fbd;hb=75fca31a5b9d5c1a99c74ab14c94438d8852fba9;hpb=a8a894a102eb7f31662d49355312d6e64e081664

diff --git a/uip/whatnow.c b/uip/whatnow.c
index efdf02d..08e3be8 100644
--- a/uip/whatnow.c
+++ b/uip/whatnow.c
@@ -45,13 +45,11 @@
 static struct swit whatnowswitches[] = {
 #define EDITRSW  0
 	{ "editor editor", 0 },
-#define NEDITSW  1
-	{ "noedit", 0 },
-#define PRMPTSW  2
+#define PRMPTSW  1
 	{ "prompt string", 4 },
-#define VERSIONSW  3
+#define VERSIONSW  2
 	{ "version", 0 },
-#define HELPSW  4
+#define HELPSW  3
 	{ "help", 0 },
 	{ NULL, 0 }
 };
@@ -95,7 +93,7 @@ static char *myprompt = "\nWhat now? ";
 ** static prototypes
 */
 static int editfile(char **, char **, char *, int, struct msgs *,
-	char *, char *, int);
+	char *, char *);
 static int sendfile(char **, char *);
 static int refile(char **, char *);
 static int removefile(char *);
@@ -113,7 +111,7 @@ static int copyf(char *, char *);
 int
 main(int argc, char **argv)
 {
-	int nedit = 0, use = 0;
+	int use = 0;
 	char *cp;
 	char *ed = NULL, *drft = NULL, *msgnam = NULL;
 	char buf[BUFSIZ], prompt[BUFSIZ];
@@ -168,10 +166,6 @@ main(int argc, char **argv)
 				if (!(ed = *argp++) || *ed == '-')
 					adios(NULL, "missing argument to %s",
 							argp[-2]);
-				nedit = 0;
-				continue;
-			case NEDITSW:
-				nedit++;
 				continue;
 
 			case PRMPTSW:
@@ -196,15 +190,16 @@ main(int argc, char **argv)
 	if ((cp = getenv("mhuse")) && *cp)
 		use = atoi(cp);
 
-	if (!ed && (!(ed = getenv("mheditor")) || !*ed)) {
-		ed = NULL;
-		nedit++;
+	if (!ed && !(ed = getenv("mheditor"))) {
+		ed = "";  /* Don't initially edit the draft */
 	}
 
-	/* start editing the draft, unless -noedit was given */
-	if (!nedit && editfile(&ed, NULL, drft, use, NULL, msgnam, NULL, 1)
-			< 0)
-		done(1);
+	/* start editing the draft, unless editor is the empty string */
+	if (*ed) {
+		if (editfile(&ed, NULL, drft, use, NULL, msgnam, NULL) <0) {
+			done(1);
+		}
+	}
 
 	snprintf(prompt, sizeof(prompt), myprompt, invo_name);
 	for (;;) {
@@ -229,7 +224,7 @@ main(int argc, char **argv)
 			if (*++argp)
 				ed = *argp++;
 			if (editfile(&ed, argp, drft, NOUSE, NULL,
-					msgnam, NULL, 1) == NOTOK)
+					msgnam, NULL) == NOTOK)
 				done(1);
 			break;
 
@@ -574,13 +569,12 @@ popen_in_dir(const char *dir, const char *cmd, const char *type)
 ** EDIT
 */
 
-static int  reedit = 0;  /* have we been here before? */
 static char *edsave = NULL;  /* the editor we used previously */
 
 
 static int
 editfile(char **ed, char **arg, char *file, int use, struct msgs *mp,
-	char *altmsg, char *cwd, int save_editor)
+	char *altmsg, char *cwd)
 {
 	int pid, status, vecp;
 	char altpath[BUFSIZ], linkpath[BUFSIZ];
@@ -591,21 +585,22 @@ editfile(char **ed, char **arg, char *file, int use, struct msgs *mp,
 	int slinked = 0;
 #endif /* HAVE_LSTAT */
 
-	/* Was there a previous edit session? */
-	if (reedit) {
-		if (!*ed) {  /* no explicit editor */
-			*ed = edsave;  /* so use the previous one */
+	if (!*ed || !**ed) {
+		/* We have no explicit editor. */
+		if (edsave) {
+			/* Use the previous editor ... */
+			*ed = edsave;
 			if (!(cp = mhbasename(*ed)))
 				cp = *ed;
 
-			/* unless we've specified it via "editor-next" */
+			/* but prefer one specified via "editor-next" */
 			cp = concat(cp, "-next", NULL);
 			if ((cp = context_find(cp)))
 				*ed = cp;
+		} else {
+			/* set initial editor */
+			*ed = defaulteditor;
 		}
-	} else if (!*ed) {
-		/* set initial editor */
-		*ed = defaulteditor;
 	}
 
 	if (altmsg) {
@@ -666,18 +661,17 @@ editfile(char **ed, char **arg, char *file, int use, struct msgs *mp,
 	default:
 		if ((status = pidwait(pid, NOTOK))) {
 			if (((status & 0xff00) != 0xff00) &&
-					(!reedit || (status & 0x00ff))) {
+					(!edsave || (status & 0x00ff))) {
 				if (!use && (status & 0xff00) && (rename(file, cp = m_backup(file)) != NOTOK)) {
 					advise(NULL, "problems with edit--draft left in %s", cp);
 				} else {
 					advise(NULL, "problems with edit--%s preserved", file);
 				}
 			}
-			status = -2;  /* maybe "reedit ? -2 : -1"? */
+			status = -2;  /* maybe "edsave ? -2 : -1"? */
 			break;
 		}
 
-		reedit++;
 #ifdef HAVE_LSTAT
 		if (altmsg && mp && !is_readonly(mp) && (slinked ?
 				lstat (linkpath, &st) != NOTOK &&
@@ -698,9 +692,8 @@ editfile(char **ed, char **arg, char *file, int use, struct msgs *mp,
 #endif /* HAVE_LSTAT */
 	}
 
-	/* normally, we remember which editor we used */
-	if (save_editor)
-		edsave = getcpy(*ed);
+	/* remember which editor we used */
+	edsave = getcpy(*ed);
 
 	*ed = NULL;
 	if (altmsg)