f names

Author: TheBlueMatt

lightning/src/routing/router.rs

@@ -994,21 +994,21 @@ where L::Target: Logger {
 						 (amount_to_transfer_over_msat < $next_hops_path_htlc_minimum_msat &&
 						  recommended_value_msat > $next_hops_path_htlc_minimum_msat));
 
-					let channel_failed_on_this_payment =
+					let payment_failed_on_this_channel =
 						payment_params.previously_failed_channels.contains(&short_channel_id);
 
 					// If HTLC minimum is larger than the amount we're going to transfer, we shouldn't
 					// bother considering this channel. If retrying with recommended_value_msat may
 					// allow us to hit the HTLC minimum limit, set htlc_minimum_limit so that we go
 					// around again with a higher amount.
 					if contributes_sufficient_value && doesnt_exceed_max_path_length &&
-						doesnt_exceed_cltv_delta_limit && !channel_failed_on_this_payment &&
+						doesnt_exceed_cltv_delta_limit && !payment_failed_on_this_channel &&
 						may_overpay_to_meet_path_minimum_msat
 					{
 						hit_minimum_limit = true;
 					} else if contributes_sufficient_value && doesnt_exceed_max_path_length &&
 						doesnt_exceed_cltv_delta_limit && over_path_minimum_msat &&
-						!channel_failed_on_this_payment
+						!payment_failed_on_this_channel
 					{
 						// Note that low contribution here (limited by available_liquidity_msat)
 						// might violate htlc_minimum_msat on the hops which are next along the
@@ -5510,7 +5510,7 @@ mod tests {
 
 	#[test]
 	fn avoids_recently_failed_paths() {
-		// Ensure that the router always avoids all of the `recently_failed_channels` channels by
+		// Ensure that the router always avoids all of the `previously_failed_channels` channels by
 		// randomly inserting channels into it until we can't find a route anymore.
 		let (secp_ctx, network, _, _, logger) = build_graph();
 		let (_, our_id, _, nodes) = get_nodes(&secp_ctx);