me/iced
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix system informations cpu_brand always being empty

Browse source 
This was caused by a misuse of the `global_cpu_info` function, which
does not contain a valid `cpu_brand` field.

To fix this, we now get the first cpu and return it's brand instead.

Fixes #2794
This commit is contained in:
AMS21 2025-02-07 08:44:07 +01:00
parent 940a079d83
commit 3c01854575
No known key found for this signature in database
GPG key ID: B31439F272FEEA96
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
winit/src/system.rs
View file

@ -17,7 +17,11 @@ pub(crate) fn information(
let mut system = System::new_all(); let mut system = System::new_all();
system.refresh_all(); system.refresh_all();
let cpu = system.global_cpu_info(); let cpu_brand = system
.cpus()
.first()
.map(|cpu| cpu.brand().to_string())
.unwrap_or_default();
let memory_used = sysinfo::get_current_pid() let memory_used = sysinfo::get_current_pid()
.and_then(|pid| system.process(pid).ok_or("Process not found")) .and_then(|pid| system.process(pid).ok_or("Process not found"))
@ -29,7 +33,7 @@ pub(crate) fn information(
system_kernel: System::kernel_version(), system_kernel: System::kernel_version(),
system_version: System::long_os_version(), system_version: System::long_os_version(),
system_short_version: System::os_version(), system_short_version: System::os_version(),
cpu_brand: cpu.brand().into(), cpu_brand,
cpu_cores: system.physical_core_count(), cpu_cores: system.physical_core_count(),
memory_total: system.total_memory(), memory_total: system.total_memory(),
memory_used, memory_used,