Correct comment in onchaintx.rs
authorMatt Corallo <git@bluematt.me>
Wed, 4 Mar 2020 22:53:16 +0000 (17:53 -0500)
committerMatt Corallo <git@bluematt.me>
Thu, 5 Mar 2020 02:06:57 +0000 (21:06 -0500)
This comment was stale and referred to a previous implementation
of #462, which changed before it was merged.

lightning/src/ln/onchaintx.rs

index c731bc079b657131df2a5fe7994ca6dc967d3f1f..e456b7164b30a1e2f776dc273db29fbaf110c34e 100644 (file)
@@ -487,8 +487,8 @@ impl OnchainTxHandler {
                let mut aggregated_soonest = ::std::u32::MAX;
                let mut spendable_outputs = Vec::new();
 
-               // Try to aggregate outputs if they're 1) belong to same parent tx, 2) their
-               // timelock expiration isn't imminent (<= CLTV_SHARED_CLAIM_BUFFER).
+               // Try to aggregate outputs if their timelock expiration isn't imminent (absolute_timelock
+               // <= CLTV_SHARED_CLAIM_BUFFER) and they don't require an immediate nLockTime (aggregable).
                for req in claimable_outpoints {
                        // Don't claim a outpoint twice that would be bad for privacy and may uselessly lock a CPFP input for a while
                        if let Some(_) = self.claimable_outpoints.get(&req.outpoint) { log_trace!(self, "Bouncing off outpoint {}:{}, already registered its claiming request", req.outpoint.txid, req.outpoint.vout); } else {