Page MenuHomePhabricator
Differential D12266

Make addrman random context private
ClosedPublic
Actions

Authored by Fabien on Oct 17 2022, 08:23.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCfb491c3b206a: Make addrman random context private
Summary

This used to be protected so the subclasses can make it deterministic in tests, and this is no longer needed since D11827.

Test Plan
ninja all check

Diff Detail

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

Event Timeline

Fabien created this revision.Oct 17 2022, 08:23
Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 17 2022, 08:23
Fabien requested review of this revision.Oct 17 2022, 08:23
Fabien added inline comments.Oct 17 2022, 08:24
src/addrman.h
517 ↗(On Diff #35734)

The linter did it

PiRK accepted this revision.Oct 17 2022, 08:45
This revision is now accepted and ready to land.Oct 17 2022, 08:45
Harbormaster completed remote builds in B20539: Diff 35734.Oct 17 2022, 08:46
Closed by commit rABCfb491c3b206a: Make addrman random context private (authored by Fabien). · Explain WhyOct 17 2022, 10:03
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCfb491c3b206a: Make addrman random context private.

Revision Contents
Changeset List

PathSize
src/
6 lines

Diff 35741

View Options

src/addrman.h

Loading...