diff options
author | Hisham Muhammad <hisham@gobolinux.org> | 2020-12-18 11:03:31 -0300 |
---|---|---|
committer | BenBE <BenBE@geshi.org> | 2020-12-19 16:02:34 +0100 |
commit | 2327260ee888146beab3da66507c7744334a6973 (patch) | |
tree | 06b3919321515cc4fdea7de0b5ead4744f3600ec /Process.c | |
parent | e8c6994f40c9c69089e9f80abb2f895d2e077c7e (diff) |
Separate tree and list sort orders
Implements the suggestion from https://github.com/htop-dev/htop/issues/399#issuecomment-747861013
Thanks to the refactors from 0bd5c8fb5da and 6393baa74e5, this was really easy
and clean to do.
It maintains the "Tree view always by PID" option in the Settings, which
results in some specific behaviors such as "clicking on the column header to
exit tree view" and "picking a new sort order to exit tree view", for the sake
of the muscle memory of long time htop users. :)
Diffstat (limited to 'Process.c')
-rw-r--r-- | Process.c | 7 |
1 files changed, 2 insertions, 5 deletions
@@ -497,7 +497,7 @@ long Process_compare(const void* v1, const void* v2) { const Settings *settings = ((const Process*)v1)->settings; int r; - if (settings->direction == 1) { + if (Settings_getActiveDirection(settings) == 1) { p1 = (const Process*)v1; p2 = (const Process*)v2; } else { @@ -505,10 +505,7 @@ long Process_compare(const void* v1, const void* v2) { p1 = (const Process*)v2; } - ProcessField key = settings->sortKey; - if (settings->treeView && settings->treeViewAlwaysByPID) { - key = PID; - } + ProcessField key = Settings_getActiveSortKey(settings); switch (key) { case PERCENT_CPU: |