f struct variant enum cause no doc comments

Author: TheBlueMatt

lightning/src/ln/channelmanager.rs

@@ -509,14 +509,18 @@ enum BackgroundEvent {
 	/// Note that any such events are lost on shutdown, so in general they must be updates which
 	/// are regenerated on startup.
 	ClosingMonitorUpdateRegeneratedOnStartup((OutPoint, ChannelMonitorUpdate)),
-	/// Handle a ChannelMonitorUpdate.
+	/// Handle a ChannelMonitorUpdate which may or may not close the channel. In general this
+	/// should be used rather than [`Self::ClosingMonitorUpdateRegeneratedOnStartup`], however in
+	/// cases where the `counterparty_node_id` is not available as the channel has closed from a
+	/// [`ChannelMonitor`] error the other variant is acceptable.
 	///
 	/// Note that any such events are lost on shutdown, so in general they must be updates which
 	/// are regenerated on startup.
-	MonitorUpdateRegeneratedOnStartup((
-		/// The counterparty node_id for the channel being updated.
-		PublicKey,
-		OutPoint, ChannelMonitorUpdate)),
+	MonitorUpdateRegeneratedOnStartup {
+		counterparty_node_id: PublicKey,
+		funding_txo: OutPoint,
+		update: ChannelMonitorUpdate
+	},
 }
 
 #[derive(Debug)]
@@ -3792,7 +3796,7 @@ where
 					// monitor updating completing.
 					let _ = self.chain_monitor.update_channel(funding_txo, &update);
 				},
-				BackgroundEvent::MonitorUpdateRegeneratedOnStartup((_, funding_txo, update)) => {
+				BackgroundEvent::MonitorUpdateRegeneratedOnStartup { funding_txo, update, .. } => {
 					// The channel has already been closed, so no use bothering to care about the
 					// monitor updating completing.
 					let _ = self.chain_monitor.update_channel(funding_txo, &update);
@@ -5713,8 +5717,9 @@ where
 					assert!(should_broadcast);
 				} else { unreachable!(); }
 				self.pending_background_events.lock().unwrap().push(
-					BackgroundEvent::MonitorUpdateRegeneratedOnStartup(
-						(counterparty_node_id, funding_txo, update)));
+					BackgroundEvent::MonitorUpdateRegeneratedOnStartup {
+						counterparty_node_id, funding_txo, update
+					});
 			}
 			self.finish_force_close_channel(failure);
 		}
@@ -7787,8 +7792,10 @@ where
 					log_error!(args.logger, " The ChannelMonitor for channel {} is at update_id {} but the ChannelManager is at update_id {}.",
 						log_bytes!(channel.channel_id()), monitor.get_latest_update_id(), channel.get_latest_monitor_update_id());
 					let (monitor_update, mut new_failed_htlcs) = channel.force_shutdown(true);
-					if let Some(monitor_update) = monitor_update {
-						pending_background_events.push(BackgroundEvent::MonitorUpdateRegeneratedOnStartup(monitor_update));
+					if let Some((counterparty_node_id, funding_txo, update)) = monitor_update {
+						pending_background_events.push(BackgroundEvent::MonitorUpdateRegeneratedOnStartup {
+							counterparty_node_id, funding_txo, update
+						});
 					}
 					failed_htlcs.append(&mut new_failed_htlcs);
 					channel_closures.push_back((events::Event::ChannelClosed {