HomePhabricator

Merge #12752: [MOVEONLY] Move compressor utility functions out of class

Description

Merge #12752: [MOVEONLY] Move compressor utility functions out of class

Summary:
76a9aac Move compressor utility functions out of class (Pieter Wuille)

Pull request description:

This is a refactor from #10785 with no functionality change.

Move the compresion utility functions out of CScriptCompressor, as a preparation for making the class templated. I'm submitting it as a separate PR as I think it's a general improvement to code readability, and to reduce the diff further on.

Tree-SHA512: 3b3d17c2b96e43f752f512dd573296a6bb15cae165fbe3c79212a0970f5196a62a59a821d5100f29638af1e7461c9171f3dccb8112f005ee08da0ec7fe0073fd

Backport of Core PR12752
https://github.com/bitcoin/bitcoin/pull/12752/

Test Plan:

make check

Reviewers: deadalnix, Fabien, jasonbcox, O1 Bitcoin ABC, #bitcoin_abc

Reviewed By: Fabien, O1 Bitcoin ABC, #bitcoin_abc

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

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Mar 30 2018, 10:51
nakihitoCommitted on Aug 19 2019, 20:14
jasonbcoxPushed on Aug 19 2019, 22:24
Reviewer
Restricted Owners Package
Differential Revision
D3903: Merge #12752: [MOVEONLY] Move compressor utility functions out of class
Parents
rSTAGINGd5489985e53c: Merge #12811: test: Make summary row bold-red if any test failed and show…
Branches
Unknown
Tags
Unknown