From 86460040ba2cc6dec80038a28595eeef65571ab1 Mon Sep 17 00:00:00 2001 From: Thomas Eizinger Date: Fri, 23 Aug 2024 23:07:57 +0100 Subject: [PATCH] chore(connlib): remove unnecessary use of `static` (#6425) The only reason we use an `AtomicU32` here is because the closure is only an `Fn` and not an `FnMut` closure which prevents us from using a regular `u32`. --- rust/connlib/tunnel/src/tests.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/rust/connlib/tunnel/src/tests.rs b/rust/connlib/tunnel/src/tests.rs index ccbf41838..a4b0f2297 100644 --- a/rust/connlib/tunnel/src/tests.rs +++ b/rust/connlib/tunnel/src/tests.rs @@ -36,7 +36,7 @@ fn tunnel_test() { ..Default::default() }; - static TEST_INDEX: AtomicU32 = AtomicU32::new(0); + let test_index = AtomicU32::new(0); let _ = std::fs::remove_dir_all("testcases"); let _ = std::fs::create_dir_all("testcases"); @@ -44,7 +44,7 @@ fn tunnel_test() { let result = TestRunner::new(config).run( &ReferenceState::sequential_strategy(5..15), |(mut ref_state, transitions, mut seen_counter)| { - let test_index = TEST_INDEX.fetch_add(1, std::sync::atomic::Ordering::Relaxed); + let test_index = test_index.fetch_add(1, std::sync::atomic::Ordering::Relaxed); let _guard = init_logging(&ref_state, test_index); std::fs::write(