From 2c07729ff3daef844e81ce2be3096079abaf10e2 Mon Sep 17 00:00:00 2001 From: Shannon Sterz Date: Thu, 6 Mar 2025 13:43:32 +0100 Subject: [PATCH] tree-wide: add parantheses to clarify precedence this resolves a clippy lint that aims to improve legibility for people unaware of rust's precendence rules [1]. [1]: https://rust-lang.github.io/rust-clippy/master/index.html#precedence Signed-off-by: Shannon Sterz --- proxmox-compression/src/zip.rs | 2 +- proxmox-schema/src/upid.rs | 2 +- proxmox-sys/src/systemd.rs | 2 +- proxmox-systemd/src/escape.rs | 2 +- proxmox-uuid/src/lib.rs | 12 ++++++------ 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/proxmox-compression/src/zip.rs b/proxmox-compression/src/zip.rs index 3ccece9b..11b29de4 100644 --- a/proxmox-compression/src/zip.rs +++ b/proxmox-compression/src/zip.rs @@ -360,7 +360,7 @@ impl ZipEntry { comment_len: 0, start_disk: 0, internal_flags: 0, - external_flags: (self.mode as u32) << 16 | (!self.is_file as u32) << 4, + external_flags: ((self.mode as u32) << 16) | ((!self.is_file as u32) << 4), offset, }, ) diff --git a/proxmox-schema/src/upid.rs b/proxmox-schema/src/upid.rs index 9bbb66a1..897fa76a 100644 --- a/proxmox-schema/src/upid.rs +++ b/proxmox-schema/src/upid.rs @@ -191,7 +191,7 @@ fn unescape_id(text: &str) -> Result { } let h1 = hex_digit(i[2])?; let h0 = hex_digit(i[3])?; - data.push(h1 << 4 | h0); + data.push((h1 << 4) | h0); i = &i[4..] } else if next == b'-' { data.push(b'/'); diff --git a/proxmox-sys/src/systemd.rs b/proxmox-sys/src/systemd.rs index 43dc5185..d57f5816 100644 --- a/proxmox-sys/src/systemd.rs +++ b/proxmox-sys/src/systemd.rs @@ -90,7 +90,7 @@ fn unescape_unit_do(text: &str) -> Result, Error> { } let h1 = parse_hex_digit(i[2])?; let h0 = parse_hex_digit(i[3])?; - data.push(h1 << 4 | h0); + data.push((h1 << 4) | h0); i = &i[4..] } else if next == b'-' { data.push(b'/'); diff --git a/proxmox-systemd/src/escape.rs b/proxmox-systemd/src/escape.rs index f73beed3..392907bc 100644 --- a/proxmox-systemd/src/escape.rs +++ b/proxmox-systemd/src/escape.rs @@ -97,7 +97,7 @@ fn unescape_unit_do(text: &str) -> Result, UnescapeError> { } let h1 = parse_hex_digit(i[2])?; let h0 = parse_hex_digit(i[3])?; - data.push(h1 << 4 | h0); + data.push((h1 << 4) | h0); i = &i[4..] } else if next == b'-' { data.push(b'/'); diff --git a/proxmox-uuid/src/lib.rs b/proxmox-uuid/src/lib.rs index 09a70b49..800a8563 100644 --- a/proxmox-uuid/src/lib.rs +++ b/proxmox-uuid/src/lib.rs @@ -103,25 +103,25 @@ impl Uuid { return Err(UuidError); } for i in 0..4 { - uuid[i] = hex_digit(src[2 * i])? << 4 | hex_digit(src[2 * i + 1])?; + uuid[i] = (hex_digit(src[2 * i])? << 4) | hex_digit(src[2 * i + 1])?; } for i in 4..6 { - uuid[i] = hex_digit(src[2 * i + 1])? << 4 | hex_digit(src[2 * i + 2])?; + uuid[i] = (hex_digit(src[2 * i + 1])? << 4) | hex_digit(src[2 * i + 2])?; } for i in 6..8 { - uuid[i] = hex_digit(src[2 * i + 2])? << 4 | hex_digit(src[2 * i + 3])?; + uuid[i] = (hex_digit(src[2 * i + 2])? << 4) | hex_digit(src[2 * i + 3])?; } for i in 8..10 { - uuid[i] = hex_digit(src[2 * i + 3])? << 4 | hex_digit(src[2 * i + 4])?; + uuid[i] = (hex_digit(src[2 * i + 3])? << 4) | hex_digit(src[2 * i + 4])?; } for i in 10..16 { - uuid[i] = hex_digit(src[2 * i + 4])? << 4 | hex_digit(src[2 * i + 5])?; + uuid[i] = (hex_digit(src[2 * i + 4])? << 4) | hex_digit(src[2 * i + 5])?; } } else if src.len() == 32 { let uuid: &mut [u8] = unsafe { &mut (*uuid)[..] }; let src = src.as_bytes(); for i in 0..16 { - uuid[i] = hex_digit(src[2 * i])? << 4 | hex_digit(src[2 * i + 1])?; + uuid[i] = (hex_digit(src[2 * i])? << 4) | hex_digit(src[2 * i + 1])?; } } else { return Err(UuidError);