From f9e37f33ce2d8b463a0bcbe7189c9bc5b36530b7 Mon Sep 17 00:00:00 2001 From: Yuval Kogman Date: Fri, 7 Feb 2020 19:10:31 +0000 Subject: doc: IsFinalTx comment about nSequence & OP_CLTV It's somewhat surprising that a transaction's nLockTime field is ignored when all nSequence fields are final, so this change aims to clarify this behavior and cross reference relevant details of OP_CHECKLOCKTIMEVERIFY. --- src/script/interpreter.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/script/interpreter.cpp') diff --git a/src/script/interpreter.cpp b/src/script/interpreter.cpp index abc0625bb1..6682d405c8 100644 --- a/src/script/interpreter.cpp +++ b/src/script/interpreter.cpp @@ -1741,9 +1741,9 @@ bool GenericTransactionSignatureChecker::CheckLockTime(const CScriptNum& nLoc if (nLockTime > (int64_t)txTo->nLockTime) return false; - // Finally the nLockTime feature can be disabled and thus - // CHECKLOCKTIMEVERIFY bypassed if every txin has been - // finalized by setting nSequence to maxint. The + // Finally the nLockTime feature can be disabled in IsFinalTx() + // and thus CHECKLOCKTIMEVERIFY bypassed if every txin has + // been finalized by setting nSequence to maxint. The // transaction would be allowed into the blockchain, making // the opcode ineffective. // -- cgit v1.2.3