[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/13] error: don't set sep when print progname
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 08/13] error: don't set sep when print progname |
Date: |
Fri, 18 Oct 2013 13:40:09 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> Il 18/10/2013 03:11, Wenchao Xia ha scritto:
>> The behavior to set sep brings trouble to modification later,
>> the logic is not changed by add tailing space in fprintf().
>>
>> Signed-off-by: Wenchao Xia <address@hidden>
>> ---
>> util/qemu-error.c | 5 ++---
>> 1 files changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/util/qemu-error.c b/util/qemu-error.c
>> index 0ccd3e9..d1e858a 100644
>> --- a/util/qemu-error.c
>> +++ b/util/qemu-error.c
>> @@ -161,8 +161,7 @@ static void error_print_loc(void)
>> const char *const *argp;
>>
>> if (!cur_mon && progname) {
>> - fprintf(stderr, "%s:", progname);
>> - sep = " ";
>> + fprintf(stderr, "%s: ", progname);
>> }
>> switch (cur_loc->kind) {
>> case LOC_CMDLINE:
>> @@ -181,7 +180,7 @@ static void error_print_loc(void)
>> error_printf(" ");
>> break;
>> default:
>> - error_printf("%s", sep);
>> + break;
>> }
>> }
>>
>>
>
> This changes behavior for LOC_FILE.
>
> Before:
>
> $ cat xyz.cfg
> [device "abc"]
> driver = def
> $ qemu-system-x86_64 -readconfig xyz.cfg
> qemu-system-x86_64:xyz.cfg:2: parse error
>
> After:
>
> $ qemu-system-x86_64 -readconfig xyz.cfg
> qemu-system-x86_64: xyz.cfg:2: parse error
>
> Could even be an improvement, but you need to note it in the commit message.
No, it is not an improvement. The old format matches exactly how other
report errors with location, e.g. jade. Please leave it that way,
- Re: [Qemu-devel] [PATCH 03/13] qapi: move MonitorEvent define, (continued)
[Qemu-devel] [PATCH 10/13] qerror: deref once in qerror_report(), Wenchao Xia, 2013/10/18
[Qemu-devel] [PATCH 09/13] error: print progname with error_vprintf(), Wenchao Xia, 2013/10/18
[Qemu-devel] [PATCH 08/13] error: don't set sep when print progname, Wenchao Xia, 2013/10/18
[Qemu-devel] [PATCH 13/13] stubs: do not call monitor_printf(), Wenchao Xia, 2013/10/18
[Qemu-devel] [PATCH 11/13] qerror: folder qerror emit logic, Wenchao Xia, 2013/10/18
[Qemu-devel] [PATCH 12/13] monitor: hide *cur_mon in monitor_get_fd(), Wenchao Xia, 2013/10/18