[coreboot-gerrit] Change in coreboot[master]: src/lib/hexdump: Use size_t for indices

[Jacob Garber (Code Review)]

Jacob Garber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33799 )

Change subject: src/lib/hexdump: Use size_t for indices
......................................................................

Patch Set 8: Code-Review+2

(2 comments)

https://review.coreboot.org/c/coreboot/+/33799/3/src/lib/hexdump.c 
File src/lib/hexdump.c:

https://review.coreboot.org/c/coreboot/+/33799/3/src/lib/hexdump.c@26 
PS3, Line 26: int j, num_bytes
> It's probably better to have them both as size_t, since otherwise there will be a size_t -> int conv […]

Done

https://review.coreboot.org/c/coreboot/+/33799/6/src/lib/hexdump.c 
File src/lib/hexdump.c:

https://review.coreboot.org/c/coreboot/+/33799/6/src/lib/hexdump.c@67 
PS6, Line 67: int
> I suppose the argument could be made that since its value at the end of the while loop will be the o […]

Done