What the developer wrote
Authored by Tobin C. Harding
Remove core:: prefix from fmt
`fmt` is already in scope.
What changed, and why it matters
This is a trivial code cleanup: it removes unnecessary 'core::' prefixes from formatting-related names in a Rust source file. The behavior of the program is unchanged, and there is no security relevance.
No security action needed; treat as normal refactoring.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit changes three occurrences of ‘core::fmt’ to just ‘fmt’ in an implementation of the Display trait for UnexpectedEofError in primitives/src/witness.rs. Since ‘fmt’ is already imported into scope, this is a purely cosmetic simplification with no functional or security impact.
Changed components
primitives/src/witness.rsInspect captured patch +2 / −2
diff --git a/primitives/src/witness.rs b/primitives/src/witness.rs
index bcb200a7..deb8b912 100644
--- a/primitives/src/witness.rs
+++ b/primitives/src/witness.rs
@@ -870,8 +870,8 @@ impl From<Infallible> for UnexpectedEofError {
fn from(never: Infallible) -> Self { match never {} }
}
-impl core::fmt::Display for UnexpectedEofError {
- fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result {
+impl fmt::Display for UnexpectedEofError {
+ fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
write!(f, "not enough witness elements for decoder, missing {}", self.missing_elements)
}
}
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.