-
Notifications
You must be signed in to change notification settings - Fork 891
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
Switch from std::tie() to structured binding. #8117
Conversation
89bfd8a
to
84f40c8
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.
🔥
Rerun tests |
Codecov Report
@@ Coverage Diff @@
## branch-0.20 #8117 +/- ##
===============================================
+ Coverage 82.88% 82.91% +0.02%
===============================================
Files 103 103
Lines 17668 17868 +200
===============================================
+ Hits 14645 14816 +171
- Misses 3023 3052 +29
Continue to review full report at Codecov.
|
Glad we can work around the structured binding lambda capture limitations by naming captures explicitly! 😌 |
@gpucibot merge |
Addendum to #7866, to switch from using
std::tie()
to structured bindings, as prescribed for C++17.This required workarounds for compiler restrictions on using aliases from structured bindings as captures in lambdas.