f add reasoning for wrapper

TheBlueMatt · TheBlueMatt · commit 28b743032a14 · 2021-10-28T18:53:59.000Z
diff --git a/lightning/src/ln/peer_handler.rs b/lightning/src/ln/peer_handler.rs
@@ -493,6 +493,10 @@ impl<Descriptor: SocketDescriptor, RM: Deref, L: Deref> PeerManager<Descriptor,
 	}
 }
 
+/// A simple wrapper that optionally prints " from <pubkey>" for an optional pubkey.
+/// This works around `format!()` taking a reference to each argument, preventing
+/// `if let Some(node_id) = peer.their_node_id { format!(.., node_id) } else { .. }` from compiling
+/// due to lifetime errors.
 struct OptionalFromDebugger<'a>(&'a Option<PublicKey>);
 impl core::fmt::Display for OptionalFromDebugger<'_> {
 	fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> Result<(), core::fmt::Error> {

Original file line number	Diff line number	Diff line change
`@@ -493,6 +493,10 @@ impl<Descriptor: SocketDescriptor, RM: Deref, L: Deref> PeerManager<Descriptor,`
`493`	`493`	`}`
`494`	`494`	`}`
`495`	`495`
	`496`	`+/// A simple wrapper that optionally prints " from <pubkey>" for an optional pubkey.`
	`497`	+/// This works around `format!()` taking a reference to each argument, preventing
	`498`	+/// `if let Some(node_id) = peer.their_node_id { format!(.., node_id) } else { .. }` from compiling
	`499`	`+/// due to lifetime errors.`
`496`	`500`	`struct OptionalFromDebugger<'a>(&'a Option<PublicKey>);`
`497`	`501`	`impl core::fmt::Display for OptionalFromDebugger<'_> {`
`498`	`502`	`fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> Result<(), core::fmt::Error> {`