refactor: remove preset coins in linear time with remove-erase idiom #24821

pull rag-hav wants to merge 1 commits into bitcoin:master from rag-hav:remove-preselect-remove-erase changing 1 files +9 −6

rag-hav commented at 6:09 PM on April 10, 2022: none

This is a followup to pr #24814.

This pr implements the filtering of coins without an additional vector using the remove-erase idiom as discussed in #24814
remove preset coins with remove-erase idiom 792f0aa211
DrahtBot added the label Refactoring on Apr 10, 2022
DrahtBot added the label Wallet on Apr 10, 2022
DrahtBot commented at 1:22 AM on April 11, 2022: member


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #24814 (refactor: improve complexity of removing preselected coins by rag-hav)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
MarcoFalke commented at 7:47 AM on April 11, 2022: member

I don't think there is a need to open two conflicting refactoring changes. Please pick the one that you and the reviewers prefer and push it to the existing pull request.
MarcoFalke closed this on Apr 11, 2022
DrahtBot locked this on Apr 11, 2023

Contributors

Labels