-
Notifications
You must be signed in to change notification settings - Fork 21
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
Fork at 257000 per Unforkening Nutocracy motion, #86
Conversation
https://nutocracy.herokuapp.com/arguments/that-it-is-time-to-un-fork-hodlcoin-and-stop-auto-hodling-for-a-year Autohodling removed from block 257000 Older version clients disconnected after block 250000 Also, bug fix to ensure interest not earned on deposits after maturation period ends.
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.
Overall this looks good, with some minor nits that don't really break anything, but may want some further consideration.
@@ -11,6 +11,8 @@ | |||
#include "serialize.h" | |||
#include "uint256.h" | |||
|
|||
static const unsigned int THEUNFORKENING = 257000; |
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.
seems like an odd place for this kind of constant as it not really a transaction primitive but a consensus rule
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.
Yeah, I initially had it in main.h, but found it was required in the interest calculation code here to fix the interest rate bug.
@@ -9,7 +9,7 @@ | |||
* network protocol versioning | |||
*/ | |||
|
|||
static const int PROTOCOL_VERSION = 70010; | |||
static const int PROTOCOL_VERSION = 250000; |
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.
is there a reason for such a massive jump in protocol version?
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 had previously thought that the version here was doubling as the block at which clients using the old code were disconnected.
@@ -22,7 +22,7 @@ static const int MIN_PEER_PROTO_VERSION = GETHEADERS_VERSION; | |||
|
|||
//! disconnect from peers once fork height reached | |||
static const int MIN_PEER_PROTO_MINERHODLFORK_VERSION = 70010; | |||
|
|||
static const int MIN_PEER_PROTO_THEUNFORKENING_VERSION = 250000; |
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.
See comment above
https://nutocracy.herokuapp.com/arguments/that-it-is-time-to-un-fork-hodlcoin-and-stop-auto-hodling-for-a-year
Autohodling removed from block 257000
Older version clients disconnected after block 250000
Also, bug fix to ensure interest not earned on deposits after maturation period ends.