-
Notifications
You must be signed in to change notification settings - Fork 83
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
Add method to assert a pointer is valid #3062
Conversation
bb76434
to
185c6e6
Compare
8af7b24
to
83dfcc6
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.
Awesome, thanks @celinval!
Can we add at least one expected
test so that we can match against the descriptions of the new pointer checks (Dangling pointer, non-null pointer, valid pointer, etc.).
Co-authored-by: Zyad Hassan <88045115+zhassan-aws@users.noreply.github.com>
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.
Approving as this is a temporary solution per the discussions.
Would you like me to document that this signature is temporary? The method is marked as unstable too. |
This should be sufficient. Thanks. |
This new method will assert that a pointer isn't null, and that its is valid for access of a given type.
I'm creating this as a draft for now as I'm still adding more tests.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.