Pass pending events to outbound_payments::abandon_payment

This makes it uniform with the outbound payment methods that generate events
and set us up for abandoning payments on behalf of the user.

Author: valentinewallace

lightning/src/ln/channelmanager.rs

@@ -2606,9 +2606,7 @@ where
 	/// [`Event::PaymentSent`]: events::Event::PaymentSent
 	pub fn abandon_payment(&self, payment_id: PaymentId) {
 		let _persistence_guard = PersistenceNotifierGuard::notify_on_drop(&self.total_consistency_lock, &self.persistence_notifier);
-		if let Some(payment_failed_ev) = self.pending_outbound_payments.abandon_payment(payment_id) {
-			self.pending_events.lock().unwrap().push(payment_failed_ev);
-		}
+		self.pending_outbound_payments.abandon_payment(payment_id, &self.pending_events);
 	}
 
 	/// Send a spontaneous payment, which is a payment that does not require the recipient to have

lightning/src/ln/outbound_payment.rs

@@ -1130,21 +1130,21 @@ impl OutboundPayments {
 		if let Some(ev) = pending_retry_ev { pending_events.push(ev); }
 	}
 
-	pub(super) fn abandon_payment(&self, payment_id: PaymentId) -> Option<events::Event> {
-		let mut failed_ev = None;
+	pub(super) fn abandon_payment(&self, payment_id: PaymentId,
+		pending_events: &Mutex<Vec<events::Event>>
+	) {
 		let mut outbounds = self.pending_outbound_payments.lock().unwrap();
 		if let hash_map::Entry::Occupied(mut payment) = outbounds.entry(payment_id) {
 			if let Ok(()) = payment.get_mut().mark_abandoned() {
 				if payment.get().remaining_parts() == 0 {
-					failed_ev = Some(events::Event::PaymentFailed {
+					pending_events.lock().unwrap().push(events::Event::PaymentFailed {
 						payment_id,
 						payment_hash: payment.get().payment_hash().expect("PendingOutboundPayments::RetriesExceeded always has a payment hash set"),
 					});
 					payment.remove();
 				}
 			}
 		}
-		failed_ev
 	}
 
 	#[cfg(test)]