caleb-brown/redict
1
0
Fork 0
mirror of https://codeberg.org/redict/redict.git synced 2025-01-23 08:38:27 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

uphold the smove contract to return 0 when the element is not a member of the source set, even if source=dest

Browse Source
This commit is contained in:
Glenn Nethercutt 2015-04-17 09:27:54 -04:00
parent 6c60526db9
commit 626b4f6907
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/t_set.c
View File

@ -343,7 +343,10 @@ void smoveCommand(redisClient *c) {
/* If srcset and dstset are equal, SMOVE is a no-op */ /* If srcset and dstset are equal, SMOVE is a no-op */
if (srcset == dstset) { if (srcset == dstset) {
if (setTypeIsMember(srcset,ele))
addReply(c,shared.cone); addReply(c,shared.cone);
else
addReply(c,shared.czero);
return; return;
} }

1
tests/unit/type/set.tcl
View File

@ -519,6 +519,7 @@ start_server {
test "SMOVE non existing key" { test "SMOVE non existing key" {
setup_move setup_move
assert_equal 0 [r smove myset1 myset2 foo] assert_equal 0 [r smove myset1 myset2 foo]
assert_equal 0 [r smove myset1 myset1 foo]
assert_equal {1 a b} [lsort [r smembers myset1]] assert_equal {1 a b} [lsort [r smembers myset1]]
assert_equal {2 3 4} [lsort [r smembers myset2]] assert_equal {2 3 4} [lsort [r smembers myset2]]
} }