Remove unnecessary per_peer_state branch

After `channels` are now stored in the `per_peer_state`, some logic can
be simplified and extra accessing of the `per_peer_state` can be
removed.

Author: ViktorTigerstrom

lightning/src/ln/channelmanager.rs

@@ -1779,7 +1779,7 @@ impl<Signer: Sign, M: Deref, T: Deref, K: Deref, F: Deref, L: Deref> ChannelMana
 						channel_id: (*channel_id).clone(),
 						counterparty: ChannelCounterparty {
 							node_id: channel.get_counterparty_node_id(),
-							features: InitFeatures::empty(),
+							features: peer_state.latest_features.clone(),
 							unspendable_punishment_reserve: to_remote_reserve_satoshis,
 							forwarding_info: channel.counterparty_forwarding_info(),
 							// Ensures that we have actually received the `htlc_minimum_msat` value
@@ -1817,12 +1817,6 @@ impl<Signer: Sign, M: Deref, T: Deref, K: Deref, F: Deref, L: Deref> ChannelMana
 				}
 			}
 		}
-		let per_peer_state = self.per_peer_state.read().unwrap();
-		for chan in res.iter_mut() {
-			if let Some(peer_state) = per_peer_state.get(&chan.counterparty.node_id) {
-				chan.counterparty.features = peer_state.lock().unwrap().latest_features.clone();
-			}
-		}
 		res
 	}