Remove unnecessary bounds on LSPS2ServiceHandler

TheBlueMatt · TheBlueMatt · commit f3085bbe336e · 2025-01-29T22:47:52.000Z
There's no reason to require a `clone`able `ChannelManager`
reference (even though they often are).
diff --git a/lightning-liquidity/src/lsps2/service.rs b/lightning-liquidity/src/lsps2/service.rs
@@ -550,7 +550,7 @@ macro_rules! get_or_insert_peer_state_entry {
 }
 
 /// The main object allowing to send and receive LSPS2 messages.
-pub struct LSPS2ServiceHandler<CM: Deref + Clone>
+pub struct LSPS2ServiceHandler<CM: Deref>
 where
 	CM::Target: AChannelManager,
 {
@@ -564,7 +564,7 @@ where
 	config: LSPS2ServiceConfig,
 }
 
-impl<CM: Deref + Clone> LSPS2ServiceHandler<CM>
+impl<CM: Deref> LSPS2ServiceHandler<CM>
 where
 	CM::Target: AChannelManager,
 {
@@ -1345,7 +1345,7 @@ where
 	}
 }
 
-impl<CM: Deref + Clone> ProtocolMessageHandler for LSPS2ServiceHandler<CM>
+impl<CM: Deref> ProtocolMessageHandler for LSPS2ServiceHandler<CM>
 where
 	CM::Target: AChannelManager,
 {

Original file line number	Diff line number	Diff line change
`@@ -550,7 +550,7 @@ macro_rules! get_or_insert_peer_state_entry {`
`550`	`550`	`}`
`551`	`551`
`552`	`552`	`/// The main object allowing to send and receive LSPS2 messages.`
`553`		`-pub struct LSPS2ServiceHandler<CM: Deref + Clone>`
	`553`	`+pub struct LSPS2ServiceHandler<CM: Deref>`
`554`	`554`	`where`
`555`	`555`	`CM::Target: AChannelManager,`
`556`	`556`	`{`
`@@ -564,7 +564,7 @@ where`
`564`	`564`	`config: LSPS2ServiceConfig,`
`565`	`565`	`}`
`566`	`566`
`567`		`-impl<CM: Deref + Clone> LSPS2ServiceHandler<CM>`
	`567`	`+impl<CM: Deref> LSPS2ServiceHandler<CM>`
`568`	`568`	`where`
`569`	`569`	`CM::Target: AChannelManager,`
`570`	`570`	`{`
`@@ -1345,7 +1345,7 @@ where`
`1345`	`1345`	`}`
`1346`	`1346`	`}`
`1347`	`1347`
`1348`		`-impl<CM: Deref + Clone> ProtocolMessageHandler for LSPS2ServiceHandler<CM>`
	`1348`	`+impl<CM: Deref> ProtocolMessageHandler for LSPS2ServiceHandler<CM>`
`1349`	`1349`	`where`
`1350`	`1350`	`CM::Target: AChannelManager,`
`1351`	`1351`	`{`