[coreboot-gerrit] Patch set updated for coreboot: acdf4c4 libpayload/libc/hexdump.c: Take `const void *memory` as argument

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Dec 19 19:40:58 CET 2014


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5403

-gerrit

commit acdf4c4a7e028c003b3119312e7c7fc30bd889dd
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Mon Mar 24 00:42:36 2014 +0100

    libpayload/libc/hexdump.c: Take `const void *memory` as argument
    
    `*memory` is not changed in `hexdump()` and just read so make it
    `const`.
    
    Change-Id: I9504d25ab5c785f05c39c9a4f48c21f68659a829
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 payloads/libpayload/include/libpayload.h | 2 +-
 payloads/libpayload/libc/hexdump.c       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/payloads/libpayload/include/libpayload.h b/payloads/libpayload/include/libpayload.h
index b36b40c..d6258de 100644
--- a/payloads/libpayload/include/libpayload.h
+++ b/payloads/libpayload/include/libpayload.h
@@ -303,7 +303,7 @@ long int labs(long int j);
 long long int llabs(long long int j);
 u8 bin2hex(u8 b);
 u8 hex2bin(u8 h);
-void hexdump(void *memory, size_t length);
+void hexdump(const void *memory, size_t length);
 void fatal(const char *msg) __attribute__ ((noreturn));
 /** @} */
 
diff --git a/payloads/libpayload/libc/hexdump.c b/payloads/libpayload/libc/hexdump.c
index 1a35f05..ee07e5a 100644
--- a/payloads/libpayload/libc/hexdump.c
+++ b/payloads/libpayload/libc/hexdump.c
@@ -29,7 +29,7 @@
 
 #include <libpayload.h>
 
-void hexdump(void *memory, size_t length)
+void hexdump(const void *memory, size_t length)
 {
 	int i;
 	uint8_t *m;



More information about the coreboot-gerrit mailing list