-
Notifications
You must be signed in to change notification settings - Fork 38
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
Const-correctness #215
Comments
I honestly just haven't thought about it! Most current usage/testing is in C right now where const-ness isn't quite as baked in to the language. One first step is could be to go through the C API functions and declare pointers that are not modified as |
Just a note that I'm bumping this to 0.4 because it would result in a lot of compiler warnings for any project that was using a development version between 0.2 and 0.3 (and because the 0.3 release is forthcoming!). This has come up a few times in existing usage...declaring input pointers (e.g., |
Hi, is there a policy / upcoming work on
const
versions of API functions, where reasonable?i.e.
const
accessors forinternal::Unique
,const
pointers in function prototypes where the underlying data is not modified, or where a const flavor of e.g.ArrowArrayView
could be used instead?The text was updated successfully, but these errors were encountered: