[coreboot-gerrit] Patch merged into coreboot/master: 60ec2ff lib/hexdump: Use `size_t` for length parameter of `hexdump32()`

gerrit at coreboot.org gerrit at coreboot.org
Mon May 5 08:59:08 CEST 2014


the following patch was just integrated into master:
commit 60ec2ff2f005cc3d361225ad24327dff14c7abf2
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sat May 3 16:21:34 2014 +0200

    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 <paulepanter at users.sourceforge.net>
    Reviewed-on: http://review.coreboot.org/5646
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <patrick at georgi-clan.de>


See http://review.coreboot.org/5646 for details.

-gerrit



More information about the coreboot-gerrit mailing list