diff options
| author | Philippe Mathieu-Daudé <f4bug@amsat.org> | 2020-08-22 20:09:50 +0200 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2020-09-11 21:25:59 +0200 |
| commit | b42581f5bb40489cb172829e9ab0d6d6dfa37026 (patch) | |
| tree | 53386cbc9808337c4bcc4d902f16f86942254271 /include/qemu-common.h | |
| parent | 67263b33af9506d78558b1edfa5a9bf976dc7b3d (diff) | |
| download | focaccia-qemu-b42581f5bb40489cb172829e9ab0d6d6dfa37026.tar.gz focaccia-qemu-b42581f5bb40489cb172829e9ab0d6d6dfa37026.zip | |
util/hexdump: Reorder qemu_hexdump() arguments
qemu_hexdump()'s pointer to the buffer and length of the buffer are closely related arguments but are widely separated in the argument list order (also, the format of <stdio.h> function prototypes is usually to have the FILE* argument coming first). Reorder the arguments as "fp, prefix, buf, size" which is more logical. Suggested-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com> Reviewed-by: Stefano Garzarella <sgarzare@redhat.com> Message-Id: <20200822180950.1343963-3-f4bug@amsat.org> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'include/qemu-common.h')
| -rw-r--r-- | include/qemu-common.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/qemu-common.h b/include/qemu-common.h index 6834883822..9cfd62669b 100644 --- a/include/qemu-common.h +++ b/include/qemu-common.h @@ -138,8 +138,8 @@ int os_parse_cmd_args(int index, const char *optarg); * Hexdump a buffer to a file. An optional string prefix is added to every line */ -void qemu_hexdump(const void *bufptr, FILE *fp, - const char *prefix, size_t size); +void qemu_hexdump(FILE *fp, const char *prefix, + const void *bufptr, size_t size); /* * helper to parse debug environment variables |