Page MenuHomePhabricator

[radix] Adds the capability to use an adapter for key manipulation
ClosedPublic

Authored by Fabien on May 12 2022, 19:18.

Details

Summary

This allow to use element which do not have a getId memeber, or do not have a valid key type in the tree.

Test Plan
ninja check-bitcoin-radix_tests

Diff Detail

Event Timeline

Fabien added a reviewer: deadalnix.
Fabien edited the summary of this revision. (Show Details)
Fabien edited the test plan for this revision. (Show Details)

Add one missing conversion and unit test

deadalnix added inline comments.
src/radix.h
414

This should be moved out of here as a next step.

src/test/radix_tests.cpp
607

Shouldn't this be BOOST_REQUIRE?

We are not trying to test ParseUint64 here.

This revision is now accepted and ready to land.May 19 2022, 17:49