]> granicus.if.org Git - procps-ng/commit
top: avoid library shame with refactored 'Ctrl' window
authorJim Warner <james.warner@comcast.net>
Wed, 4 May 2022 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@dropbear.xyz>
Wed, 4 May 2022 06:45:57 +0000 (16:45 +1000)
commitbc4b499ed2403caf2cfb9b87067cd004e890c6fc
tree947d798f1e3797c3f83ee861a28f49afc649c926
parentb47693c078c256e897baf6b50f3aaf6accb5d35e
top: avoid library shame with refactored 'Ctrl' window

Well darn it, whoever wrote that new library caught me
with my pants down (again?). Shoot, they were not just
down but somehow missing altogether. Here's the story.

Any item from that library supported by dynamic memory
can only be represented in user's stacks exactly once.

Should any string based enumerator be duplicated among
the items array, for any instance beyond the first the
library will return '[ duplicate ENUM ]' for a result.

That's where I lost my pants. While command lines were
given special handling (and never duplicated) I failed
to turn on CGROUPS, SUPGRPS & ENVIRON when testing the
Ctrl-G, Ctrl-U & Ctrl-N keys. If any of those 3 are on
that's when a Ctrl window sees a 'duplicate' notation.

[ and who runs top with such fields displayed anyway ]

In responding to this oops, the internals were changed
quite dramatically & vastly simplified in the process.

More importantly, the 'duplicate' results are no more.

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