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
Preventing surges in sigma #462
Conversation
Ready to review. The test failure caused by the HD mint test failure. |
@@ -0,0 +1,59 @@ | |||
#ifndef HASH_FUNCTIONS_H__ | |||
#define HASH_FUNCTIONS_H__ | |||
|
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.
When renaming a file you need to change the constants at the top
void CSigmaState::Containers::AddMint(sigma::PublicCoin const & pubCoin, CMintedCoinInfo const & coinInfo) { | ||
mintedPubCoins.insert(std::make_pair(pubCoin, coinInfo)); | ||
mintMetaInfo[coinInfo.coinGroupId][coinInfo.denomination] += 1; | ||
CheckSurgeCondition(coinInfo.coinGroupId, coinInfo.denomination); |
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.
How adding a mint can create a surge condition? Same for removing a spend
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.
There are possible improvements to be done later. It will work for now
b76ec92
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.
Will work
No description provided.