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

Copying a Roaring64MapSetBitForwardIterator into an iterator from another map does not work #589

Closed
tudor opened this issue Feb 9, 2024 · 1 comment

Comments

@tudor
Copy link
Contributor

tudor commented Feb 9, 2024

Roaring64Map m1;
auto it = m1.begin();
// ...
Roaring64Map m2;
it = m2.begin();

This leads to trouble, because Roaring64MapSetBitForwardIterator's copy assignment does not copy the map reference. So if you then use it.move(<value>), map_iter is going to point into the wrong map (m1).

@tudor
Copy link
Contributor Author

tudor commented Feb 9, 2024

Fix in #590

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

No branches or pull requests

2 participants