Skip to content
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

Make redis use const pointers where possible #6303

Open
petdance opened this issue Aug 4, 2019 · 0 comments
Open

Make redis use const pointers where possible #6303

petdance opened this issue Aug 4, 2019 · 0 comments

Comments

@petdance
Copy link

petdance commented Aug 4, 2019

I submitted PR #6302 last night to make functions return const char * when returning literal strings. This should prevent any inadvertant illegal modification of strings.

My next steps would be to start working on making pointer arguments to functions be const pointers where possible. Most places in redis use const pointers, but there are places that don't. I'm hoping that updating those places that don't will make code safer and let the compiler more accurately track uninitialized variables.

Is this something you'd be interested in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant