From 204a14b8cffba6a23e4e313e248bed7ef5cd8260 Mon Sep 17 00:00:00 2001 From: sundb Date: Fri, 13 Nov 2020 21:16:40 +0800 Subject: [PATCH] Fix make warning in hellohook,c and testmodule.c (#8044) * Fix build warning in hellohook,c and testmodule.c * Change ci->id type to (unsigned long long) --- src/modules/hellohook.c | 2 +- src/modules/testmodule.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/modules/hellohook.c b/src/modules/hellohook.c index 7ab78ed07..b451bd975 100644 --- a/src/modules/hellohook.c +++ b/src/modules/hellohook.c @@ -47,7 +47,7 @@ void clientChangeCallback(RedisModuleCtx *ctx, RedisModuleEvent e, uint64_t sub, printf("Client %s event for client #%llu %s:%d\n", (sub == REDISMODULE_SUBEVENT_CLIENT_CHANGE_CONNECTED) ? "connection" : "disconnection", - ci->id,ci->addr,ci->port); + (unsigned long long)ci->id,ci->addr,ci->port); } void flushdbCallback(RedisModuleCtx *ctx, RedisModuleEvent e, uint64_t sub, void *data) diff --git a/src/modules/testmodule.c b/src/modules/testmodule.c index 5381380e5..5634530dd 100644 --- a/src/modules/testmodule.c +++ b/src/modules/testmodule.c @@ -220,7 +220,7 @@ int TestNotifications(RedisModuleCtx *ctx, RedisModuleString **argv, int argc) { } else { rep = RedisModule_CallReplyStringPtr(r, &sz); if (sz != 1 || *rep != '1') { - FAIL("Got reply '%.*s'. expected '1'", sz, rep); + FAIL("Got reply '%.*s'. expected '1'", (int)sz, rep); } } /* For l we expect nothing since we didn't subscribe to list events */ @@ -235,7 +235,7 @@ int TestNotifications(RedisModuleCtx *ctx, RedisModuleString **argv, int argc) { } else { rep = RedisModule_CallReplyStringPtr(r, &sz); if (sz != 1 || *rep != '2') { - FAIL("Got reply '%.*s'. expected '2'", sz, rep); + FAIL("Got reply '%.*s'. expected '2'", (int)sz, rep); } }