]> granicus.if.org Git - procps-ng/commit
top: bust up that overly large 'summary_show' function
authorJim Warner <james.warner@comcast.net>
Mon, 3 Jan 2022 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@dropbear.xyz>
Fri, 7 Jan 2022 08:19:15 +0000 (19:19 +1100)
commitcbff1dd10645cb8c7fc55dda08b06fbafed05932
tree3e07c860855861721e53cd40b7c4500626b921dc
parent0e744d3656b1e0299f1083b7f7eabf60fb080f10
top: bust up that overly large 'summary_show' function

Over the years the summary_show function has increased
from around 77 lines of code & comments to its current
size of 243 lines. This is well beyond an ideal length
of available screen rows. So this patch will split it.

We'll take the cpu and memory duties and make separate
functions out of them. Of course, this will incur some
additional call overhead but, given current cpu/memory
logic, any such increase really becomes insignificant.

Now summary_show's a svelte 57 lines of code/comments.

[ this is like what was done to that do_key function ]
[ a decade ago except overhead of new function calls ]
[ plus table lookup was even less of a concern since ]
[ a human was involved, not normal iterative output. ]

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