-
Notifications
You must be signed in to change notification settings - Fork 171
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
refactor: use std::ranges::all_of
, any_of
, none_of
#3593
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.
I think all of these are in <algorithm>
and not actually in <ranges>
, so you might want to adjust the includes.
This reverts commit 80b44d4.
LGTM, just a small, non-critical suggestion. |
Invalidated by push of 17e4f2a
Invalidated by push of bad2f91
Quality Gate passedIssues Measures |
I couldn't replace all instances, since https://github.com/acts-project/acts/blame/464675980c50a6070f5635127ee27fd29de5a679/Core/include/Acts/Propagator/MultiEigenStepperLoop.hpp#L458 seems not to return an iterator, that can be interpreted by ranges algorithms.