Remove unused ClosedMonitorUpdateRegeneratedOnStartup variant

This commit is contained in:
Wilmer Paulino 2025-02-06 15:28:38 -08:00
parent 222cd42186
commit d9721e6f5e
No known key found for this signature in database
GPG key ID: 634FE5FC544DCA31

View file

@ -1062,22 +1062,9 @@ impl ClaimablePayments {
/// [`ChannelMonitorUpdate`]s are applied.
#[derive(Debug)]
enum BackgroundEvent {
/// Handle a ChannelMonitorUpdate which closes the channel or for an already-closed channel.
/// This is only separated from [`Self::MonitorUpdateRegeneratedOnStartup`] as for truly
/// ancient [`ChannelMonitor`]s that haven't seen an update since LDK 0.0.118 we may not have
/// the counterparty node ID available.
///
/// Note that any such events are lost on shutdown, so in general they must be updates which
/// are regenerated on startup.
ClosedMonitorUpdateRegeneratedOnStartup((OutPoint, ChannelId, ChannelMonitorUpdate)),
/// Handle a ChannelMonitorUpdate which may or may not close the channel and may unblock the
/// channel to continue normal operation.
///
/// In general this should be used rather than
/// [`Self::ClosedMonitorUpdateRegeneratedOnStartup`], 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.
///
/// Any such events that exist in [`ChannelManager::pending_background_events`] will *also* be
/// tracked in [`PeerState::in_flight_monitor_updates`].
///
@ -6421,11 +6408,6 @@ where
for event in background_events.drain(..) {
match event {
BackgroundEvent::ClosedMonitorUpdateRegeneratedOnStartup((_funding_txo, channel_id, update)) => {
// The channel has already been closed, so no use bothering to care about the
// monitor updating completing.
let _ = self.chain_monitor.update_channel(channel_id, &update);
},
BackgroundEvent::MonitorUpdateRegeneratedOnStartup { counterparty_node_id, funding_txo, channel_id, update } => {
self.apply_post_close_monitor_update(counterparty_node_id, channel_id, funding_txo, update);
},