Page MenuHomePhabricator

Unsubtree Univalue
ClosedPublic

Authored by Fabien on Tue, Jun 4, 16:27.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCd367f8912132: Unsubtree Univalue
Summary
With this PR I'm proposing we stop treating Univalue like a subtree, or upstream project/fork, and going forward, treat it as part of this codebase, which we can refactor directly (with pulls to this repo. Ideally, after this is merged, our univalue subtree repo could be marked as "archived". In this repo, I think there is a good chance that the Univalue code will ultimately be refactored away into "modern" C++, i.e using std::variant (at least one person has played around with doing this).

Backport of core#25369.

Depends on D16274.

For ease of review, only changes similar to the source material are included. However more changes will be added in follow-ups such as autotools removal and clang-format linting of the files.

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien requested review of this revision.Tue, Jun 4, 16:27
This revision is now accepted and ready to land.Tue, Jun 4, 18:54
This revision was automatically updated to reflect the committed changes.