summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorantirez <antirez@gmail.com>2013-07-28 11:00:09 +0200
committerantirez <antirez@gmail.com>2013-07-28 11:02:20 +0200
commit78644f5e3ec3b970d9a6e30c339af7098ea30e58 (patch)
tree7fcb7d79bc5f2cdfeea1a4e3d3309b2430cf94bf
parentaea8020554feceafc831b18273cbe4e512a5094e (diff)
downloadredis-78644f5e3ec3b970d9a6e30c339af7098ea30e58.tar.gz
Remove dead variable bothsds from object.c.
Thanks to @run and @badboy for spotting this. Triva: clang was not able to provide me a warning about that when compiling. This closes #1024 and #1207, committing the change myself as the pull requests no longer apply cleanly after other changes to the same function.
-rw-r--r--src/object.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/object.c b/src/object.c
index 472d4a34b..5202c0c55 100644
--- a/src/object.c
+++ b/src/object.c
@@ -347,13 +347,11 @@ int compareStringObjectsWithFlags(robj *a, robj *b, int flags) {
redisAssertWithInfo(NULL,a,a->type == REDIS_STRING && b->type == REDIS_STRING);
char bufa[128], bufb[128], *astr, *bstr;
size_t alen, blen, minlen;
- int bothsds = 1;
if (a == b) return 0;
if (a->encoding != REDIS_ENCODING_RAW) {
alen = ll2string(bufa,sizeof(bufa),(long) a->ptr);
astr = bufa;
- bothsds = 0;
} else {
astr = a->ptr;
alen = sdslen(astr);
@@ -361,7 +359,6 @@ int compareStringObjectsWithFlags(robj *a, robj *b, int flags) {
if (b->encoding != REDIS_ENCODING_RAW) {
blen = ll2string(bufb,sizeof(bufb),(long) b->ptr);
bstr = bufb;
- bothsds = 0;
} else {
bstr = b->ptr;
blen = sdslen(bstr);