diff options
| author | Philippe Mathieu-Daudé | 2020-08-22 20:09:50 +0200 |
|---|---|---|
| committer | Laurent Vivier | 2020-09-11 21:25:59 +0200 |
| commit | b42581f5bb40489cb172829e9ab0d6d6dfa37026 (patch) | |
| tree | 53386cbc9808337c4bcc4d902f16f86942254271 /hw/net/fsl_etsec/etsec.c | |
| parent | util/hexdump: Convert to take a void pointer argument (diff) | |
| download | qemu-b42581f5bb40489cb172829e9ab0d6d6dfa37026.tar.gz qemu-b42581f5bb40489cb172829e9ab0d6d6dfa37026.tar.xz 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 'hw/net/fsl_etsec/etsec.c')
| -rw-r--r-- | hw/net/fsl_etsec/etsec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/net/fsl_etsec/etsec.c b/hw/net/fsl_etsec/etsec.c index 5c40303f93..93886bba60 100644 --- a/hw/net/fsl_etsec/etsec.c +++ b/hw/net/fsl_etsec/etsec.c @@ -357,7 +357,7 @@ static ssize_t etsec_receive(NetClientState *nc, #if defined(HEX_DUMP) fprintf(stderr, "%s receive size:%zd\n", nc->name, size); - qemu_hexdump(buf, stderr, "", size); + qemu_hexdump(stderr, "", buf, size); #endif /* Flush is unnecessary as are already in receiving path */ etsec->need_flush = false; |