RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-05-14 09:24:57 +00:00
Code Issues Activity Actions

ps: Allow multiple filtering options to be used simultaneously

Browse source 
Previously, it was assumed that only one filtering option, such as
`-u` or `-p` would be used at a time. With this PR, processes are now
shown if they match any of the specified filters.
This commit is contained in:
Tim Ledbetter 2023-07-09 16:26:08 +01:00 • committed by Sam Atkins
parent a758e27153
commit aeb87d6e78
2 changed files with 16 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

23
Userland/Utilities/ps.cpp
View file

@ -163,15 +163,20 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
auto& processes = all_processes.processes;
// Filter
if (!pid_list.is_empty()) {
processes.remove_all_matching([&](auto& process) { return !pid_list.contains_slow(process.pid); });
} else if (!uid_list.is_empty()) {
processes.remove_all_matching([&](auto& process) { return !uid_list.contains_slow(process.uid); });
} else if (every_terminal_process_flag) {
processes.remove_all_matching([&](auto& process) { return process.tty.is_empty(); });
} else if (!every_process_flag) {
// Default is to show processes from the current TTY
processes.remove_all_matching([&](Core::ProcessStatistics& process) { return process.tty.view() != this_pseudo_tty_name.bytes_as_string_view(); });
if (!every_process_flag) {
Vector<Core::ProcessStatistics> filtered_processes;
for (auto const& process : processes) {
// Default is to show processes from the current TTY
if ((!provided_filtering_option && process.tty == this_pseudo_tty_name.bytes_as_string_view())
|| (!pid_list.is_empty() && pid_list.contains_slow(process.pid))
|| (!uid_list.is_empty() && uid_list.contains_slow(process.uid))
|| (every_terminal_process_flag && !process.tty.is_empty())) {
filtered_processes.append(process);
}
}
processes = move(filtered_processes);
}
// Sort