Merge pull request #6579 from oranagra/rm_reply_string_opt

Slightly more efficient RM_ReplyWithEmptyString
This commit is contained in:
Salvatore Sanfilippo 2019-11-19 10:54:00 +01:00 committed by GitHub
commit a7122f4518
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1389,7 +1389,7 @@ int RM_ReplyWithString(RedisModuleCtx *ctx, RedisModuleString *str) {
int RM_ReplyWithEmptyString(RedisModuleCtx *ctx) { int RM_ReplyWithEmptyString(RedisModuleCtx *ctx) {
client *c = moduleGetReplyClient(ctx); client *c = moduleGetReplyClient(ctx);
if (c == NULL) return REDISMODULE_OK; if (c == NULL) return REDISMODULE_OK;
addReplyBulkCBuffer(c, "", 0); addReply(c,shared.emptybulk);
return REDISMODULE_OK; return REDISMODULE_OK;
} }
@ -1404,8 +1404,7 @@ int RM_ReplyWithVerbatimString(RedisModuleCtx *ctx, const char *buf, size_t len)
return REDISMODULE_OK; return REDISMODULE_OK;
} }
/* Reply to the client with a NULL. In the RESP protocol a NULL is encoded /* Reply to the client with a NULL.
* as the string "$-1\r\n".
* *
* The function always returns REDISMODULE_OK. */ * The function always returns REDISMODULE_OK. */
int RM_ReplyWithNull(RedisModuleCtx *ctx) { int RM_ReplyWithNull(RedisModuleCtx *ctx) {