-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Clean up overuse of auto in MapConcat.cpp #11152
Conversation
This pull request was exported from Phabricator. Differential Revision: D63723010 |
✅ Deploy Preview for meta-velox canceled.
|
This pull request was exported from Phabricator. Differential Revision: D63723010 |
Summary: Pull Request resolved: facebookincubator#11152 Wherever the type of any variable is not clear due to the use of auto, it was excpliticly declared instead. Differential Revision: D63723010
5aa29a7
to
bcc537c
Compare
Summary: Wherever the type of any variable is not clear due to the use of auto, it was excpliticly declared instead. Differential Revision: D63723010
bcc537c
to
eb4d2d5
Compare
This pull request was exported from Phabricator. Differential Revision: D63723010 |
Summary: Wherever the type of any variable is not clear due to the use of auto, it was excpliticly declared instead. Differential Revision: D63723010
eb4d2d5
to
92a4e9d
Compare
This pull request was exported from Phabricator. Differential Revision: D63723010 |
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.
Thanks Daniel!
This pull request has been merged in b543230. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Summary: Wherever the type of any variable is not clear due to the use of auto, it was excpliticly declared instead.
Differential Revision: D63723010