Don't decode object on STRLEN when not necessary

diff --git a/src/t_string.c b/src/t_string.c
index 3b91f5e6d3392b2078c600b5c0f3c5d4debd79f5..d537c8b909fe20942dfa615c18fcf26549157638 100644 (file)
--- a/src/t_string.c
+++ b/src/t_string.c
@@ -358,7 +358,14 @@ void strlenCommand(redisClient *c) {
     if ((o = lookupKeyReadOrReply(c,c->argv[1],shared.czero)) == NULL ||
         checkType(c,o,REDIS_STRING)) return;
 
-    o = getDecodedObject(o);
-    addReplyLongLong(c,sdslen(o->ptr));
-    decrRefCount(o);
+    if (o->encoding == REDIS_ENCODING_RAW) {
+        addReplyLongLong(c,sdslen(o->ptr));
+    } else if (o->encoding == REDIS_ENCODING_INT) {
+        char llbuf[32];
+        int len = ll2string(llbuf,sizeof(llbuf),(long)o->ptr);
+        addReplyLongLong(c,len);
+    } else {
+        redisPanic("Unknown string encoding");
+    }
 }
+