From 8811d49009fd0bbf8ce0877e6d5bf3e54c29e85a Mon Sep 17 00:00:00 2001 From: antirez Date: Tue, 27 Aug 2013 12:36:52 +0200 Subject: [PATCH] Remove useful check from tryObjectEncoding(). We are sure the string is large, since when the sds optimization branch is entered it means that it was not possible to encode it as EMBSTR for size concerns. --- src/object.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/object.c b/src/object.c index b3acd2cc..bfbd3ba1 100644 --- a/src/object.c +++ b/src/object.c @@ -372,8 +372,7 @@ robj *tryObjectEncoding(robj *o) { * We do that only for relatively large strings as this branch * is only entered if the length of the string is greater than * REDIS_ENCODING_EMBSTR_SIZE_LIMIT. */ - if (len > 64 && - o->encoding == REDIS_ENCODING_RAW && + if (o->encoding == REDIS_ENCODING_RAW && sdsavail(s) > len/10) { o->ptr = sdsRemoveFreeSpace(o->ptr);