apply (some) clippy suggestions

This commit is contained in:
2025-12-03 12:42:56 +00:00
parent 4477c81b81
commit 99db8bcd72
3 changed files with 11 additions and 11 deletions
+3 -3
View File
@@ -1,3 +1,3 @@
pub(crate) mod handle; pub mod handle;
pub(crate) mod memory; pub mod memory;
pub(crate) mod snapshot; pub mod snapshot;
+1 -1
View File
@@ -23,7 +23,7 @@ impl Drop for Handle {
fn drop(&mut self) { fn drop(&mut self) {
if let Err(why) = unsafe { CloseHandle(**self) } { if let Err(why) = unsafe { CloseHandle(**self) } {
log::error!("failed to close handle: {why}"); log::error!("failed to close handle: {why}");
}; }
} }
} }
+7 -7
View File
@@ -1,7 +1,7 @@
use crate::Error; use crate::Error;
use windows::Win32::System::Diagnostics::ToolHelp::{ use windows::Win32::System::Diagnostics::ToolHelp::{
CreateToolhelp32Snapshot, Module32FirstW, Module32NextW, Process32FirstW, Process32NextW, CreateToolhelp32Snapshot, MODULEENTRY32W, Module32FirstW, Module32NextW, PROCESSENTRY32W,
MODULEENTRY32W, PROCESSENTRY32W, TH32CS_SNAPMODULE, TH32CS_SNAPMODULE32, TH32CS_SNAPPROCESS, Process32FirstW, Process32NextW, TH32CS_SNAPMODULE, TH32CS_SNAPMODULE32, TH32CS_SNAPPROCESS,
}; };
#[derive(Debug)] #[derive(Debug)]
@@ -20,7 +20,7 @@ impl ProcessSnapshot {
dwSize: u32::try_from(size_of::<PROCESSENTRY32W>())?, dwSize: u32::try_from(size_of::<PROCESSENTRY32W>())?,
..Default::default() ..Default::default()
}; };
unsafe { Process32FirstW(snapshot, &mut process_entry_32_w) }.map_err(|why| { unsafe { Process32FirstW(snapshot, &raw mut process_entry_32_w) }.map_err(|why| {
Error::CreateSnapshotError(format!("failed to get first process from snapshot: {why}")) Error::CreateSnapshotError(format!("failed to get first process from snapshot: {why}"))
})?; })?;
let mut processes = Vec::new(); let mut processes = Vec::new();
@@ -33,9 +33,9 @@ impl ProcessSnapshot {
name, name,
}; };
processes.push(process); processes.push(process);
if unsafe { Process32NextW(snapshot, &mut process_entry_32_w) }.is_err() { if unsafe { Process32NextW(snapshot, &raw mut process_entry_32_w) }.is_err() {
break; break;
}; }
} }
Ok(processes) Ok(processes)
} }
@@ -60,7 +60,7 @@ impl ModuleSnapshot {
dwSize: u32::try_from(size_of::<MODULEENTRY32W>())?, dwSize: u32::try_from(size_of::<MODULEENTRY32W>())?,
..Default::default() ..Default::default()
}; };
unsafe { Module32FirstW(snapshot, &mut module_entry_32_w) }.map_err(|why| { unsafe { Module32FirstW(snapshot, &raw mut module_entry_32_w) }.map_err(|why| {
Error::CreateSnapshotError(format!("failed to get first module from snapshot: {why}")) Error::CreateSnapshotError(format!("failed to get first module from snapshot: {why}"))
})?; })?;
let mut modules = Vec::new(); let mut modules = Vec::new();
@@ -78,7 +78,7 @@ impl ModuleSnapshot {
base_size: module_entry_32_w.modBaseSize as usize, base_size: module_entry_32_w.modBaseSize as usize,
}; };
modules.push(module); modules.push(module);
if unsafe { Module32NextW(snapshot, &mut module_entry_32_w).is_err() } { if unsafe { Module32NextW(snapshot, &raw mut module_entry_32_w).is_err() } {
break; break;
} }
} }