rustfmt: Remove skip from request_refund_payment
What changed, and why it matters
This commit is purely a code-formatting cleanup. It removes a `#[rustfmt::skip]` directive from a Rust function and lets rustfmt reformat the code. There is no functional change, no bug fix, and no security relevance.
No action required. This is a cosmetic/style commit with no security implications.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff removes the #[rustfmt::skip] attribute from ChannelManager::request_refund_payment in lightning/src/ln/channelmanager.rs and reformats the function signature and body to conform to rustfmt style. The only non-whitespace/punctuation change is moving the let entropy = &*self.entropy_source; statement earlier in the function. No logic, API, or behavior changes are present.
Changed components
lightning/src/ln/channelmanager.rsInspect captured patch +9 / −10
diff --git a/lightning/src/ln/channelmanager.rs b/lightning/src/ln/channelmanager.rs
index 8b2bc1f..d8008b6 100644
--- a/lightning/src/ln/channelmanager.rs
+++ b/lightning/src/ln/channelmanager.rs
@@ -12728,24 +12728,23 @@ where
///
/// [`BlindedPaymentPath`]: crate::blinded_path::payment::BlindedPaymentPath
/// [`Bolt12Invoice`]: crate::offers::invoice::Bolt12Invoice
- #[rustfmt::skip]
pub fn request_refund_payment(
- &self, refund: &Refund
+ &self, refund: &Refund,
) -> Result<Bolt12Invoice, Bolt12SemanticError> {
let secp_ctx = &self.secp_ctx;
+ let entropy = &*self.entropy_source;
let _persistence_guard = PersistenceNotifierGuard::notify_on_drop(self);
- let entropy = &*self.entropy_source;
let builder = self.flow.create_invoice_builder_from_refund(
- &self.router, entropy, refund, self.list_usable_channels(),
+ &self.router,
+ entropy,
+ refund,
+ self.list_usable_channels(),
|amount_msats, relative_expiry| {
- self.create_inbound_payment(
- Some(amount_msats),
- relative_expiry,
- None
- ).map_err(|()| Bolt12SemanticError::InvalidAmount)
- }
+ self.create_inbound_payment(Some(amount_msats), relative_expiry, None)
+ .map_err(|()| Bolt12SemanticError::InvalidAmount)
+ },
)?;
let invoice = builder.allow_mpp().build_and_sign(secp_ctx)?;
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.