Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: Adds tree view #223

Merged
merged 21 commits into from
Sep 7, 2020
Merged

feature: Adds tree view #223

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
6df7014
Add back highlighting to CPU legend if not focused
ClementTsang Sep 3, 2020
8c4ea05
Get *very* basic tree-mode working
ClementTsang Sep 4, 2020
94d9267
Fix husky
ClementTsang Sep 4, 2020
d4d489a
Dang conflicts
ClementTsang Sep 4, 2020
8023f11
Merge branch 'master' into forestry
ClementTsang Sep 4, 2020
94dfafe
Add some comments to taste
ClementTsang Sep 4, 2020
7f7c8ee
Switch to using pid_t
ClementTsang Sep 4, 2020
cf8c01c
Apparently windows doesn't like that
ClementTsang Sep 4, 2020
c2b6065
Rudimentary trees work and look ok...
ClementTsang Sep 4, 2020
86f921e
Fix line generation, basic version of filtering
ClementTsang Sep 5, 2020
e44c025
Now we have sorting in tree mode
ClementTsang Sep 6, 2020
78046b3
Windows implementation
ClementTsang Sep 6, 2020
d8cc0b3
Let sort apply to top level as well
ClementTsang Sep 6, 2020
f442c50
attempt to fix conflict
ClementTsang Sep 6, 2020
243a743
Simplify tree orphan step
ClementTsang Sep 6, 2020
53ad270
Merge branch 'master' into forestry
ClementTsang Sep 6, 2020
5a403e1
Fix merge error
ClementTsang Sep 6, 2020
9ab2f2d
Update docs
ClementTsang Sep 6, 2020
0229261
Update changelog
ClementTsang Sep 7, 2020
d43b913
Update widths for now
ClementTsang Sep 7, 2020
3aba4b4
Fix sort being reversed for trees
ClementTsang Sep 7, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add some comments to taste
  • Loading branch information
@ClementTsang
ClementTsang committed Sep 4, 2020
commit 94dfafeadc10e3e7d71aae3f60749d2f4196ad00
6 changes: 2 additions & 4 deletions src/data_conversion.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -457,7 +457,6 @@ pub fn tree_process_data(
) -> Vec<ConvertedProcessData> {
// Let's first build up a (really terrible) parent -> child mapping...
// At the same time, let's make a mapping of PID -> process data!
// TODO: ideally... I shouldn't have to do this... this seems kinda... geh.
let mut parent_child_mapping: HashMap<u32, Vec<u32>> = HashMap::default();
let mut pid_process_mapping: HashMap<u32, &ConvertedProcessData> = HashMap::default();

Expand All @@ -480,7 +479,7 @@ pub fn tree_process_data(
if let Some(zero_pid) = parent_child_mapping.get(&0) {
pids_to_explore.extend(zero_pid);
} else {
// FIXME: Remove this, this is for debugging
// FIXME: Remove this, this is for debugging... mainly because idk how the heck windows will behave
debug!("PID 0 had no children during tree building...");
}

Expand All @@ -494,8 +493,7 @@ pub fn tree_process_data(
}

// Now let's "rearrange" our current list of converted process data into the correct
// order required...

// order required... and we're done!
explored_pids
.iter()
.filter_map(|pid| match pid_process_mapping.remove(pid) {
Expand Down