Page MenuHomePhabricator

Have Seeder's ProcessMessage return fall through
AbandonedPublic

Authored by nakihito on May 18 2020, 18:13.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

Almost every state in ProcessMessage()` results in
PeerMessagingState::AwaitingMessages. Instead of having three
separate return points, having one is easier to maintain.

Test Plan
ninja
ninja check-bitcoin-seeder

Event Timeline

Owners added a reviewer: Restricted Owners Package.May 18 2020, 18:13

We did the exact opposite in D4571 . Should we revert it?