From cd11e47d88caeeb0ddf0a914db73e5ad6fd71f0c Mon Sep 17 00:00:00 2001 From: Quentin Monnet Date: Mon, 6 Jul 2026 12:10:40 +0100 Subject: [PATCH] test(nat): Remove warning for shuttle tests We have a warning when building the shuttle tests for nat: $ cargo check --features shuttle --all-targets -p dataplane-nat [...] warning: function `vpcd1` is never used --> nat/src/masquerade/apalloc/test_alloc.rs:43:12 | 43 | pub fn vpcd1() -> VpcDiscriminant { | ^^^^^ | = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default It's not blocking, but let's suppress it anyway. On Copilot review's suggestion, suppress "dead_code" warnings rather than "unused", that might be broader (such as unused variables or imports inside of body functions). Signed-off-by: Quentin Monnet --- nat/src/masquerade/apalloc/test_alloc.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/nat/src/masquerade/apalloc/test_alloc.rs b/nat/src/masquerade/apalloc/test_alloc.rs index 714b17c0b8..342cc93011 100644 --- a/nat/src/masquerade/apalloc/test_alloc.rs +++ b/nat/src/masquerade/apalloc/test_alloc.rs @@ -21,15 +21,15 @@ mod context { use std::net::{IpAddr, Ipv4Addr}; use std::str::FromStr; - #[allow(unused)] + #[allow(dead_code)] pub fn addr_v4(ip: &str) -> Ipv4Addr { Ipv4Addr::from_str(ip).unwrap() } - #[allow(unused)] + #[allow(dead_code)] pub fn addr_v4_bits(ip: &str) -> u32 { addr_v4(ip).to_bits() } - #[allow(unused)] + #[allow(dead_code)] pub fn ipaddr(ip: &str) -> IpAddr { IpAddr::from_str(ip).unwrap() } @@ -40,6 +40,7 @@ mod context { pub fn vni2() -> Vni { Vni::new_checked(200).unwrap() } + #[allow(dead_code)] pub fn vpcd1() -> VpcDiscriminant { VpcDiscriminant::from_vni(vni1()) }