Optimize the performance of sdscatrepr in printable characters
sdscatrepr is not the hot path in redis, but it's still useful to have make it less wasteful.
@@ -997,6 +997,7 @@ void sdsfreesplitres(sds *tokens, int count) {
* After the call, the modified sds string is no longer valid and all the
* references must be substituted with the new pointer returned by the call. */
sds sdscatrepr(sds s, const char *p, size_t len) {
+ s = sdsMakeRoomFor(s, len + 2);
s = sdscatlen(s,"\"",1);
while(len--) {
switch(*p) {
@@ -1011,7 +1012,7 @@ sds sdscatrepr(sds s, const char *p, size_t len) {
case '\b': s = sdscatlen(s,"\\b",2); break;
default:
if (isprint(*p))
- s = sdscatprintf(s,"%c",*p);
+ s = sdscatlen(s, p, 1);
else
s = sdscatprintf(s,"\\x%02x",(unsigned char)*p);
break;
