Page MenuHomePhabricator

Remove all warning in the seeder
ClosedPublic

Authored by deadalnix on Sep 27 2017, 10:49.

Details

Summary

As per title.

Test Plan

Build and check there are no warning anymore.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
seederwarning
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 930
Build 930: arc lint + arc unit

Event Timeline

Remove another warning in protocol.cpp

sickpig added inline comments.
src/seeder/main.cpp
43 ↗(On Diff #1428)

what about changing it into Bitcoin-cash-seeder?

sickpig requested changes to this revision.Sep 27 2017, 15:53
This revision now requires changes to proceed.Sep 27 2017, 15:53
deadalnix edited edge metadata.
deadalnix added inline comments.
src/seeder/main.cpp
43 ↗(On Diff #1428)

That is completely off topic here.

sickpig added inline comments.
src/seeder/main.cpp
43 ↗(On Diff #1428)

guess so and I saw that you already change it in another diff

This revision is now accepted and ready to land.Sep 28 2017, 09:26
This revision was automatically updated to reflect the committed changes.