Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix sAdd, sRem return type and value #232

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

Closed
wants to merge 2 commits into from
Closed

Fix sAdd, sRem return type and value #232

wants to merge 2 commits into from

Conversation

summerwind
Copy link

I found the mistake to the return value of sAdd, sRem in README.
These return value was checked by the following test files.
https://gist.github.com/3384113

@summerwind
Copy link
Author

Ooops.. I just re-fixed.

nicolasff added a commit that referenced this pull request Sep 2, 2012
Thanks @summerwind for reporting this issue.
Fixes Pull Request #232.
@nicolasff
Copy link
Member

The command has changed in Redis, sadd and srem should now return the number of elements added or removed. It used to be just one or zero, but since the addition of variadic commands it can now be an arbitrary number. See docs for SADD and SREM.

I have changed it on master; thanks for reporting the issue!

@nicolasff nicolasff closed this Sep 2, 2012
@summerwind
Copy link
Author

Thanks Nicolas.

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

Successfully merging this pull request may close these issues.

2 participants