Prefactor: drop #[rustfmt::skip] on broadcast_latest_holder_commitment_txn
What changed, and why it matters
This commit is purely a code formatting cleanup. It removes a `#[rustfmt::skip]` annotation and lets rustfmt reformat the `broadcast_latest_holder_commitment_txn` function. There is no change to program logic, behavior, or security.
No action needed. This is a non-functional style refactor.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff removes #[rustfmt::skip] from ChannelMonitor::broadcast_latest_holder_commitment_txn and adjusts line breaks/commas to match rustfmt style. The function signature, generic bounds, body, and call to inner.queue_latest_holder_commitment_txn_for_broadcast remain semantically identical. No functional code was modified.
Changed components
lightning/src/chain/channelmonitor.rsInspect captured patch +8 / −6
diff --git a/lightning/src/chain/channelmonitor.rs b/lightning/src/chain/channelmonitor.rs
index 4acba6f..37e337c 100644
--- a/lightning/src/chain/channelmonitor.rs
+++ b/lightning/src/chain/channelmonitor.rs
@@ -2327,19 +2327,21 @@ impl<Signer: EcdsaChannelSigner> ChannelMonitor<Signer> {
/// close channel with their commitment transaction after a substantial amount of time. Best
/// may be to contact the other node operator out-of-band to coordinate other options available
/// to you.
- #[rustfmt::skip]
pub fn broadcast_latest_holder_commitment_txn<B: Deref, F: Deref, L: Deref>(
- &self, broadcaster: &B, fee_estimator: &F, logger: &L
- )
- where
+ &self, broadcaster: &B, fee_estimator: &F, logger: &L,
+ ) where
B::Target: BroadcasterInterface,
F::Target: FeeEstimator,
- L::Target: Logger
+ L::Target: Logger,
{
let mut inner = self.inner.lock().unwrap();
let fee_estimator = LowerBoundedFeeEstimator::new(&**fee_estimator);
let logger = WithChannelMonitor::from_impl(logger, &*inner, None);
- inner.queue_latest_holder_commitment_txn_for_broadcast(broadcaster, &fee_estimator, &logger);
+ inner.queue_latest_holder_commitment_txn_for_broadcast(
+ broadcaster,
+ &fee_estimator,
+ &logger,
+ );
}
/// Unsafe test-only version of `broadcast_latest_holder_commitment_txn` used by our test framework
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.