]> granicus.if.org Git - procps-ng/commitdiff
top: revert part of a patch regarding separate threads
authorJim Warner <james.warner@comcast.net>
Thu, 30 Sep 2021 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@dropbear.xyz>
Sat, 2 Oct 2021 03:00:00 +0000 (13:00 +1000)
In the patch referenced below, some logic was moved in
that frame_make function in an effort to overlap those
background threads with additional main thread duties.

Unfortunately, the auto-size column feature was broken
in the process. So this patch will revert a portion of
the earlier patch so as to restore the broken feature.

Reference(s):
. Sep 2021, top overlap additional processing
commit f11f43b4f7118f284f9aec19f6885d163848ffe7

Signed-off-by: Jim Warner <james.warner@comcast.net>
top/top.c

index c52b4eeff3ace6071879bd175ea5209fa318320b..2c92212c8f4c0bfb148768161607db76cb30fdc5 100644 (file)
--- a/top/top.c
+++ b/top/top.c
@@ -6361,6 +6361,10 @@ static void frame_make (void) {
    WIN_t *w = Curwin;             // avoid gcc bloat with a local copy
    int i, scrlins;
 
+   // check auto-sized width increases from the last iteration...
+   if (AUTOX_MODE && Autox_found)
+      widths_resize();
+
    /* deal with potential signal(s) since the last time around
       plus any input which may change 'tasks_refresh' needs... */
    if (Frames_signal)
@@ -6382,10 +6386,6 @@ static void frame_make (void) {
    memory_refresh(NULL);
 #endif
 
-   // check auto-sized width increases from the last iteration...
-   if (AUTOX_MODE && Autox_found)
-      widths_resize();
-
    // whoa either first time or thread/task mode change, (re)prime the pump...
    if (Pseudo_row == PROC_XTRA) {
       usleep(LIB_USLEEP);