Add comment describing when a completion action can be discarded 2023-07-all-compl-actions
authorMatt Corallo <git@bluematt.me>
Wed, 5 Jul 2023 17:06:19 +0000 (17:06 +0000)
committerMatt Corallo <git@bluematt.me>
Wed, 12 Jul 2023 20:53:10 +0000 (20:53 +0000)
commit550cf91439a2aa2befe89704e13c781800e8612e
tree884912309437d0fafbe56693d1cc262e624e0308
parentf9521a4bdaa4cfdaf08f6797d1a84e96475c9f49
Add comment describing when a completion action can be discarded

In an older PR a reviewer had asked why the discarding of a channel
being blocked on another monitor update is okay if the blocked
channel has since closed. At the time, this was not actually okay -
the monitor updates in the channel weren't moved to the
`ChannelManager` on close so the whole pipeline was busted, but
with the changes in 4041f0899f86eaf6a0a4576a91918fa54026ac46 the
handling of channel closes with pending monitor updates is now
correct, and so is the existing code block.
lightning/src/ln/channelmanager.rs