From 519be66fc59558971dd653afe69ccaf1a633b492 Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Sun, 23 Dec 2018 20:03:35 +0100 Subject: Fix -Wsign-compare trouble Mostly by changing to `unsigned` types where applicable, sometimes `signed` types, and casting as a last resort. Change-Id: I08895543ffb7a48058bcf91ef6500ca113f2d305 Signed-off-by: Nico Huber Reviewed-on: https://review.coreboot.org/c/flashrom/+/30409 Tested-by: build bot (Jenkins) Reviewed-by: Jacob Garber --- helpers.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'helpers.c') diff --git a/helpers.c b/helpers.c index a714908b..cfa9812c 100644 --- a/helpers.c +++ b/helpers.c @@ -30,9 +30,9 @@ uint32_t address_to_bits(uint32_t addr) return 32 - lzb; } -int bitcount(unsigned long a) +unsigned int bitcount(unsigned long a) { - int i = 0; + unsigned int i = 0; for (; a != 0; a >>= 1) if (a & 1) i++; -- cgit v1.2.3