Replaced nHashType with SigHashType class in wallet.cpp
ClosedPublic
Actions

Authored by jasonbcox on Jan 1 2018, 01:34.

Details

Reviewers

deadalnix
schancel

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rSTAGINGa55815a15e4b: Replaced nHashType with SigHashType class in wallet.cpp
rABCa55815a15e4b: Replaced nHashType with SigHashType class in wallet.cpp

Summary

Continue work replacing nHashType with SigHashType (see T61). Depends on D860.

Test Plan

Existing tests pass.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jasonbcox created this revision.Jan 1 2018, 01:34

Harbormaster completed remote builds in B1470: Diff 2271.Jan 1 2018, 01:34

Owners added a reviewer: Restricted Owners Package.Jan 1 2018, 01:34

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 1 2018, 01:34

Herald added a reviewer: Restricted Project. · View Herald Transcript

Separated enum cleanup to another diff: D860

Harbormaster completed remote builds in B1474: Diff 2276.Jan 1 2018, 23:12

jasonbcox edited the summary of this revision. (Show Details)Jan 2 2018, 00:03

jasonbcox added a parent revision: D860: Removed duplicate SIGHASH enum.

deadalnix accepted this revision.Jan 2 2018, 00:55

This revision is now accepted and ready to land.Jan 2 2018, 00:55

Closed by commit rABCa55815a15e4b: Replaced nHashType with SigHashType class in wallet.cpp (authored by jasonbcox). · Explain WhyJan 3 2018, 03:24

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

wallet/

wallet.cpp

5 lines

Diff 2290

View Options

Replaced nHashType with SigHashType class in wallet.cppClosedPublicActions