Page MenuHomePhabricator

Merge #13429: Return the script type from Solver
ClosedPublic

Authored by nakihito on Jan 14 2020, 00:16.

Details

Summary

984d72ec659361d8c1a6f3c6864e839a807817a7 Return the script type from Solver (Ben Woosley)

Pull request description:

Because false is synonymous with TX_NONSTANDARD, this conveys the same
information and makes the handling explicitly based on script type,
simplifying each call site.

Prior to this change it was common for the return value to be ignored, or for the
return value and TX_NONSTANDARD to be redundantly handled.

Tree-SHA512: 31864f856b8cb75f4b782d12678070e8b1cfe9665c6f57cfb25e7ac8bcea8a22f9a78d7c8cf0101c841f2a612400666fb91798bffe88de856e98b873703b0965

Backport of Core PR13429 (Modified to fit our codebase)
https://github.com/bitcoin/bitcoin/pull/13429/

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR13429
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8912
Build 15798: Default Diff Build & Tests
Build 15797: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jan 14 2020, 00:16
deadalnix requested changes to this revision.Jan 14 2020, 01:24

The change in src/rpc/rawtransaction.cpp is missing.

This revision now requires changes to proceed.Jan 14 2020, 01:24

The change in src/rpc/rawtransaction.cpp is missing.

This change is for segwit code that was introduced here: https://github.com/bitcoin/bitcoin/pull/12321/files#diff-01aa7d1d32f1b9e5a836c9c411978918R597 so I skipped it.

This revision is now accepted and ready to land.Jan 15 2020, 15:18