Page MenuHomePhabricator

[avalanche] Use constexpr for global constants
ClosedPublic

Authored by deadalnix on Jul 4 2020, 23:32.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABC7bd7f8c0caea: [avalanche] Use constexpr for global constants
Summary

Also remove useless anonymous namespace.

Test Plan
ninja all check-avalanche

Diff Detail

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

Event Timeline

Also do AVALANCHE_TIME_STEP_MILLISECONDS

majcosta added a subscriber: majcosta.

Nit: consider getting rid of the AVALANCHE_ prefix unless where needed to convey that info; this is all static and we're in src/avalanche after all

This revision is now accepted and ready to land.Jul 5 2020, 00:11