Page MenuHomePhabricator
Differential D12304

Remove confusing CAddrDB
ClosedPublic
Actions

Authored by Fabien on Oct 19 2022, 10:03.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABC9f83b8b17167: Remove confusing CAddrDB
Summary

Backport of core#22915.

Depends on D12302.

Test Plan
ninja all check-all

Diff Detail

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

Event Timeline

Fabien created this revision.Oct 19 2022, 10:03
Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 19 2022, 10:03
Fabien requested review of this revision.Oct 19 2022, 10:03
teamcity edited the summary of this revision. (Show Details)Oct 19 2022, 10:03
Harbormaster completed remote builds in B20603: Diff 35832.Oct 19 2022, 10:17
PiRK accepted this revision.Oct 19 2022, 11:32
PiRK added a subscriber: PiRK.
PiRK added inline comments.
src/test/addrman_tests.cpp
1053 ↗(On Diff #35832)

missing typo fix ("Even though")

This revision is now accepted and ready to land.Oct 19 2022, 11:32
Closed by commit rABC9f83b8b17167: Remove confusing CAddrDB (authored by MarcoFalke <falke.marco@gmail.com>, committed by Fabien). · Explain WhyOct 19 2022, 12:32
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABC9f83b8b17167: Remove confusing CAddrDB.

Revision Contents
Changeset List

PathSize
src/
24 lines
16 lines
10 lines
2 lines
5 lines
1 line
5 lines
qt/
1 line
1 line
test/
19 lines
fuzz/
1 line

Diff 35842

View Options

src/addrdb.h

Loading...
View Options

src/addrdb.cpp

Loading...
View Options

src/addrman.h

Loading...
View Options

src/addrman.cpp

Loading...
View Options

src/init.cpp

Loading...
View Options

src/net.h

Loading...
View Options

src/net.cpp

Loading...
View Options

src/qt/bantablemodel.h

Loading...
View Options

src/qt/walletframe.cpp

Loading...
View Options

src/test/addrman_tests.cpp

Loading...
View Options

src/test/fuzz/integer.cpp

Loading...