f no need to borrow deref

Author: domZippilli

lightning/src/util/persist.rs

@@ -402,7 +402,7 @@ where
 	/// [`io::ErrorKind::NotFound`] variant correctly. For more information, please see the
 	/// documentation for [`MonitorUpdatingPersister`].
 	pub fn read_all_channel_monitors_with_updates<B: Deref, F: Deref + Clone>(
-		&self, broadcaster: &B, fee_estimator: F,
+		&self, broadcaster: B, fee_estimator: F,
 	) -> io::Result<Vec<(BlockHash, ChannelMonitor<<SP::Target as SignerProvider>::Signer>)>>
 	where
 		ES::Target: EntropySource + Sized,
@@ -417,7 +417,7 @@ where
 		let mut res = Vec::with_capacity(monitor_list.len());
 		for monitor_key in monitor_list {
 			res.push(self.read_channel_monitor_with_updates(
-				broadcaster,
+				&broadcaster,
 				&fee_estimator,
 				monitor_key,
 			)?)
@@ -434,7 +434,7 @@ where
 	/// Loading a large number of monitors will be faster if done in parallel. You can use this
 	/// function to accomplish this. Take care to limit the number of parallel readers.
 	pub fn read_channel_monitor_with_updates<B: Deref, F: Deref + Clone>(
-		&self, broadcaster: &B, fee_estimator: &F, monitor_key: String,
+		&self, broadcaster: B, fee_estimator: F, monitor_key: String,
 	) -> io::Result<(BlockHash, ChannelMonitor<<SP::Target as SignerProvider>::Signer>)>
 	where
 		ES::Target: EntropySource + Sized,
@@ -460,8 +460,7 @@ where
 				Err(err) => return Err(err),
 			};
 
-			monitor
-				.update_monitor(&update, broadcaster, fee_estimator.clone(), &self.logger)
+			monitor.update_monitor(&update, &broadcaster, fee_estimator.clone(), &self.logger)
 				.map_err(|e| {
 					log_error!(
 						self.logger,
@@ -711,8 +710,8 @@ where
 		&self, funding_txo: OutPoint, update: Option<&ChannelMonitorUpdate>,
 		monitor: &ChannelMonitor<ChannelSigner>, monitor_update_call_id: MonitorUpdateId,
 	) -> chain::ChannelMonitorUpdateStatus {
-		// IMPORTANT: monitor_update_call_id: [`MonitorUpdateId`] is not to be confused with
-		// [`ChannelMonitorUpdate`]'s update_id.
+		// IMPORTANT: monitor_update_call_id: MonitorUpdateId is not to be confused with
+		// ChannelMonitorUpdate's update_id.
 		if let Some(update) = update {
 			if update.update_id != CLOSED_CHANNEL_UPDATE_ID
 				&& update.update_id % self.maximum_pending_updates != 0