f minor tweaks and grammar

Author: TheBlueMatt

lightning/src/chain/channelmonitor.rs

@@ -2931,7 +2931,7 @@ impl<Signer: Sign> ChannelMonitorImpl<Signer> {
 						htlc_value_satoshis,
 					}));
 					self.htlcs_resolved_on_chain.push(IrrevocablyResolvedHTLC {
-						commitment_tx_output_idx: commitment_tx_output_idx, resolving_txid: Some(entry.txid),
+						commitment_tx_output_idx, resolving_txid: Some(entry.txid),
 						payment_preimage: None,
 					});
 				},

lightning/src/ln/payment_tests.rs

@@ -589,6 +589,7 @@ fn do_test_completed_payment_not_retryable_on_reload(use_dust: bool) {
 
 	let mut nodes = create_network(3, &node_cfgs, &node_chanmgrs);
 
+	// Because we set nodes[1] to manually accept channels, just open a 0-conf channel.
 	let (funding_tx, chan_id) = open_zero_conf_channel(&nodes[0], &nodes[1], None);
 	confirm_transaction(&nodes[0], &funding_tx);
 	confirm_transaction(&nodes[1], &funding_tx);
@@ -731,7 +732,7 @@ fn do_test_completed_payment_not_retryable_on_reload(use_dust: bool) {
 	assert_eq!(nodes[0].node.list_usable_channels().len(), 1);
 
 	// If we attempt to retry prior to the HTLC-Timeout (or commitment transaction, for dust HTLCs)
-	// confirming, we will fail as its considered still-pending...
+	// confirming, we will fail as it's considered still-pending...
 	let (new_route, _, _, _) = get_route_and_payment_hash!(nodes[0], nodes[2], if use_dust { 1_000 } else { 1_000_000 });
 	assert!(nodes[0].node.retry_payment(&new_route, payment_id).is_err());
 	assert!(nodes[0].node.get_and_clear_pending_msg_events().is_empty());
@@ -756,7 +757,8 @@ fn do_test_completed_payment_not_retryable_on_reload(use_dust: bool) {
 	reload_node!(second_new_chain_monitor, second_nodes_0_deserialized, second_persister);
 	reconnect_nodes(&nodes[0], &nodes[1], (true, true), (0, 0), (0, 0), (0, 0), (0, 0), (0, 0), (false, false));
 
-	// Now resend the payment, delivering the HTLC through to nodes[1] XXX
+	// Now resend the payment, delivering the HTLC and actually claiming it this time. This ensures
+	// the payment is not (spuriously) listed as still pending.
 	assert!(nodes[0].node.retry_payment(&new_route, payment_id).is_ok());
 	check_added_monitors!(nodes[0], 1);
 	pass_along_route(&nodes[0], &[&[&nodes[1], &nodes[2]]], if use_dust { 1_000 } else { 1_000_000 }, payment_hash, payment_secret);
@@ -771,6 +773,7 @@ fn do_test_completed_payment_not_retryable_on_reload(use_dust: bool) {
 	get_monitor!(nodes[0], chan_id_3).write(&mut chan_1_monitor_serialized).unwrap();
 	nodes_0_serialized = nodes[0].node.encode();
 
+	// Ensure that after reload we cannot retry the payment.
 	reload_node!(third_new_chain_monitor, third_nodes_0_deserialized, third_persister);
 	reconnect_nodes(&nodes[0], &nodes[1], (false, false), (0, 0), (0, 0), (0, 0), (0, 0), (0, 0), (false, false));