mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2024-09-30 00:31:14 +00:00
LibCore: Make usernames optional in ProcessStatisticsReader
This commit is contained in:
parent
71728f3ea6
commit
76c67b7ae9
Notes:
sideshowbarker
2024-07-18 00:41:35 +09:00
Author: https://github.com/petelliott Commit: https://github.com/SerenityOS/serenity/commit/76c67b7ae9 Pull-request: https://github.com/SerenityOS/serenity/pull/15142 Reviewed-by: https://github.com/ADKaster Reviewed-by: https://github.com/bgianfo
|
@ -16,7 +16,7 @@ namespace Core {
|
|||
|
||||
HashMap<uid_t, String> ProcessStatisticsReader::s_usernames;
|
||||
|
||||
Optional<AllProcessesStatistics> ProcessStatisticsReader::get_all(RefPtr<Core::File>& proc_all_file)
|
||||
Optional<AllProcessesStatistics> ProcessStatisticsReader::get_all(RefPtr<Core::File>& proc_all_file, bool include_usernames)
|
||||
{
|
||||
if (proc_all_file) {
|
||||
if (!proc_all_file->seek(0, Core::SeekMode::SetPosition)) {
|
||||
|
@ -93,7 +93,9 @@ Optional<AllProcessesStatistics> ProcessStatisticsReader::get_all(RefPtr<Core::F
|
|||
});
|
||||
|
||||
// and synthetic data last
|
||||
if (include_usernames) {
|
||||
process.username = username_from_uid(process.uid);
|
||||
}
|
||||
all_processes_statistics.processes.append(move(process));
|
||||
});
|
||||
|
||||
|
@ -102,10 +104,10 @@ Optional<AllProcessesStatistics> ProcessStatisticsReader::get_all(RefPtr<Core::F
|
|||
return all_processes_statistics;
|
||||
}
|
||||
|
||||
Optional<AllProcessesStatistics> ProcessStatisticsReader::get_all()
|
||||
Optional<AllProcessesStatistics> ProcessStatisticsReader::get_all(bool include_usernames)
|
||||
{
|
||||
RefPtr<Core::File> proc_all_file;
|
||||
return get_all(proc_all_file);
|
||||
return get_all(proc_all_file, include_usernames);
|
||||
}
|
||||
|
||||
String ProcessStatisticsReader::username_from_uid(uid_t uid)
|
||||
|
|
|
@ -72,8 +72,8 @@ struct AllProcessesStatistics {
|
|||
|
||||
class ProcessStatisticsReader {
|
||||
public:
|
||||
static Optional<AllProcessesStatistics> get_all(RefPtr<Core::File>&);
|
||||
static Optional<AllProcessesStatistics> get_all();
|
||||
static Optional<AllProcessesStatistics> get_all(RefPtr<Core::File>&, bool include_usernames = true);
|
||||
static Optional<AllProcessesStatistics> get_all(bool include_usernames = true);
|
||||
|
||||
private:
|
||||
static String username_from_uid(uid_t);
|
||||
|
|
Loading…
Reference in a new issue