What the developer wrote
Authored by Abeeujah
Remove duplicate write_err import
Duplicate import of `write_err` triggers the unused imports lint
warning, the second import statement is redundant as Rust sequentially
resolves imported modules.
`CompactSizeDecoderError` is in the global error module scope here where
the first instance of `write_err` is imported, the second instance in
the `fmt` Display implementation is shadowed by the first import in
the global scope, trigerring the unused import lint.
What changed, and why it matters
This commit removes a single duplicate Rust import statement that was causing a compiler lint warning about an unused import. It is a routine code cleanup with no security relevance.
No security action needed. Treat as normal maintenance / lint cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change deletes a redundant use internals::write_err; inside an impl core::fmt::Display block. The same import was already present in the module’s global scope, so the inner import was shadowed and flagged by the unused imports lint. The Display implementation continues to use write_err from the global import. No functional behavior changes.
Changed components
consensus_encoding/src/error.rsInspect captured patch +0 / −1
diff --git a/consensus_encoding/src/error.rs b/consensus_encoding/src/error.rs
index d8643ff0..d98fcbc3 100644
--- a/consensus_encoding/src/error.rs
+++ b/consensus_encoding/src/error.rs
@@ -141,7 +141,6 @@ impl From<Infallible> for CompactSizeDecoderError {
impl core::fmt::Display for CompactSizeDecoderError {
fn fmt(&self, f: &mut core::fmt::Formatter) -> core::fmt::Result {
- use internals::write_err;
use CompactSizeDecoderErrorInner as E;
match self.0 {
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.