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

Fix override_refines container iteration crash. #47

Open
wants to merge 2 commits into
base: vlasiator-version
Choose a base branch
from

Conversation

ursg
Copy link

@ursg ursg commented Jun 19, 2024

This is one of those "how did this ever work" crashes?
Instead of iterating through a container while modifying it, use std::erase_if.

@markusbattarbee
Copy link

Same fix for line 9622?

@ursg
Copy link
Author

ursg commented Jun 19, 2024

Ah, good catch.

@markusbattarbee
Copy link

so would std::remove_if() work in order to not require C++20 as that isn't yet available on all our platforms?

@lkotipal
Copy link

Right, std::erase_if() should just be shorthand for erase-remove.

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.

3 participants