Skip to content
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

rfc11 changes (annual stake) #29

Merged
merged 2 commits into from Nov 21, 2019
Merged

rfc11 changes (annual stake) #29

merged 2 commits into from Nov 21, 2019

Conversation

Nagalim
Copy link
Member

@Nagalim Nagalim commented Nov 12, 2019

nAnnualStake defined based on coinage.
Limiting coinage to 1-year, both for the annual stake calculation and for nSubsidy directly.
Mention of added complexity due to 2 indices
Description of imprecise nAnnualStake.
Trivializing seasonal timing attack and removal of timestamp attack (the description of the timestamp attack was not accurate and is mostly irrelevant as a practical attack because random third-party blocks added during the attack will cause it to be ineffective and ultimately it reduces to a simple timing attack).

`nAnnualStake` defined based on coinage.
Limiting coinage to 1-year, both for the annual stake calculation and for `nSubsidy` directly.
Mention of added complexity due to 2 indices
Description of imprecise `nAnnualStake`.
Trivializing seasonal timing attack and removal of timestamp attack (the description of the timestamp attack was not accurate and is mostly irrelevant as a practical attack because random third-party blocks added during the attack will cause it to be ineffective and ultimately it reduces to a simple timing attack).
Two spaces after titles
@peerchemist peerchemist merged commit 583a981 into peercoin:master Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants