Remove the special-case behavior for fire actions that return

DITEM_FAILURE - formerly they would simply act as an implicit "continue",
but this is wrong.  If you want this behavior, you should now return
with the DITEM_CONTINUE flag set.

Also make the semantics of DITEM_RESTORE quite a bit different - rather
than restore the screen back to pre-menu state, we restore the menu
itself.  This is more correct for a variety of reasons when dealing with
nested menus (whoops!).

7 files changed, 104 insertions, 176 deletions

diff --git a/gnu/lib/libdialog/TESTS/menu1.c b/gnu/lib/libdialog/TESTS/menu1.c
index 5e5a1ad..a06eade 100644
--- a/gnu/lib/libdialog/TESTS/menu1.c
+++ b/gnu/lib/libdialog/TESTS/menu1.c
@@ -13,7 +13,7 @@
  * the author assume any responsibility for damages incurred with
  * its use.
  *
- * $Id: menu1.c,v 1.2 1996/04/07 03:20:52 jkh Exp $
+ * $Id: menu1.c,v 1.3 1996/04/16 12:17:22 jkh Exp $
  */
 
 #include <stdio.h>
@@ -29,10 +29,6 @@ static enum { nowhere, berlin, rome, ny } where;
 static int
 _menu1_berlin_action(dialogMenuItem *self)
 {
-    WINDOW *w;
-    int st = DITEM_FAILURE;
-    
-    w = dupwin(newscr);
     if (where == berlin) {
 	dialog_mesgbox("excuse me?", "But you're already *in* Berlin!", -1, -1);
     }
@@ -40,19 +36,12 @@ _menu1_berlin_action(dialogMenuItem *self)
 	where = berlin;
 	dialog_mesgbox("whoosh!", "Welcome to Berlin!  Have a beer!", -1, -1);
     }
-    touchwin(w);
-    wrefresh(w);
-    delwin(w);
-    return st;
+    return DITEM_SUCCESS | DITEM_RESTORE | DITEM_CONTINUE;
 }
 
 static int
 _menu1_rome_action(dialogMenuItem *self)
 {
-    WINDOW *w;
-    int st = DITEM_FAILURE;
-    
-    w = dupwin(newscr);
     if (where == rome) {
 	dialog_mesgbox("The wine must be getting to you..", "You're already in Rome!", -1, -1);
     }
@@ -60,19 +49,12 @@ _menu1_rome_action(dialogMenuItem *self)
 	where = rome;
 	dialog_mesgbox("whoosh!", "Welcome to Rome!  Have a coffee!", -1, -1);
     }
-    touchwin(w);
-    wrefresh(w);
-    delwin(w);
-    return st;
+    return DITEM_SUCCESS | DITEM_RESTORE | DITEM_CONTINUE;
 }
 
 static int
 _menu1_ny_action(dialogMenuItem *self)
 {
-    WINDOW *w;
-    int st = DITEM_FAILURE;
-    
-    w = dupwin(newscr);
     if (where == ny) {
 	dialog_mesgbox("Say what?", "You're already there!", -1, -1);
     }
@@ -80,10 +62,7 @@ _menu1_ny_action(dialogMenuItem *self)
 	where = ny;
 	dialog_mesgbox("whoosh!", "Welcome to New York!  Now go someplace else!", -1, -1);
     }
-    touchwin(w);
-    wrefresh(w);
-    delwin(w);
-    return st;
+    return DITEM_SUCCESS | DITEM_RESTORE | DITEM_CONTINUE;
 }
 
 /* menu1 - show off the "fire" action hook */
diff --git a/gnu/lib/libdialog/TESTS/menu2.c b/gnu/lib/libdialog/TESTS/menu2.c
index 40f279e..c1602f7 100644
--- a/gnu/lib/libdialog/TESTS/menu2.c
+++ b/gnu/lib/libdialog/TESTS/menu2.c
@@ -13,7 +13,7 @@
  * the author assume any responsibility for damages incurred with
  * its use.
  *
- * $Id: menu2.c,v 1.2 1996/04/07 03:20:54 jkh Exp $
+ * $Id: menu2.c,v 1.3 1996/04/16 12:17:23 jkh Exp $
  */
 
 #include <stdio.h>
@@ -29,61 +29,37 @@ static enum { nowhere, berlin, rome, ny } where;
 static int
 _menu1_berlin_action(dialogMenuItem *self)
 {
-    WINDOW *w;
-    int st = DITEM_FAILURE;
-    
-    w = dupwin(newscr);
-    if (where == berlin) {
+    if (where == berlin)
 	dialog_mesgbox("excuse me?", "But you're already *in* Berlin!", -1, -1);
-    }
     else {
 	where = berlin;
 	dialog_mesgbox("whoosh!", "Welcome to Berlin!  Have a beer!", -1, -1);
     }
-    touchwin(w);
-    wrefresh(w);
-    delwin(w);
-    return st;
+    return DITEM_SUCCESS | DITEM_RESTORE | DITEM_CONTINUE;
 }
 
 static int
 _menu1_rome_action(dialogMenuItem *self)
 {
-    WINDOW *w;
-    int st = DITEM_FAILURE;
-    
-    w = dupwin(newscr);
-    if (where == rome) {
+    if (where == rome)
 	dialog_mesgbox("The wine must be getting to you..", "You're already in Rome!", -1, -1);
-    }
     else {
 	where = rome;
 	dialog_mesgbox("whoosh!", "Welcome to Rome!  Have a coffee!", -1, -1);
     }
-    touchwin(w);
-    wrefresh(w);
-    delwin(w);
-    return st;
+    return DITEM_SUCCESS | DITEM_RESTORE | DITEM_CONTINUE;
 }
 
 static int
 _menu1_ny_action(dialogMenuItem *self)
 {
-    WINDOW *w;
-    int st = DITEM_FAILURE;
-    
-    w = dupwin(newscr);
-    if (where == ny) {
+    if (where == ny)
 	dialog_mesgbox("Say what?", "You're already there!", -1, -1);
-    }
     else {
 	where = ny;
 	dialog_mesgbox("whoosh!", "Welcome to New York!  Now go someplace else!", -1, -1);
     }
-    touchwin(w);
-    wrefresh(w);
-    delwin(w);
-    return st;
+    return DITEM_SUCCESS | DITEM_RESTORE | DITEM_CONTINUE;
 }
 
 /* menu1 - show off the "fire" action hook */
diff --git a/gnu/lib/libdialog/TESTS/menu3.c b/gnu/lib/libdialog/TESTS/menu3.c
index f1acd3c..edc99d3 100644
--- a/gnu/lib/libdialog/TESTS/menu3.c
+++ b/gnu/lib/libdialog/TESTS/menu3.c
@@ -13,7 +13,7 @@
  * the author assume any responsibility for damages incurred with
  * its use.
  *
- * $Id: menu3.c,v 1.2 1996/04/07 03:20:55 jkh Exp $
+ * $Id: menu3.c,v 1.3 1996/04/16 12:17:24 jkh Exp $
  */
 
 #include <stdio.h>
@@ -35,14 +35,8 @@ stop(dialogMenuItem *self)
 static int
 maybe(dialogMenuItem *self)
 {
-    WINDOW *w;
-    
-    w = dupwin(newscr);
     dialog_mesgbox("!", "I said don't rush me!  I'm THINKING!", -1, -1);
-    touchwin(w);
-    wrefresh(w);
-    delwin(w);
-    return DITEM_FAILURE;
+    return DITEM_SUCCESS | DITEM_RESTORE | DITEM_CONTINUE;
 }
 
 /* Dummy menu just to show of the ability */
@@ -60,7 +54,7 @@ preinsure(dialogMenuItem *self, int is_selected)
 	
 	/* This has to be here first if you want to see selection traverse properly in the invoking menu */
 	refresh();
-	
+
 	w = dupwin(newscr);
 	DialogX = 1;
 	DialogY = 13;
diff --git a/gnu/lib/libdialog/checklist.c b/gnu/lib/libdialog/checklist.c
index a43b23c..8e8bdbc 100644
--- a/gnu/lib/libdialog/checklist.c
+++ b/gnu/lib/libdialog/checklist.c
@@ -44,7 +44,7 @@ dialog_checklist(unsigned char *title, unsigned char *prompt, int height, int wi
 	l, k, scroll = 0, max_choice, *status;
     int redraw_menu = FALSE;
     char okButton, cancelButton;
-    WINDOW *dialog, *list, *save;
+    WINDOW *dialog, *list;
     unsigned char **items;
     dialogMenuItem *ditems;
     
@@ -109,14 +109,12 @@ dialog_checklist(unsigned char *title, unsigned char *prompt, int height, int wi
     y = (LINES - height)/2;
 
 draw:
-    save = dupwin(newscr);
 #ifdef HAVE_NCURSES
     if (use_shadow)
 	draw_shadow(stdscr, y, x, height, width);
 #endif
     dialog = newwin(height, width, y, x);
     if (dialog == NULL) {
-	delwin(save);
 	endwin();
 	fprintf(stderr, "\nnewwin(%d,%d,%d,%d) failed, maybe wrong dims\n", height,width, y, x);
 	return -1;
@@ -155,7 +153,6 @@ draw:
     list = subwin(dialog, list_height, list_width, y + box_y + 1, x + box_x + 1);
     if (list == NULL) {
 	delwin(dialog);
-	delwin(save);
 	endwin();
 	fprintf(stderr, "\nsubwin(dialog,%d,%d,%d,%d) failed, maybe wrong dims\n", list_height, list_width,
 		y + box_y + 1, x + box_x + 1);
@@ -204,19 +201,17 @@ draw:
 	/* Shortcut to OK? */
 	if (toupper(key) == okButton) {
 	    if (ditems && result && ditems[OK_BUTTON].fire) {
-		int st = ditems[OK_BUTTON].fire(&ditems[OK_BUTTON]);
-		
-		if (DITEM_STATUS(st) == DITEM_FAILURE) {
-		    wrefresh(dialog);
-		    continue;
-		}
-		else {
-		    delwin(dialog);
-		    if (st & DITEM_RESTORE) {
-			touchwin(save);
-			wrefresh(save);
-		    }
+		int st;
+		WINDOW *save;
+
+		save = dupwin(newscr);
+		st = ditems[OK_BUTTON].fire(&ditems[OK_BUTTON]);
+		delwin(dialog);
+		if (st & DITEM_RESTORE) {
+		    touchwin(save);
+		    wrefresh(save);
 		}
+		delwin(save);
 	    }
 	    else {
 		delwin(dialog);
@@ -228,27 +223,23 @@ draw:
 		    }
 		}
 	    }
-	    delwin(save);
 	    return 0;
 	}
 	/* Shortcut to cancel? */
 	else if (toupper(key) == cancelButton) {
 	    if (ditems && result && ditems[CANCEL_BUTTON].fire) {
 		int st;
+		WINDOW *save;
 
+		save = dupwin(newscr);
 		st = ditems[CANCEL_BUTTON].fire(&ditems[CANCEL_BUTTON]);
-
-		if (DITEM_STATUS(st) == DITEM_FAILURE) {
-		    wrefresh(dialog);
-		    continue;
-		}
-		else if (st & DITEM_RESTORE) {
+		if (st & DITEM_RESTORE) {
 		    touchwin(save);
 		    wrefresh(save);
 		}
+		delwin(save);
 	    }
 	    delwin(dialog);
-	    delwin(save);
 	    return 1;
 	}
 	
@@ -315,31 +306,34 @@ draw:
 		char lbra = 0, rbra = 0, mark = 0;
 		
 		if (ditems) {
-		    if (ditems[scroll+choice].fire) {
-			int st = ditems[scroll+choice].fire(&ditems[scroll+choice]);
+		    if (ditems[scroll + choice].fire) {
+			int st;
+			WINDOW *save;
 
+			save = dupwin(newscr);
+			st = ditems[scroll + choice].fire(&ditems[scroll + choice]);
 			if (st & DITEM_REDRAW) {
-			    for (i = 0; i < max_choice; i++)
-				print_item(list, items[(scroll+i)*3], items[(scroll+i)*3 + 1], status[scroll+i], i, i == choice,
-					   DREF(ditems, scroll + i));
+			    for (i = 0; i < max_choice; i++) {
+				status[i] = ditems[i].checked ? ditems[i].checked(&ditems[i]) : FALSE;
+				print_item(list, items[(scroll + i) * 3], items[(scroll + i) * 3 + 1],
+					   status[scroll + i], i, i == choice, DREF(ditems, scroll + i));
+			    }
 			    wnoutrefresh(list);
-			    print_arrows(dialog, scroll, list_height, item_no, box_x, box_y, check_x + 4, cur_x, cur_y);
-			    wrefresh(dialog);
-			}
-			if (DITEM_STATUS(st) == DITEM_FAILURE) {
+			    print_arrows(dialog, scroll, list_height, item_no, box_x, box_y, check_x + 4,
+					 cur_x, cur_y);
 			    wrefresh(dialog);
-			    continue;
 			}
 			else if (st & DITEM_RESTORE) {
 			    touchwin(save);
 			    wrefresh(save);
 			}
-			if (st & DITEM_RECREATE) {
+			else if (st & DITEM_RECREATE) {
 			    delwin(save);
 			    delwin(dialog);
 			    goto draw;
 			}
-			else if (st & DITEM_LEAVE_MENU) {
+			delwin(save);
+			if (st & DITEM_LEAVE_MENU) {
 			    /* Allow a fire action to take us out of the menu */
 			    key = ESC;
 			    break;
@@ -475,7 +469,6 @@ draw:
 		}
 	    }
 	    delwin(dialog);
-	    delwin(save);
 	    return button;
 	    break;
 	    
@@ -501,7 +494,6 @@ draw:
 	}
     }
     delwin(dialog);
-    delwin(save);
     return -1;    /* ESC pressed */
 }
 /* End of dialog_checklist() */
diff --git a/gnu/lib/libdialog/dialog.h b/gnu/lib/libdialog/dialog.h
index 9619adf..37f05af 100644
--- a/gnu/lib/libdialog/dialog.h
+++ b/gnu/lib/libdialog/dialog.h
@@ -48,6 +48,7 @@
 #define DITEM_REDRAW		(1 << 17)
 #define DITEM_RECREATE		(1 << 18)
 #define DITEM_RESTORE		(1 << 19)
+#define DITEM_CONTINUE		(1 << 20)
 
 
 /* negative offsets for buttons in item lists, if specified */
diff --git a/gnu/lib/libdialog/menubox.c b/gnu/lib/libdialog/menubox.c
index 9308aaa..4823c2c 100644
--- a/gnu/lib/libdialog/menubox.c
+++ b/gnu/lib/libdialog/menubox.c
@@ -42,7 +42,7 @@ dialog_menu(unsigned char *title, unsigned char *prompt, int height, int width,
     int i, j, x, y, cur_x, cur_y, box_x, box_y, key = 0, button = 0, choice = 0,
 	l, k, scroll = 0, max_choice, redraw_menu = FALSE;
     char okButton, cancelButton;
-    WINDOW *dialog, *menu, *save;
+    WINDOW *dialog, *menu;
     unsigned char **items;
     dialogMenuItem *ditems;
     
@@ -100,14 +100,12 @@ dialog_menu(unsigned char *title, unsigned char *prompt, int height, int width,
     y = DialogY ? DialogY : (LINES - height)/2;
     
 draw:
-    save = dupwin(newscr);
 #ifdef HAVE_NCURSES
     if (use_shadow)
 	draw_shadow(stdscr, y, x, height, width);
 #endif
     dialog = newwin(height, width, y, x);
     if (dialog == NULL) {
-	delwin(save);
 	endwin();
 	fprintf(stderr, "\nnewwin(%d,%d,%d,%d) failed, maybe wrong dims\n", height,width,y,x);
 	return -1;
@@ -145,7 +143,6 @@ draw:
     /* create new window for the menu */
     menu = subwin(dialog, menu_height, menu_width, y + box_y + 1, x + box_x + 1);
     if (menu == NULL) {
-	delwin(save);
 	endwin();
 	fprintf(stderr, "\nsubwin(dialog,%d,%d,%d,%d) failed, maybe wrong dims\n", menu_height,menu_width,y+box_y+1,x+box_x+1);
 	return -1;
@@ -193,41 +190,36 @@ draw:
 	if (toupper(key) == okButton) {
 	    if (ditems && result && ditems[OK_BUTTON].fire) {
 		int status;
-		
+		WINDOW *save;
+
+		save = dupwin(newscr);
 		status = ditems[OK_BUTTON].fire(&ditems[OK_BUTTON]);
-		if (DITEM_STATUS(status) == DITEM_FAILURE)
-		    continue;
-		else {
-		    delwin(dialog);
-		    if (status & DITEM_RESTORE) {
-			touchwin(save);
-			wrefresh(save);
-		    }
+		if (status & DITEM_RESTORE) {
+		    touchwin(save);
+		    wrefresh(save);
 		}
-		
+		delwin(save);
 	    }
-	    else {
-		delwin(dialog);
+	    else
 		strcpy(result, items[(scroll + choice) * 2]);
-	    }
-	    delwin(save);
+	    delwin(dialog);
 	    return 0;
 	}
 	/* Shortcut to cancel? */
 	else if (toupper(key) == cancelButton) {
 	    if (ditems && result && ditems[CANCEL_BUTTON].fire) {
 		int status;
-		
+		WINDOW *save;
+
+		save = dupwin(newscr);
 		status = ditems[CANCEL_BUTTON].fire(&ditems[CANCEL_BUTTON]);
-		if (DITEM_STATUS(status) == DITEM_FAILURE)
-		    continue;
-		else if (status & DITEM_RESTORE) {
+		if (status & DITEM_RESTORE) {
 		    touchwin(save);
 		    wrefresh(save);
 		}
+		delwin(save);
 	    }
 	    delwin(dialog);
-	    delwin(save);
 	    return 1;
 	}
 	
@@ -384,25 +376,27 @@ draw:
 		   to accept some hints as to what to do in the aftermath. */
 		if (ditems && ditems[scroll + choice].fire) {
 		    int status;
-		    
+		    WINDOW *save;
+
+		    save = dupwin(newscr);
 		    status = ditems[scroll + choice].fire(&ditems[scroll + choice]);
-		    if (DITEM_STATUS(status) == DITEM_FAILURE)
-			continue;
-		    else if (status & DITEM_RESTORE) {
+		    if (status & DITEM_RESTORE) {
 			touchwin(save);
 			wrefresh(save);
 		    }
 		    else if (status & DITEM_RECREATE) {
-			delwin(save);
 			delwin(dialog);
+			delwin(save);
 			goto draw;
 		    }
+		    delwin(save);
+		    if (status & DITEM_CONTINUE)
+			continue;
 		}
 		else if (result)
 		    strcpy(result, items[(scroll+choice)*2]);
 	    }
 	    delwin(dialog);
-	    delwin(save);
 	    return button;
 	    
 	case ESC:
@@ -427,7 +421,6 @@ draw:
     }
     
     delwin(dialog);
-    delwin(save);
     return -1;    /* ESC pressed */
 }
 /* End of dialog_menu() */
diff --git a/gnu/lib/libdialog/radiolist.c b/gnu/lib/libdialog/radiolist.c
index fd0c256..e56d8e7 100644
--- a/gnu/lib/libdialog/radiolist.c
+++ b/gnu/lib/libdialog/radiolist.c
@@ -44,7 +44,7 @@ dialog_radiolist(unsigned char *title, unsigned char *prompt, int height, int wi
 	l, k, scroll = 0, max_choice, *status, was_on = 0;
     int redraw_menu = FALSE;
     char okButton, cancelButton;
-    WINDOW *dialog, *list, *save;
+    WINDOW *dialog, *list;
     unsigned char **items;
     dialogMenuItem *ditems;
     
@@ -122,14 +122,12 @@ dialog_radiolist(unsigned char *title, unsigned char *prompt, int height, int wi
     y = DialogY ? DialogY : (LINES - height)/2;
 
 draw:
-    save = dupwin(newscr);
 #ifdef HAVE_NCURSES
     if (use_shadow)
 	draw_shadow(stdscr, y, x, height, width);
 #endif
     dialog = newwin(height, width, y, x);
     if (dialog == NULL) {
-	delwin(save);
 	endwin();
 	fprintf(stderr, "\nnewwin(%d,%d,%d,%d) failed, maybe wrong dims\n", height,width,y,x);
 	return -1;
@@ -167,7 +165,6 @@ draw:
     /* create new window for the list */
     list = subwin(dialog, list_height, list_width, y + box_y + 1, x + box_x + 1);
     if (list == NULL) {
-	delwin(save);
 	delwin(dialog);
 	endwin();
 	fprintf(stderr, "\nsubwin(dialog,%d,%d,%d,%d) failed, maybe wrong dims\n", list_height,list_width,y+box_y+1,x+box_x+1);
@@ -214,20 +211,17 @@ draw:
 	if (toupper(key) == okButton) {
 	    if (ditems && result && ditems[OK_BUTTON].fire) {
 		int st;
+		WINDOW *save;
 
+		save = dupwin(newscr);
 		st = ditems[OK_BUTTON].fire(&ditems[OK_BUTTON]);
-		if (DITEM_STATUS(st) == DITEM_FAILURE)
-		    continue;
-		else {
-		    delwin(dialog);
-		    if (st & DITEM_RESTORE) {
-			touchwin(save);
-			wrefresh(save);
-		    }
+		if (st & DITEM_RESTORE) {
+		    touchwin(save);
+		    wrefresh(save);
 		}
+		delwin(save);
 	    }
 	    else {
-		delwin(dialog);
 		*result = '\0';
 		for (i = 0; i < item_no; i++) {
 		    if (status[i]) {
@@ -236,32 +230,30 @@ draw:
 		    }
 		}
 	    }
-	    delwin(save);
+	    delwin(dialog);
 	    return 0;
 	}
 	/* Shortcut to cancel */
 	else if (toupper(key) == cancelButton) {
 	    if (ditems && result && ditems[CANCEL_BUTTON].fire) {
 		int st;
+		WINDOW *save;
 
+		save = dupwin(newscr);
 		st = ditems[CANCEL_BUTTON].fire(&ditems[CANCEL_BUTTON]);
-		if (DITEM_STATUS(st) == DITEM_FAILURE) {
-		    wrefresh(dialog);
-		    continue;
-		}
-		else if (st & DITEM_RESTORE) {
+		if (st & DITEM_RESTORE) {
 		    touchwin(save);
 		    wrefresh(save);
 		}
+		delwin(save);
 	    }
 	    delwin(dialog);
-	    delwin(save);
 	    return 1;
 	}
 	
 	/* Check if key pressed matches first character of any item tag in list */
 	for (i = 0; i < max_choice; i++)
-	    if (toupper(key) == toupper(items[(scroll+i)*3][0]))
+	    if (toupper(key) == toupper(items[(scroll + i) * 3][0]))
 		break;
 	
 	if (i < max_choice || (key >= '1' && key <= MIN('9', '0' + max_choice)) ||
@@ -293,7 +285,7 @@ draw:
 	    }
 	    else if (key == KEY_DOWN || key == '+') {
 		if (choice == max_choice - 1) {
-		    if (scroll+choice < item_no-1) {
+		    if (scroll + choice < item_no - 1) {
 			/* Scroll list up */
 			getyx(dialog, cur_y, cur_x);    /* Save cursor position */
 			if (list_height > 1) {
@@ -321,31 +313,33 @@ draw:
 		    continue;
 		else if (ditems) {
 		    if (ditems[scroll + choice].fire) {
-			int st = ditems[scroll + choice].fire(&ditems[scroll + choice]);
-			
+			int st;
+			WINDOW *save;
+
+			save = dupwin(newscr);
+			st = ditems[scroll + choice].fire(&ditems[scroll + choice]);
 			if (st & DITEM_REDRAW) {
-			    for (i = 0; i < max_choice; i++)
+			    for (i = 0; i < max_choice; i++) {
+				status[i] = ditems[i].checked ? ditems[i].checked(&ditems[i]) : FALSE;
 				print_item(list, items[(scroll + i) * 3], items[(scroll + i) * 3 + 1],
 					   status[scroll + i], i, i == choice,
 					   DREF(ditems, scroll + i));
+			    }
 			    wnoutrefresh(list);
 			    wmove(dialog, cur_y, cur_x);  /* Restore cursor to previous position */
 			    wrefresh(dialog);
 			}
-			if (DITEM_STATUS(st) == DITEM_FAILURE) {
-			    wrefresh(dialog);
-			    continue;
-			}
 			else if (st & DITEM_RESTORE) {
 			    touchwin(save);
 			    wrefresh(save);
 			}
-			if (st & DITEM_RECREATE) {
+			else if (st & DITEM_RECREATE) {
 			    delwin(save);
 			    delwin(dialog);
 			    goto draw;
 			}
-			else if (st & DITEM_LEAVE_MENU) {
+			delwin(save);
+			if (st & DITEM_LEAVE_MENU) {
 			    /* Allow a fire action to take us out of the menu */
 			    key = ESC;
 			    break;
@@ -372,12 +366,12 @@ draw:
 	    if (i != choice) {
 		/* De-highlight current item */
 		getyx(dialog, cur_y, cur_x);    /* Save cursor position */
-		print_item(list, items[(scroll+choice)*3], items[(scroll+choice)*3 +1], status[scroll+choice], choice, FALSE,
-			   DREF(ditems, scroll + choice));
+		print_item(list, items[(scroll + choice) * 3], items[(scroll + choice) * 3 +1],
+			   status[scroll + choice], choice, FALSE, DREF(ditems, scroll + choice));
 		/* Highlight new item */
 		choice = i;
-		print_item(list, items[(scroll+choice)*3], items[(scroll+choice)*3 + 1], status[scroll+choice], choice, TRUE,
-			   DREF(ditems, scroll + choice));
+		print_item(list, items[(scroll + choice) * 3], items[(scroll + choice) * 3 + 1],
+			   status[scroll + choice], choice, TRUE, DREF(ditems, scroll + choice));
 		wnoutrefresh(list);
 		wmove(dialog, cur_y, cur_x);  /* Restore cursor to previous position */
 		wrefresh(dialog);
@@ -455,12 +449,12 @@ draw:
 	case '\n':
 	    if (!button && result) {
 		if (ditems && ditems[button ? CANCEL_BUTTON : OK_BUTTON].fire) {
-		    int st = ditems[button ? CANCEL_BUTTON : OK_BUTTON].fire(&ditems[button ? CANCEL_BUTTON : OK_BUTTON]);
-		    if (DITEM_STATUS(st) == DITEM_FAILURE) {
-			wrefresh(dialog);
-			continue;
-		    }
-		    else if (st & DITEM_RESTORE) {
+		    int st;
+		    WINDOW *save;
+
+		    save = dupwin(newscr);
+		    st = ditems[button ? CANCEL_BUTTON : OK_BUTTON].fire(&ditems[button ? CANCEL_BUTTON : OK_BUTTON]);
+		    if (st & DITEM_RESTORE) {
 			touchwin(save);
 			wrefresh(save);
 		    }
@@ -469,6 +463,7 @@ draw:
 			delwin(dialog);
 			goto draw;
 		    }
+		    delwin(save);
 		}
 		else {
 		    *result = '\0';
@@ -481,7 +476,6 @@ draw:
 		}
 	    }
 	    delwin(dialog);
-	    delwin(save);
 	    return button;
 	    break;
 	    
@@ -506,7 +500,6 @@ draw:
     }
     
     delwin(dialog);
-    delwin(save);
     free(status);
     return -1;    /* ESC pressed */
 }