Moves some constants from seeder/dns_tests.cpp to dns.h.
Details
Details
- Reviewers
Fabien - Group Reviewers
Restricted Owners Package (Owns No Changed Paths) Restricted Project - Commits
- rSTAGING0da65ecee208: Add constants to dns.h
rABC0da65ecee208: Add constants to dns.h
ninja ninja check-bitcoin-seeder
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- AddConstantsToDNS
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 9779 Build 17433: Default Diff Build & Tests Build 17432: arc lint + arc unit
Event Timeline
src/seeder/dns.cpp | ||
---|---|---|
77 ↗ | (On Diff #16888) | Whatever the type is, if you go from signed to unsigned you need to be sure that it is safe when using the - operator (which is done a few lines below) because this is a behavior change. |
130 ↗ | (On Diff #16888) | Mind changing k to ok while you're at it ? |
142 ↗ | (On Diff #16888) | There is a type for pointer arithmetic: https://en.cppreference.com/w/cpp/types/ptrdiff_t |
src/seeder/dns.h | ||
14 ↗ | (On Diff #16888) | 2 questions:
|
Comment Actions
Removed static and changed const to constexpr. Also undid changes to variables in parse_name() and write_name(). Changed MAX_LABEL_LENGTH from size_t to int.