HomePhabricator

Merge #13258: uint256: Remove unnecessary crypto/common.h dependency

Description

Merge #13258: uint256: Remove unnecessary crypto/common.h dependency

Summary:
bf2e01097 uint256: Remove unnecessary crypto/common.h use (Karl-Johan Alm)

Pull request description:

This is an alternative to #13242 which keeps the `ReadLE64` part, but moves the `crypto/common.h` dependency into `crypto/common.h` as a function outside of `uint256`.

**Reason:** this change will remove dependencies for `uint256` to `crypto/common.h`, `compat/endian.h`, and `compat/byteswap.h`.

This PR removes the need to update tests to be endian-aware/-independent, but keeps the (arguably dubious) `ReadLE64` part (which was only introduced to fix the tests, not for any functionality).

Tree-SHA512: 78b35123cdb185b3b3ec59aba5ca8a5db72624d147f2d6a5484ffa5ce626a72f782a01dc6893fc8f5619b03e2eae7b5a03b0df5d43460f3bda428e719e188aec

Backport of Core PR13258
https://github.com/bitcoin/bitcoin/pull/13258/

Test Plan:

make check

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

Reviewed By: deadalnix, O1 Bitcoin ABC, #bitcoin_abc

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

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Nov 30 2018, 17:21
nakihitoCommitted on Oct 22 2019, 16:28
jasonbcoxPushed on Oct 22 2019, 17:46
Reviewer
Restricted Owners Package
Differential Revision
D4277: Merge #13258: uint256: Remove unnecessary crypto/common.h dependency
Parents
rSTAGINGbc8ad5ec1d81: Merge #11491: [gui] Add proxy icon in statusbar
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/13650, tag: phabricator/base/13649, tag: phabricator/base/13647, tag: phabricator/base/13646