Remove now-redundant abandon_payment in check_retry_payments

pay_internal now internally calls abandon_payment

Author: valentinewallace

lightning/src/ln/outbound_payment.rs

@@ -533,12 +533,8 @@ impl OutboundPayments {
 			}
 			core::mem::drop(outbounds);
 			if let Some((payment_id, payment_hash, route_params)) = retry_id_route_params {
-				if let Err(e) = self.pay_internal(payment_id, payment_hash, None, route_params, router, first_hops(), inflight_htlcs(), entropy_source, node_signer, best_block_height, logger, pending_events, &send_payment_along_path) {
-					log_info!(logger, "Errored retrying payment: {:?}", e);
-					// If we error on retry, there is no chance of the payment succeeding and no HTLCs have
-					// been irrevocably committed to, so we can safely abandon.
-					self.abandon_payment(payment_id, pending_events);
-				}
+				let retry_res = self.pay_internal(payment_id, payment_hash, None, route_params, router, first_hops(), inflight_htlcs(), entropy_source, node_signer, best_block_height, logger, pending_events, &send_payment_along_path);
+				log_info!(logger, "Result retrying payment {}: {:?}", log_bytes!(payment_id.0), retry_res);
 			} else { break }
 		}