fix(wol): clippy + cfg-gate the Windows client module — main compiles again
The Wake-on-LAN batch landed with lints that fail `clippy -D warnings` (doc continuation, char-array split, io::Error::other, redundant closure) and an ungated `mod wol;` in the Windows client, which pulls windows-only crates into the non-Windows stub build. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
@@ -23,7 +23,7 @@ pub extern "system" fn Java_io_unom_punktfunk_kit_NativeBridge_nativeWakeOnLan<'
|
|||||||
};
|
};
|
||||||
let last_ip: String = env
|
let last_ip: String = env
|
||||||
.get_string(&last_ip)
|
.get_string(&last_ip)
|
||||||
.map(|s| Into::<String>::into(s))
|
.map(Into::<String>::into)
|
||||||
.unwrap_or_default();
|
.unwrap_or_default();
|
||||||
let macs: Vec<[u8; 6]> = macs_csv
|
let macs: Vec<[u8; 6]> = macs_csv
|
||||||
.split(',')
|
.split(',')
|
||||||
|
|||||||
@@ -43,6 +43,7 @@ mod trust;
|
|||||||
#[cfg(windows)]
|
#[cfg(windows)]
|
||||||
mod video;
|
mod video;
|
||||||
|
|
||||||
|
#[cfg(windows)]
|
||||||
mod wol;
|
mod wol;
|
||||||
|
|
||||||
#[cfg(windows)]
|
#[cfg(windows)]
|
||||||
|
|||||||
@@ -13,6 +13,7 @@
|
|||||||
//! * the **limited broadcast** `255.255.255.255`, and
|
//! * the **limited broadcast** `255.255.255.255`, and
|
||||||
//! * optionally a **unicast** to the host's last-known IP (covers the brief window where the
|
//! * optionally a **unicast** to the host's last-known IP (covers the brief window where the
|
||||||
//! host is reachable but hasn't re-advertised, and NICs that wake on a directed unicast),
|
//! host is reachable but hasn't re-advertised, and NICs that wake on a directed unicast),
|
||||||
|
//!
|
||||||
//! on the two conventional WoL ports (9 and 7), repeated a few times to survive UDP loss.
|
//! on the two conventional WoL ports (9 and 7), repeated a few times to survive UDP loss.
|
||||||
|
|
||||||
use std::io;
|
use std::io;
|
||||||
@@ -36,7 +37,7 @@ const BURST: usize = 3;
|
|||||||
pub fn parse_mac(s: &str) -> Option<Mac> {
|
pub fn parse_mac(s: &str) -> Option<Mac> {
|
||||||
let mut m = [0u8; 6];
|
let mut m = [0u8; 6];
|
||||||
let mut n = 0;
|
let mut n = 0;
|
||||||
for part in s.split(|c| c == ':' || c == '-') {
|
for part in s.split([':', '-']) {
|
||||||
if n == 6 {
|
if n == 6 {
|
||||||
return None; // too many octets
|
return None; // too many octets
|
||||||
}
|
}
|
||||||
@@ -103,10 +104,7 @@ pub fn send_magic_packet(macs: &[Mac], last_known_ip: Option<Ipv4Addr>) -> io::R
|
|||||||
if sent_any {
|
if sent_any {
|
||||||
Ok(())
|
Ok(())
|
||||||
} else {
|
} else {
|
||||||
Err(io::Error::new(
|
Err(io::Error::other("no magic packet could be sent"))
|
||||||
io::ErrorKind::Other,
|
|
||||||
"no magic packet could be sent",
|
|
||||||
))
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user