From 51bee082475aca4c74daf7fbd9a26a505f58c1b4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ralph=20K=C3=BCpper?= Date: Wed, 5 Aug 2026 13:59:04 +0200 Subject: [PATCH] style: rustfmt after #7434 cargo fmt --all -- --check failed on main. lint is a REQUIRED context, so while it is red every merge bypasses a required gate. Two hunks, six lines: an assert! in gc/tests/telemetry_verifier.rs over rustfmt's default fn_call_width of 60, and a pub(crate) use in object/field_get_set.rs added below the pub use block where reorder_imports wants it above. Not a toolchain artifact: CI uses dtolnay/rust-toolchain@stable and the repo has no rust-toolchain.toml or rustfmt.toml. --- changelog.d/7442-fmt-after-7434.md | 7 +++++++ crates/perry-runtime/src/gc/tests/telemetry_verifier.rs | 5 ++++- crates/perry-runtime/src/object/field_get_set.rs | 2 +- 3 files changed, 12 insertions(+), 2 deletions(-) create mode 100644 changelog.d/7442-fmt-after-7434.md diff --git a/changelog.d/7442-fmt-after-7434.md b/changelog.d/7442-fmt-after-7434.md new file mode 100644 index 0000000000..042df26e37 --- /dev/null +++ b/changelog.d/7442-fmt-after-7434.md @@ -0,0 +1,7 @@ +**Fixed** `cargo fmt --all -- --check` failed on `main` after #7434. + +`lint` is a required context, so a red `lint` means every subsequent merge +bypasses a required gate. Two hunks, six lines: an `assert!` in +`gc/tests/telemetry_verifier.rs` exceeding rustfmt's default `fn_call_width`, and +a `pub(crate) use` in `object/field_get_set.rs` added below the `pub use` block +where `reorder_imports` wants it above. diff --git a/crates/perry-runtime/src/gc/tests/telemetry_verifier.rs b/crates/perry-runtime/src/gc/tests/telemetry_verifier.rs index 7fdf77ec4d..7746d38529 100644 --- a/crates/perry-runtime/src/gc/tests/telemetry_verifier.rs +++ b/crates/perry-runtime/src/gc/tests/telemetry_verifier.rs @@ -396,7 +396,10 @@ fn test_pause_ring_records_max_and_window() { fn test_record_collection_bumps_read_pic_epoch() { use std::sync::atomic::Ordering; let before = crate::object::PERRY_IC_EPOCH.load(Ordering::Relaxed); - assert!(before >= 1, "epoch starts at 1 so zeroinitializer never hits"); + assert!( + before >= 1, + "epoch starts at 1 so zeroinitializer never hits" + ); GC_STATS.with(|stats| { stats.borrow_mut().record_collection(0, 1); }); diff --git a/crates/perry-runtime/src/object/field_get_set.rs b/crates/perry-runtime/src/object/field_get_set.rs index 1f7841d355..497436d078 100644 --- a/crates/perry-runtime/src/object/field_get_set.rs +++ b/crates/perry-runtime/src/object/field_get_set.rs @@ -192,6 +192,7 @@ pub(crate) use has_property::{ wide_key_index_note_hit, WIDE_KEY_INDEX_MIN_KEYS, }; pub use has_property::{js_in_operator, js_object_has_property}; +pub(crate) use ic_miss::pic_epoch_bump; pub(crate) use ic_miss::{ is_array_method_value_name, is_primitive_proto_method, is_timer_handle_method_key, set_method_value_name, @@ -201,7 +202,6 @@ pub use ic_miss::{ js_object_get_field_ic_miss, js_object_set_field_by_property_id, js_private_brand_check, js_private_guard, PERRY_IC_EPOCH, }; -pub(crate) use ic_miss::pic_epoch_bump; #[cfg(test)] mod buffer_ic_miss_tests {