From 60ec2ff2f005cc3d361225ad24327dff14c7abf2 Mon Sep 17 00:00:00 2001 From: Paul Menzel Date: Sat, 3 May 2014 16:21:34 +0200 Subject: lib/hexdump: Use `size_t` for length parameter of `hexdump32()` In the signature of the function `hexdump32()` it does not make sense to represent a length, assumed to be positive, as a signed integer. With this change, it is no longer necessary to cast a pointer to unsigned long when passing it to `hexdump32()`. The same change for the function `hexdump()` was done in commit 3dd0e72d [1]. lib/hexdump: Take const void * and size_t as arguments [1] http://review.coreboot.org/4575 Change-Id: Id97f5daff95f94e862ee8b5be896a6629b125a13 Signed-off-by: Paul Menzel Reviewed-on: http://review.coreboot.org/5646 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi --- src/lib/hexdump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/lib') diff --git a/src/lib/hexdump.c b/src/lib/hexdump.c index 44a5aa433f..a15c5cdb11 100644 --- a/src/lib/hexdump.c +++ b/src/lib/hexdump.c @@ -62,7 +62,7 @@ void hexdump(const void *memory, size_t length) } } -void hexdump32(char LEVEL, const void *d, int len) +void hexdump32(char LEVEL, const void *d, size_t len) { int count = 0; -- cgit v1.2.3