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
Remove Replaced transactions from cache #2088
Remove Replaced transactions from cache #2088
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.
ACK after @yahiheb's formatting fix
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.
Code is fine, logic seems good too, but @nopara73 approval required.
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. (Resolve conflict and mergable.)
90b7309
to
ddb3432
Compare
Done. |
Oh, wanted to merge it, but there's still a conflict. |
26f56c6
ddb3432
to
26f56c6
Compare
Rebased. |
In case a transaction is replaced (RBF) by other one or when there is a chain reorg the coins created by those transactions are removed from the Coins list. The transactions that created those coins has to be removed too.