From 32b0e8c3ba9a5c59e556d9a550fd21029a4510f1 Mon Sep 17 00:00:00 2001 From: Sitsofe Wheeler Date: Mon, 31 Aug 2020 11:44:11 +0100 Subject: [PATCH] memlock: avoid type confusion in format string Windows GCC has the followng grumble: t/memlock.c: In function 'worker': t/memlock.c:25:26: error: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t' {aka 'long long unsigned int'} [-Werror=format=] 25 | printf("loop%d: did %lu MiB\n", i+1, size/(1024UL*1024UL)); | ~~^ ~~~~~~~~~~~~~~~~~~~~ | | | | long unsigned int size_t {aka long long unsigned int} | %llu Fix it up by just using the original variable that was already in megabytes. Signed-off-by: Sitsofe Wheeler --- t/memlock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/memlock.c b/t/memlock.c index 418dc3c4..9f5a3ea8 100644 --- a/t/memlock.c +++ b/t/memlock.c @@ -22,7 +22,7 @@ static void *worker(void *data) for (index = 0; index + 4096 < size; index += 4096) memset(&buf[index+512], 0x89, 512); if (first) { - printf("loop%d: did %lu MiB\n", i+1, size/(1024UL*1024UL)); + printf("loop%d: did %lu MiB\n", i+1, td->mib); first = 0; } } -- 2.25.1