[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/15] vl: Make -machine $TYPE, help and -accel
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 10/15] vl: Make -machine $TYPE, help and -accel help print to stdout |
Date: |
Mon, 08 Apr 2019 14:33:57 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Marcel Apfelbaum <address@hidden> writes:
> Hi Markus,
>
> On 4/8/19 11:36 AM, Markus Armbruster wrote:
>> Command line help help explicitly requested by the user should be
>> printed to stdout, not stderr. We do elsewhere. Adjust -machine
>> $TYPE,help and -accel help to match: use printf() instead of
>> error_printf().
>>
>> Cc: Marcel Apfelbaum <address@hidden>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> vl.c | 10 +++++-----
>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/vl.c b/vl.c
>> index c696ad2a13..792ef36001 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -1556,12 +1556,12 @@ static int machine_help_func(QemuOpts *opts,
>> MachineState *machine)
>> continue;
>> }
>> - error_printf("%s.%s=%s",
>> MACHINE_GET_CLASS(machine)->name,
>> - prop->name, prop->type);
>> + printf("%s.%s=%s", MACHINE_GET_CLASS(machine)->name,
>> + prop->name, prop->type);
>
> Does the line above have an alignment issue?
Oops! Will fix.
> Anyway,
>
> Reviewed-by: Marcel Apfelbaum <address@hidden>
Thank you!
- Re: [Qemu-devel] [PATCH 03/15] char-pty: Drop "char device redirected to" message, (continued)
[Qemu-devel] [PATCH 05/15] mips/boston: Report errors with error_report(), not error_printf(), Markus Armbruster, 2019/04/08
[Qemu-devel] [PATCH 01/15] qemu-img: Use error_vreport() in error_exit(), Markus Armbruster, 2019/04/08
[Qemu-devel] [PATCH 10/15] vl: Make -machine $TYPE, help and -accel help print to stdout, Markus Armbruster, 2019/04/08
[Qemu-devel] [PATCH 04/15] loader-fit: Wean off error_printf(), Markus Armbruster, 2019/04/08
[Qemu-devel] [PATCH 12/15] qemu-print: New qemu_printf(), qemu_vprintf() etc., Markus Armbruster, 2019/04/08
[Qemu-devel] [PATCH 02/15] block/ssh: Do not report read/write/flush errors to the user, Markus Armbruster, 2019/04/08