Comments, style.

src/chain/input.cpp

@@ -35,6 +35,11 @@ namespace libbitcoin {
 namespace system {
 namespace chain {
 
+// Product overflows guarded by script size limit.
+static_assert(max_script_size < 
+    max_size_t / multisig_default_sigops / heavy_sigops_factor,
+    "input sigop overflow guard");
+
 // Constructors.
 // ----------------------------------------------------------------------------
 
@@ -319,11 +324,6 @@ bool input::extract_sigop_script(chain::script& out,
     return true;
 }
 
-// Product overflows guarded by script size limit.
-static_assert(max_script_size < 
-    max_size_t / multisig_default_sigops / heavy_sigops_factor,
-    "input sigop overflow guard");
-
 // TODO: Prior to block 79400 sigops were limited only by policy.
 // TODO: Create legacy sigops fork flag and pass here, return 0 if false.
 // TODO: this was an unbipped flag day soft fork, prior to BIP16/141.

src/chain/script.cpp

@@ -473,7 +473,7 @@ constexpr bool is_multiple_sigop(opcode code) NOEXCEPT
     return code == opcode::checkmultisig || code == opcode::checkmultisigverify;
 }
 
-// TODO: compute in script evaluation and add coinbase input scripts.
+// TODO: compute in or at script evaluation and add coinbase input scripts.
 // TODO: this precludes second deserialization of script for sigop counting.
 size_t script::signature_operations(bool accurate) const NOEXCEPT
 {

src/chain/transaction.cpp

@@ -1321,6 +1321,8 @@ code transaction::connect(const context& ctx) const NOEXCEPT
             return ec;
     }
 
+    // TODO: accumulate sigops from each connect result and add coinbase.
+    // TODO: return in override with out parameter. more impactful with segwit.
     return error::transaction_success;
 }