Rename struct fields in units serde test
What the developer wrote
Authored by Jakub Gladysz
Rename struct fields in units serde test
What changed, and why it matters
This commit only renames three test-only struct fields from single-letter names (a, b, c) to descriptive names (block_height, block_height_interval, weight) inside a single Rust test file. It does not change any production code, logic, serialization format, or behavior. There is no security relevance.
No action needed. This is a non-functional code-quality/test readability change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies units/tests/serde.rs, a test file for serde serialization/deserialization of types in the rust-bitcoin units crate. It renames fields in a private test struct and the corresponding constructor. The types, serde attributes, and data values are unchanged. No library code, public API, or serialization contract is affected.
Changed components
units/tests/serde.rsInspect captured patch +6 / −6
diff --git a/units/tests/serde.rs b/units/tests/serde.rs
index a75510b6..ef16094c 100644
--- a/units/tests/serde.rs
+++ b/units/tests/serde.rs
@@ -45,9 +45,9 @@ struct Serde {
#[serde(with = "fee_rate::serde::as_sat_per_kwu_floor::opt")]
opt_kwu: Option<FeeRate>,
- a: BlockHeight,
- b: BlockHeightInterval,
- c: Weight,
+ block_height: BlockHeight,
+ block_height_interval: BlockHeightInterval,
+ weight: Weight,
}
impl Serde {
@@ -74,9 +74,9 @@ impl Serde {
opt_vb_ceil: Some(FeeRate::BROADCAST_MIN),
opt_kwu: Some(FeeRate::BROADCAST_MIN),
- a: BlockHeight::MAX,
- b: BlockHeightInterval::MAX,
- c: Weight::MAX,
+ block_height: BlockHeight::MAX,
+ block_height_interval: BlockHeightInterval::MAX,
+ weight: Weight::MAX,
}
}
}
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.