-
Notifications
You must be signed in to change notification settings - Fork 559
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 support to casting RedisResult to CString. #660
Conversation
The linter seems to fail on code that isn't a part of this PR. |
#663 cleans up the unrelated clippy failures. |
This seems reasonable to me. @jaymell what do you think? |
13ad495
to
36a641a
Compare
Looks good. Should we add a couple simple tests? |
@jaymell I'll be happy to add some, but I don't see equivalent |
Nothing special, there are some similar examples if you look at the other type tests, e.g., make sure that a null byte in a value returns the error we expect. |
043435e
to
16794e3
Compare
0572b16
to
8b3a940
Compare
@jaymell added tests. |
Thanks! |
No description provided.