-
Notifications
You must be signed in to change notification settings - Fork 793
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
Avoiding copying in range-based loops #1481
Conversation
93a0278
to
eddc346
Compare
Signed-off-by: asapelkin <asapelkin0x01@ya.ru>
eddc346
to
17f39bd
Compare
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.
Good catch @asapelkin - thanks for contributing! Changes are fine by me
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.
Nice one 👍
/backport to stable-2.6 |
The backport to stable-2.6 failed. Please do this backport manually. |
/backport to stable-2.6 |
The backport to stable-2.6 failed. Please do this backport manually. |
Small optimization. Used const references in range-based loops to avoid copying every items