Page MenuHomePhabricator

Adds PeerMessagingState enum to seeder/bitcoin.*
ClosedPublic

Authored by nakihito on Mon, Mar 9, 21:22.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABC411bd9b97e91: Adds PeerMessagingState enum to seeder/bitcoin.*
Summary

Moves enum from just the p2p_messaging_tests.cpp to bitcoin.h allowing
use in the rest of the seeder code.

Test Plan
ninja check-bitcoin-seeder

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nakihito created this revision.Mon, Mar 9, 21:22
Owners added a reviewer: Restricted Owners Package.Mon, Mar 9, 21:22
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Mar 9, 21:22
deadalnix accepted this revision.Thu, Mar 12, 15:21
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/seeder/bitcoin.h
24 ↗(On Diff #16857)

Put the false state first.

This revision is now accepted and ready to land.Thu, Mar 12, 15:21
nakihito updated this revision to Diff 16902.Thu, Mar 12, 17:57

Put false state first in enum.

This revision was automatically updated to reflect the committed changes.