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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nakihito created this revision.Jan 14 2020, 00:16
Owners added a reviewer: Restricted Owners Package.Jan 14 2020, 00:16
Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 14 2020, 00:16
nakihito planned changes to this revision.Jan 14 2020, 00:16
nakihito requested review of this revision.Jan 14 2020, 01:21
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
nakihito requested review of this revision.Jan 14 2020, 19:40

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.

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