Page MenuHomePhabricator

VerifyScript: accumulate ScriptExecutionMetrics and return them
ClosedPublic

Authored by markblundeberg on Jan 5 2020, 04:11.

Details

Summary

This exposes the input-total metrics for VerifyScript callers, so it
can be used in CheckInputs (-> mempool tracking, scriptcache, connectblock
checking).

Test Plan

ninja check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
sigchecksD
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8742
Build 15468: Default Diff Build & Tests
Build 15467: arc lint + arc unit

Event Timeline

markblundeberg retitled this revision from add ScriptExecutionMetrics return-by-ref to VerifyScript to VerifyScript: accumulate ScriptExecutionMetrics and return them.
markblundeberg edited the summary of this revision. (Show Details)

absorb things from D4617 and flip dependency

deadalnix requested changes to this revision.Jan 5 2020, 09:38
deadalnix added inline comments.
src/script/interpreter.cpp
1709 ↗(On Diff #15162)

This is a pretty bad API. You can instead create a local variable, use that, and merge it at the end.

This revision now requires changes to proceed.Jan 5 2020, 09:38

copy to outparam rather than reset-modifying outparam

This revision is now accepted and ready to land.Jan 8 2020, 13:23