From: Max Kellermann <max@duempel.org>
Date: Thu, 25 Sep 2008 17:19:39 +0000 (+0200)
Subject: screen_utils: don't call wmove() twice
X-Git-Tag: v0.12_alpha1~149
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=f87fea3cd0792de52c0620c329b67db096a7a687;p=ncmpc.git

screen_utils: don't call wmove() twice

The second wmove() is redundant, because the cursor is already there
after waddstr().  This also repairs wide characters in the prompt.
---

diff --git a/src/screen_utils.c b/src/screen_utils.c
index 6029596..5dfc426 100644
--- a/src/screen_utils.c
+++ b/src/screen_utils.c
@@ -28,7 +28,6 @@
 
 #include <stdlib.h>
 #include <unistd.h>
-#include <string.h>
 
 #define FIND_PROMPT  _("Find: ")
 #define RFIND_PROMPT _("Find backward: ")
@@ -46,13 +45,11 @@ int
 screen_getch(WINDOW *w, const char *prompt)
 {
 	int key = -1;
-	int prompt_len = strlen(prompt);
 
 	colors_use(w, COLOR_STATUS_ALERT);
 	wclear(w);
 	wmove(w, 0, 0);
 	waddstr(w, prompt);
-	wmove(w, 0, prompt_len);
 
 	echo();
 	curs_set(1);