[backport#17850] Convert addrdb/addrman to new serialization
ClosedPublic
Actions

Authored by PiRK on Nov 13 2020, 12:59.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCe59a7e12537a: [backport#17850] Convert addrdb/addrman to new serialization

Summary

The review for the change in src/addrdb.h needs to take into account the fact that PR19129 was merged before PR17850.
See 2ad5838 and D7146.

This is a backport of Core PR17850 [2/2]
https://github.com/bitcoin/bitcoin/pull/17850/commits/9250a087d2f450f37342010aea0a5d583eea508b

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr17850

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 14026
Build 28059: Build Diff	build-clang-tidy · build-diff · build-debug · build-without-wallet · lint-circular-dependencies · build-clang
Build 28058: arc lint + arc unit

Event Timeline

PiRK created this revision.Nov 13 2020, 12:59

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 13 2020, 12:59

PiRK requested review of this revision.Nov 13 2020, 12:59

teamcity edited the summary of this revision. (Show Details)Nov 13 2020, 12:59

PiRK edited the summary of this revision. (Show Details)Nov 13 2020, 13:02

Harbormaster completed remote builds in B14026: Diff 25717.Nov 13 2020, 13:07

Fabien accepted this revision.Nov 13 2020, 14:10

This revision is now accepted and ready to land.Nov 13 2020, 14:10

PiRK retitled this revision from Convert addrdb/addrman to new serialization to [backport#17850] Convert addrdb/addrman to new serialization.Nov 13 2020, 14:53

Closed by commit rABCe59a7e12537a: [backport#17850] Convert addrdb/addrman to new serialization (authored by Pieter Wuille <pieter.wuille@gmail.com>, committed by PiRK). · Explain WhyNov 13 2020, 15:08

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCe59a7e12537a: [backport#17850] Convert addrdb/addrman to new serialization.