Page MenuHomePhabricator

[backport#13868] Remove unused fScriptChecks parameter from CheckInputs
ClosedPublic

Authored by majcosta on Sep 28 2020, 23:16.

Details

Summary

Remove unused fScriptChecks parameter from CheckInputs (Matt Corallo)

Pull request description:

fScriptChecks = false just short-circuits the entire function, so
passing it in is entirely useless.

This is extracted from #13233 /cc TheBlueMatt.

Recommend reviewing with `git show --ignore-all-space`, i.e.:
https://github.com/bitcoin/bitcoin/pull/13868/files?w=1

Backport of Core PR13868

Test Plan
ninja check check-functional

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

This revision is now accepted and ready to land.Sep 28 2020, 23:30