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:00:09 +0200
commitbf56948fd0c9dfd3bf7ac63e8dc50b47a4f6e98c (patch)
treed6e716b9d0a30f7cd5b7103264fd1475cc01bf34
parent320fa02b9b48ee1c63d88db6344fc0d328e24853 (diff)
downloadredis-bf56948fd0c9dfd3bf7ac63e8dc50b47a4f6e98c.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 9efb590f8..159e92ada 100644
--- a/src/object.c
+++ b/src/object.c
@@ -418,7 +418,6 @@ 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 (sdsEncodedObject(a)) {
@@ -427,7 +426,6 @@ int compareStringObjectsWithFlags(robj *a, robj *b, int flags) {
} else {
alen = ll2string(bufa,sizeof(bufa),(long) a->ptr);
astr = bufa;
- bothsds = 0;
}
if (sdsEncodedObject(b)) {
bstr = b->ptr;
@@ -435,7 +433,6 @@ int compareStringObjectsWithFlags(robj *a, robj *b, int flags) {
} else {
blen = ll2string(bufb,sizeof(bufb),(long) b->ptr);
bstr = bufb;
- bothsds = 0;
}
if (flags & REDIS_COMPARE_COLL) {
return strcoll(astr,bstr);