f comments tweaks

Author: TheBlueMatt

lightning/src/chain/channelmonitor.rs

@@ -1529,6 +1529,8 @@ impl<Signer: Sign> ChannelMonitor<Signer> {
 						// We should assert that funding_spend_confirmed is_some() here, but we
 						// have some unit tests which violate HTLC transaction CSVs entirely and
 						// would fail.
+						// TODO: Once tests all connect transactions at consensus-valid times, we
+						// should assert here like we do in `get_claimable_balances`.
 					} else if htlc.offered == $holder_commitment {
 						// If the payment was outbound, check if there's an HTLCUpdate
 						// indicating we have spent this HTLC with a timeout, claiming it back
@@ -1560,7 +1562,7 @@ impl<Signer: Sign> ChannelMonitor<Signer> {
 
 		// We're only concerned with the confirmation count of HTLC transactions, and don't
 		// actually care how many confirmations a commitment transaction may or may not have. Thus,
-		// we look for both a FundingSpendConfirmation event, or at funding_spend_confirmed.
+		// we look for both a FundingSpendConfirmation event, or a funding_spend_confirmed.
 		let confirmed_txid = us.funding_spend_confirmed.or_else(|| {
 			us.onchain_events_awaiting_threshold_conf.iter().find_map(|event| {
 				if let OnchainEvent::FundingSpendConfirmation { .. } = event.event {