f fix comments

Author: TheBlueMatt

lightning/src/ln/channelmanager.rs

@@ -3599,9 +3599,10 @@ impl<Signer: Sign, M: Deref, T: Deref, K: Deref, F: Deref, L: Deref> ChannelMana
 			let updates = channel.get_mut().monitor_updating_restored(&self.logger, self.get_our_node_id(), self.genesis_hash, self.best_block.read().unwrap().height());
 			let channel_update = if updates.funding_locked.is_some() && channel.get().is_usable() {
 				// We only send a channel_update in the case where we are just now sending a
-				// funding_locked and the channel is in a usable state. Further, we rely on the
-				// normal announcement_signatures process to send a channel_update for public
-				// channels, only generating a unicast channel_update if this is a private channel.
+				// funding_locked and the channel is in a usable state. We may re-send a
+				// channel_update later through the announcement_signatures process for public
+				// channels, but there's no reason not to just inform our counterparty of our fees
+				// now.
 				Some(events::MessageSendEvent::SendChannelUpdate {
 					node_id: channel.get().get_counterparty_node_id(),
 					msg: self.get_channel_update_for_unicast(channel.get()).unwrap(),
@@ -3782,6 +3783,11 @@ impl<Signer: Sign, M: Deref, T: Deref, K: Deref, F: Deref, L: Deref> ChannelMana
 						msg: announcement_sigs,
 					});
 				} else if chan.get().is_usable() {
+					// If we're sending an announcement_signatures, we'll send the (public)
+					// channel_update after sending a channel_announcement when we receive our
+					// counterparty's announcement_signatures. Thus, we only bother to send a
+					// channel_update here if the channel is not public, i.e. we're not sending an
+					// announcement_signatures.
 					log_trace!(self.logger, "Sending private initial channel_update for our counterparty on channel {}", log_bytes!(chan.get().channel_id()));
 					channel_state.pending_msg_events.push(events::MessageSendEvent::SendChannelUpdate {
 						node_id: counterparty_node_id.clone(),