units: Group private and public modules separately
What the developer wrote
Authored by Tobin C. Harding
units: Group private and public modules separately
As is customary group the private modules above the public ones.
What changed, and why it matters
This commit simply reorders module declarations in a Rust source file so that private modules are listed before public ones. It makes no functional changes to the code and has no security relevance.
No action required; this is a non-functional style/refactoring change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff in units/src/amount/mod.rs only moves existing mod statements around: private modules (signed, result, tests, unsigned, verification) are grouped above public modules (error, serde). No code is added, removed, or modified; visibility, behavior, and API surface remain identical.
Changed components
units/src/amount/mod.rsInspect captured patch +5 / −5
diff --git a/units/src/amount/mod.rs b/units/src/amount/mod.rs
index dfa04db4..ce99c988 100644
--- a/units/src/amount/mod.rs
+++ b/units/src/amount/mod.rs
@@ -5,18 +5,18 @@
//! This module mainly introduces the [`Amount`] and [`SignedAmount`] types.
//! We refer to the documentation on the types for more information.
-pub mod error;
-mod result;
-#[cfg(feature = "serde")]
-pub mod serde;
-
mod signed;
+mod result;
#[cfg(test)]
mod tests;
mod unsigned;
#[cfg(kani)]
mod verification;
+pub mod error;
+#[cfg(feature = "serde")]
+pub mod serde;
+
use core::cmp::Ordering;
use core::convert::Infallible;
use core::fmt;
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.