Fix multiple popup windows

The window was not deleted if an "already in use"-key was detected,
and a new one was created as the loop was reentered.

Create/delete of the popup are moved outside the loop.
A redrawwin() call is needed to have the window displayed again.

Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>

1 files changed, 7 insertions, 11 deletions

diff --git a/src/custom.c b/src/custom.c
index 3da9da4..52bdd5b 100644
--- a/src/custom.c
+++ b/src/custom.c
@@ -988,21 +988,16 @@ void custom_keys_config(void)
 		case KEY_ADD_ITEM:
 #define WINROW 10
 #define WINCOL 50
+			grabwin = popup(WINROW, WINCOL,
+					(row - WINROW) / 2, (col - WINCOL) / 2,
+					_("Press the key you want to assign to:"),
+					keys_get_label(selrow), 0);
 			for (;;) {
-				grabwin =
-				    popup(WINROW, WINCOL,
-					  (row - WINROW) / 2,
-					  (col - WINCOL) / 2,
-					  _("Press the key you want to assign to:"),
-					  keys_get_label(selrow), 0);
-
 				ch = keys_wgetch(grabwin);
 				enum key action = keys_get_action(ch);
 				/* Is the key already used by this action? */
-				if (action == selrow) {
-					delwin(grabwin);
+				if (action == selrow)
 					break;
-				}
 				/* Is the key used by another action? */
 				if (keys_assign_binding(ch, selrow)) {
 					char *keystr = keys_int2str(ch);
@@ -1018,13 +1013,14 @@ void custom_keys_config(void)
 								selelm,
 								LINESPERKEY);
 					wins_scrollwin_display(&kwin);
+					wins_redrawwin(grabwin);
 					continue;
 				}
 				nbrowelm++;
 				selelm = nbrowelm - 1;
-				delwin(grabwin);
 				break;
 			}
+			delwin(grabwin);
 #undef WINROW
 #undef WINCOL
 			break;