Make LSPSDateTime Copy rather than explicitely _clone_ing

Author: martinsaposnic

lightning-liquidity/src/lsps0/ser.rs

@@ -218,7 +218,7 @@ impl wire::Type for RawLSPSMessage {
 pub struct LSPSRequestId(pub String);
 
 /// An object representing datetimes as described in bLIP-50 / LSPS0.
-#[derive(Clone, Debug, PartialEq, Eq, Hash, Deserialize, Serialize)]
+#[derive(Clone, Debug, Copy, PartialEq, Eq, Hash, Deserialize, Serialize)]
 #[serde(transparent)]
 pub struct LSPSDateTime(pub chrono::DateTime<chrono::Utc>);
 

lightning-liquidity/src/lsps1/service.rs

@@ -262,7 +262,7 @@ where
 							let order_id = self.generate_order_id();
 							let channel = OutboundCRChannel::new(
 								params.order.clone(),
-								created_at.clone(),
+								created_at,
 								order_id.clone(),
 								payment.clone(),
 							);

lightning-liquidity/src/lsps5/service.rs

@@ -172,7 +172,7 @@ where
 			Entry::Occupied(mut entry) => {
 				no_change = entry.get().url == params.webhook;
 				let (last_used, last_notification_sent) = if no_change {
-					(entry.get().last_used.clone(), entry.get().last_notification_sent.clone())
+					(entry.get().last_used, entry.get().last_notification_sent.clone())
 				} else {
 					(now, new_hash_map())
 				};
@@ -392,11 +392,11 @@ where
 			if webhook
 				.last_notification_sent
 				.get(&notification.method)
-				.map(|last_sent| now.clone().duration_since(&last_sent))
+				.map(|last_sent| now.duration_since(&last_sent))
 				.map_or(true, |duration| duration >= self.config.notification_cooldown_hours)
 			{
-				webhook.last_notification_sent.insert(notification.method.clone(), now.clone());
-				webhook.last_used = now.clone();
+				webhook.last_notification_sent.insert(notification.method.clone(), now);
+				webhook.last_used = now;
 				self.send_notification(
 					client_id,
 					app_name.clone(),