New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Subsidy halving interval #802
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that it'd be better to squash trivial commit like added period. If you're going to do it you might as well format the number as 210_000
which is easier to read.
4994d7b
to
94dd57d
Compare
Done! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 94dd57d
This is the kind of feature addition that I'm fine with adding to RC. |
Just a though: I generally add "units" to name of constants when possible. So |
In the case of constants from Bitcoin Core, it's probably best to use whatever name is used in core, adapted to the local naming convention. (I.e. SCREAMING_SNAKE_CASE). If you Google nSubsidyHalvingInterval you get a lot of hits, so keeping it consistent with those hits, and with core, is probably a good idea. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 94dd57d
Fixes #801.