HomePhabricator

clean up CheckInputs() policy-ban logic and comments

Description

clean up CheckInputs() policy-ban logic and comments

Summary:
I think this variable change makes it clearer what is actually going on.

Also, the comments here were out-of-date, referring to the now-mandatory flag STRICTENC, and referring to monolith upgrade.

Test Plan: make check and run functional tests

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Subscribers: teamcity, schancel

Differential Revision: https://reviews.bitcoinabc.org/D2511

Details

Provenance
Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>Authored on Feb 7 2019, 17:16
markblundebergPushed on Feb 7 2019, 18:19
Reviewer
Restricted Project
Differential Revision
D2511: clean up CheckInputs() policy-ban logic and comments
Parents
rABC06c232f38061: Merge #10712: Add change output if necessary to reduce excess fee
Branches
Unknown
Tags
Unknown

Event Timeline

Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com> committed rABC69ff7e29f430: clean up CheckInputs() policy-ban logic and comments (authored by Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>).Feb 7 2019, 18:19