-
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
Pass EvalCtx and result VectorPtr by Reference #1612
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.
Nice.
@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
8ce1895
to
bef1e52
Compare
@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Passes non-optional non-const parameters by reference. This avoids nullability specifiers required by Mac Circle CI builds. Adds nullability declarations to pointers tat need them. The scope of tis is expression/. The function base class signatures are not changed.
@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: Passes non-optional non-const parameters by reference. This avoids nullability specifiers required by Mac Circle CI builds. Adds nullability declarations to pointers that need them. The scope of tis is expressions. The function base class signatures are not changed. Pull Request resolved: facebookincubator#1612 Reviewed By: mbasmanova Differential Revision: D36378417 Pulled By: oerling fbshipit-source-id: 2cd39123977fcff9c986992c3896d3a27c43a22f
Summary: Passes non-optional non-const parameters by reference. This avoids nullability specifiers required by Mac Circle CI builds. Adds nullability declarations to pointers that need them. The scope of tis is expressions. The function base class signatures are not changed. Pull Request resolved: facebookincubator#1612 Reviewed By: mbasmanova Differential Revision: D36378417 Pulled By: oerling fbshipit-source-id: 2cd39123977fcff9c986992c3896d3a27c43a22f
…#1618) What changes were proposed in this pull request? add format check for ch backend (Fixes: facebookincubator#1612) How was this patch tested? unit tests
Passes non-optional non-const parameters by reference. This avoids
nullability specifiers required by Mac Circle CI builds. Adds
nullability declarations to pointers that need them.
The scope of tis is expressions. The function base class signatures
are not changed.