Merge pull request #3605 from vincenzopalazzo/macros/fix-warnings

TheBlueMatt · web-flow · commit 2d2c542a9095 · 2025-02-16T16:32:28.000Z
[RFC] remove the cfg test that it is not needed in a error code path
diff --git a/lightning/src/ln/channelmanager.rs b/lightning/src/ln/channelmanager.rs
@@ -8310,7 +8310,6 @@ This indicates a bug inside LDK. Please report this error at https://github.com/
 			), ChannelId([0; 32])));
 		}
 
-		#[cfg(not(test))]
 		if msg.data.len() > MAX_PEER_STORAGE_SIZE {
 			log_debug!(logger, "Sending warning to peer and ignoring peer storage request from {} as its over 1KiB", log_pubkey!(counterparty_node_id));
 

Original file line number	Diff line number	Diff line change
`@@ -8310,7 +8310,6 @@ This indicates a bug inside LDK. Please report this error at https://github.com/`
`8310`	`8310`	`), ChannelId([0; 32])));`
`8311`	`8311`	`}`
`8312`	`8312`
`8313`		`- #[cfg(not(test))]`
`8314`	`8313`	`if msg.data.len() > MAX_PEER_STORAGE_SIZE {`
`8315`	`8314`	`log_debug!(logger, "Sending warning to peer and ignoring peer storage request from {} as its over 1KiB", log_pubkey!(counterparty_node_id));`
`8316`	`8315`