f Introduce ternary enum PaymentStoreUpdateResult

Author: tnull

src/payment/bolt11.rs

@@ -16,7 +16,7 @@ use crate::liquidity::LiquiditySource;
 use crate::logger::{log_error, log_info, LdkLogger, Logger};
 use crate::payment::store::{
 	LSPFeeLimits, PaymentDetails, PaymentDetailsUpdate, PaymentDirection, PaymentKind,
-	PaymentStatus, PaymentStore,
+	PaymentStatus, PaymentStore, PaymentStoreUpdateResult,
 };
 use crate::payment::SendingParameters;
 use crate::peer_store::{PeerInfo, PeerStore};
@@ -394,8 +394,8 @@ impl Bolt11Payment {
 	/// failed back, e.g., if the correct preimage can't be retrieved in time before the claim
 	/// deadline has been reached.
 	///
-	/// Will check that the payment is known and pending before failing the payment, and will
-	/// return an error otherwise.
+	/// Will check that the payment is known before failing the payment, and will return an error
+	/// otherwise.
 	///
 	/// [`receive_for_hash`]: Self::receive_for_hash
 	/// [`receive_variable_amount_for_hash`]: Self::receive_variable_amount_for_hash
@@ -409,8 +409,9 @@ impl Bolt11Payment {
 		};
 
 		match self.payment_store.update(&update) {
-			Ok(true) => (),
-			Ok(false) => {
+			Ok(PaymentStoreUpdateResult::EntryUpdated)
+			| Ok(PaymentStoreUpdateResult::EntryUnchanged) => (),
+			Ok(PaymentStoreUpdateResult::EntryNotFound) => {
 				log_error!(
 					self.logger,
 					"Failed to manually fail unknown payment with hash {}",
@@ -421,7 +422,7 @@ impl Bolt11Payment {
 			Err(e) => {
 				log_error!(
 					self.logger,
-					"Failed to manually fail unknown payment with hash {}: {}",
+					"Failed to manually fail payment with hash {}: {}",
 					payment_hash,
 					e
 				);

src/payment/store.rs

@@ -590,6 +590,12 @@ impl From<&PaymentDetails> for PaymentDetailsUpdate {
 	}
 }
 
+pub(crate) enum PaymentStoreUpdateResult {
+	EntryUpdated,
+	EntryUnchanged,
+	EntryNotFound,
+}
+
 pub(crate) struct PaymentStore<L: Deref>
 where
 	L::Target: LdkLogger,
@@ -670,17 +676,22 @@ where
 		self.payments.lock().unwrap().get(id).cloned()
 	}
 
-	pub(crate) fn update(&self, update: &PaymentDetailsUpdate) -> Result<bool, Error> {
-		let mut updated = false;
+	pub(crate) fn update(
+		&self, update: &PaymentDetailsUpdate,
+	) -> Result<PaymentStoreUpdateResult, Error> {
 		let mut locked_payments = self.payments.lock().unwrap();
 
 		if let Some(payment) = locked_payments.get_mut(&update.id) {
-			updated = payment.update(update);
+			let updated = payment.update(update);
 			if updated {
 				self.persist_info(&update.id, payment)?;
+				Ok(PaymentStoreUpdateResult::EntryUpdated)
+			} else {
+				Ok(PaymentStoreUpdateResult::EntryUnchanged)
 			}
+		} else {
+			Ok(PaymentStoreUpdateResult::EntryNotFound)
 		}
-		Ok(updated)
 	}
 
 	pub(crate) fn list_filter<F: FnMut(&&PaymentDetails) -> bool>(