What the developer wrote
Authored by Elias Rohrer
Drop unused imports
Co-Authored-By: HAL 9000
What changed, and why it matters
This commit simply removes two unused import statements from Rust source files. It does not change any program logic, data handling, or security behavior. There is no security relevance.
No security action needed. This is a routine code-quality cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit deletes use crate::impl_writeable_tlv_based; from lightning/src/chain/mod.rs and removes impl_writeable_tlv_based from a grouped import in lightning/src/util/sweep.rs. These are compile-time-only import cleanups with no functional diff. The macro remains available elsewhere in the crate; it is just no longer imported into these two modules where it was not used.
Changed components
lightning/src/chain/mod.rslightning/src/util/sweep.rsInspect captured patch +1 / −2
diff --git a/lightning/src/chain/mod.rs b/lightning/src/chain/mod.rs
index e8baa7a..bc47f1b 100644
--- a/lightning/src/chain/mod.rs
+++ b/lightning/src/chain/mod.rs
@@ -20,7 +20,6 @@ use bitcoin::secp256k1::PublicKey;
use crate::chain::channelmonitor::{ChannelMonitor, ChannelMonitorUpdate, MonitorEvent};
use crate::chain::transaction::{OutPoint, TransactionData};
-use crate::impl_writeable_tlv_based;
use crate::ln::types::ChannelId;
use crate::sign::ecdsa::EcdsaChannelSigner;
use crate::sign::HTLCDescriptor;
diff --git a/lightning/src/util/sweep.rs b/lightning/src/util/sweep.rs
index f0c2d6d..6815b39 100644
--- a/lightning/src/util/sweep.rs
+++ b/lightning/src/util/sweep.rs
@@ -28,7 +28,7 @@ use crate::util::persist::{
OUTPUT_SWEEPER_PERSISTENCE_PRIMARY_NAMESPACE, OUTPUT_SWEEPER_PERSISTENCE_SECONDARY_NAMESPACE,
};
use crate::util::ser::{Readable, ReadableArgs, Writeable};
-use crate::{impl_writeable_tlv_based, log_debug, log_error};
+use crate::{log_debug, log_error};
use bitcoin::block::Header;
use bitcoin::locktime::absolute::LockTime;
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.