File 0008-top-tweak-cursor-state-code-to-swat-an-obs... of Package procps

Overview Repositories Revisions Requests Users Attributes Meta

File 0008-top-tweak-cursor-state-code-to-swat-an-obscure-bugle.patch of Package procps (Revision b7bb9355db92dad1ab5a522e07d57590)

Currently displaying revision b7bb9355db92dad1ab5a522e07d57590 , Show latest

From 30e90e426980c3122c668dd2a6b9236d4db6e048 Mon Sep 17 00:00:00 2001
From: Jim Warner <james.warner@comcast.net>
Date: Fri, 5 Jul 2013 00:00:00 -0500
Subject: [PATCH] top: tweak cursor state code to swat an obscure buglet

An obscure bug has been discovered where a 'W' with an
old rcfile, warning against overwrite, would display a
cursor that should normally be hidden. This followed a
user's reply. So some logic was rearranged just a bit.

Signed-off-by: Jim Warner <james.warner@comcast.net>
---
 top/top.c |   11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git top/top.c top/top.c
index 978071c..8978cd1 100644
--- top/top.c
+++ top/top.c
@@ -3104,6 +3104,7 @@ signify_that:
          case '/':
          case 'n':
             insp_find_str(key, &curcol, &curlin);
+            // must re-hide cursor in case a prompt for a string makes it huge
             putp((Cursor_state = Cap_curs_hide));
             break;
          case '=':
@@ -4901,23 +4902,21 @@ static void do_key (int ch) {
    };
    int i;
 
-   putp((Cursor_state = Cap_curs_hide));
    switch (ch) {
       case 0:                // ignored (always)
       case kbd_ESC:          // ignored (sometimes)
-         return;
+         goto all_done;
       case 'q':              // no return from this guy
          bye_bye(NULL);
       case 'W':              // no need for rebuilds
          write_rcfile();
-         return;
+         goto all_done;
       default:               // and now, the real work...
          for (i = 0; i < MAXTBL(key_tab); ++i)
             if (strchr(key_tab[i].keys, ch)) {
                key_tab[i].func(ch);
                Frames_signal = BREAK_kbd;
-               putp((Cursor_state = Cap_curs_hide));
-               return;
+               goto all_done;
             }
    };
    /* Frames_signal above will force a rebuild of all column headers and
@@ -4943,6 +4942,8 @@ static void do_key (int ch) {
     */
 
    show_msg(N_txt(UNKNOWN_cmds_txt));
+all_done:
+   putp((Cursor_state = Cap_curs_hide));
 } // end: do_key
 
 
-- 
1.7.9.2

Places

File 0008-top-tweak-cursor-state-code-to-swat-an-obscure-bugle.patch of Package procps (Revision b7bb9355db92dad1ab5a522e07d57590)

Places