Code

screen_utils: don't call wmove() twice
authorMax Kellermann <max@duempel.org>
Thu, 25 Sep 2008 17:19:39 +0000 (19:19 +0200)
committerMax Kellermann <max@duempel.org>
Thu, 25 Sep 2008 17:19:39 +0000 (19:19 +0200)
The second wmove() is redundant, because the cursor is already there
after waddstr().  This also repairs wide characters in the prompt.

src/screen_utils.c

index 6029596bdc47ea43488f93f08c48b912dd86f3f0..5dfc4263031b9479a355d2c3e2af8b1f22bb6f42 100644 (file)
@@ -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);