Differential D11473

Rename the Uint256KeyWrapper struct
ClosedPublic
Actions

Authored by Fabien on May 16 2022, 10:03.

Details

Reviewers

sdulfari

Group Reviewers

Restricted Project

Commits

rABCd1c416b8e3e8: Rename the Uint256KeyWrapper struct

Summary

This is not a wrapper to a key, this is actually a type to be used as a 256 bits radix key. There is no change in behavior.

Test Plan

ninja all check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

rename_uint256keywrapper

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19045
Build 37842: Build Diff	build-without-wallet · build-clang · build-diff · lint-circular-dependencies · build-debug · build-clang-tidy
Build 37841: arc lint + arc unit

Event Timeline

Fabien created this revision.May 16 2022, 10:03

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 16 2022, 10:03

Fabien requested review of this revision.May 16 2022, 10:03

Harbormaster completed remote builds in B19044: Diff 33538.May 16 2022, 10:18

mass rename is fubared

Fix replace typo

Harbormaster completed remote builds in B19045: Diff 33539.May 16 2022, 11:38

sdulfari accepted this revision.May 16 2022, 15:45

This revision is now accepted and ready to land.May 16 2022, 15:45

Closed by commit rABCd1c416b8e3e8: Rename the Uint256KeyWrapper struct (authored by Fabien). · Explain WhyMay 17 2022, 09:28

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCd1c416b8e3e8: Rename the Uint256KeyWrapper struct.

Revision Contents
Changeset List

Path

Size

src/

radix.h

14 lines

test/

radix_tests.cpp

15 lines

Diff 33539

View Options

src/radix.h