f rewrite comment again

Author: TheBlueMatt

lightning/src/ln/channel.rs

@@ -3180,9 +3180,12 @@ impl<Signer: WriteableEcdsaChannelSigner> Channel<Signer> {
 				let mut reason = OutboundHTLCOutcome::Success(None);
 				mem::swap(outcome, &mut reason);
 				if let OutboundHTLCOutcome::Success(Some(preimage)) = reason {
-					// While failing to handle the `Success(None)` case here could result in
-					// forgetting some HTLC claims, it is only reachable for HTLCs claimed on LDK
-					// 0.0.104 or before, then upgrading LDK before the HTLC is fully resolved.
+					// If a user (a) receives an HTLC claim using LDK 0.0.104 or before, then (b)
+					// upgrades to LDK 0.0.114 or later before the HTLC is fully resolved, we could
+					// have a `Success(None)` reason. In this case we could forget some HTLC
+					// claims, but such an upgrade is unlikely and including claimed HTLCs here
+					// fixes a bug which the user was exposed to on 0.0.104 when they started the
+					// claim anyway.
 					claimed_htlcs.push((SentHTLCId::from_source(&htlc.source), preimage));
 				}
 				htlc.state = OutboundHTLCState::AwaitingRemoteRevokeToRemove(reason);