f use a constant

Author: TheBlueMatt

lightning/src/ln/channelmanager.rs

@@ -3309,6 +3309,11 @@ macro_rules! handle_monitor_update_completion {
 	} }
 }
 
+#[cfg(not(any(test, feature = "_externalize_tests")))]
+const SYNC_UPDATE: usize = 1;
+#[cfg(not(any(test, feature = "_externalize_tests")))]
+const ASYNC_UPDATE: usize = 2;
+
 macro_rules! handle_new_monitor_update {
 	($self: ident, $update_res: expr, $logger: expr, $channel_id: expr, _internal, $completed: expr) => { {
 		debug_assert!($self.background_events_processed_since_startup.load(Ordering::Acquire));
@@ -3320,7 +3325,7 @@ macro_rules! handle_new_monitor_update {
 			},
 			ChannelMonitorUpdateStatus::InProgress => {
 				#[cfg(not(any(test, feature = "_externalize_tests")))]
-				if $self.monitor_update_type.swap(1, Ordering::Relaxed) == 2 {
+				if $self.monitor_update_type.swap(ASYNC_UPDATE, Ordering::Relaxed) == SYNC_UPDATE {
 					panic!("Cannot use both ChannelMonitorUpdateStatus modes InProgress and Completed without restart");
 				}
 				log_debug!($logger, "ChannelMonitor update for {} in flight, holding messages until the update completes.",
@@ -3329,7 +3334,7 @@ macro_rules! handle_new_monitor_update {
 			},
 			ChannelMonitorUpdateStatus::Completed => {
 				#[cfg(not(any(test, feature = "_externalize_tests")))]
-				if $self.monitor_update_type.swap(2, Ordering::Relaxed) == 1 {
+				if $self.monitor_update_type.swap(SYNC_UPDATE, Ordering::Relaxed) == ASYNC_UPDATE {
 					panic!("Cannot use both ChannelMonitorUpdateStatus modes InProgress and Completed without restart");
 				}
 				$completed;