Refactor Signature encoding check to prepare for Schnorr inclusion.
AbandonedPublic
Actions

Authored by Mengerian on Feb 1 2019, 14:33.

Details

Reviewers

markblundeberg
deadalnix

Group Reviewers

Restricted Project

Summary

This makes is easy to permit Schnorr signatures in
CheckRawSignatureEncoding when SCRIPT_ENABLE_SCHNORR
is active.

Depends on D2477

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

sighash-encoding

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4814
Build 7691: Bitcoin ABC Buildbot (legacy)
Build 7690: arc lint + arc unit

Event Timeline

Mengerian created this revision.Feb 1 2019, 14:33

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 1 2019, 14:33

Herald added a subscriber: schancel. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Mengerian mentioned this in D2477: Refactor signature encoding checks.Feb 1 2019, 14:40

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4814: Diff 7092.Feb 1 2019, 14:48

I combined the changes, this is now superseded by D2477

Revision Contents
Changeset List

Path

Size

src/

script/

sigencoding.cpp

59 lines

Diff 7092

View Options

src/script/sigencoding.cpp

Refactor Signature encoding check to prepare for Schnorr inclusion.AbandonedPublicActions