f fix compile

Author: TheBlueMatt

lightning-invoice/src/utils.rs

@@ -381,13 +381,13 @@ where
 /// channel with a higher or equal inbound capacity than `min_inbound_capacity_msat` exists
 /// * If any public channel exists, the returned `RouteHint`s will be empty, and the sender will
 /// need to find the path by looking at the public channels instead
-fn filter_channels(channels: Vec<ChannelDetails>, min_inbound_capacity_msat: Option<u64>) -> Vec<RouteHint>{
-	let mut filtered_channels: HashMap<PublicKey, &ChannelDetails> = HashMap::new();
+fn filter_channels(mut channels: Vec<ChannelDetails>, min_inbound_capacity_msat: Option<u64>) -> Vec<RouteHint>{
+	let mut filtered_channels: HashMap<PublicKey, ChannelDetails> = HashMap::new();
 	let min_inbound_capacity = min_inbound_capacity_msat.unwrap_or(0);
 	let mut min_capacity_channel_exists = false;
 	let mut online_channel_exists = false;
 
-	for channel in channels.filter(|chan| chan.is_channel_ready) {
+	for channel in channels.drain(..).filter(|chan| chan.is_channel_ready) {
 		if channel.get_inbound_payment_scid().is_none() || channel.counterparty.forwarding_info.is_none() {
 			continue;
 		}
@@ -418,7 +418,7 @@ fn filter_channels(channels: Vec<ChannelDetails>, min_inbound_capacity_msat: Opt
 		}
 	}
 
-	let route_hint_from_channel = |channel: &ChannelDetails| {
+	let route_hint_from_channel = |channel: ChannelDetails| {
 		let forwarding_info = channel.counterparty.forwarding_info.as_ref().unwrap();
 		RouteHint(vec![RouteHintHop {
 			src_node_id: channel.counterparty.node_id,
@@ -447,7 +447,7 @@ fn filter_channels(channels: Vec<ChannelDetails>, min_inbound_capacity_msat: Opt
 				channel.is_usable
 			} else { true }
 		})
-		.map(|(_counterparty_id, channel)| route_hint_from_channel(&channel))
+		.map(|(_counterparty_id, channel)| route_hint_from_channel(channel))
 		.collect::<Vec<RouteHint>>()
 }