[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp |
Date: |
Tue, 23 Jul 2013 08:47:36 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Andreas Färber <address@hidden> writes:
> Am 22.07.2013 23:03, schrieb Seiji Aguchi:
>> Convert stderr messages calling error_get_pretty()
>> to error_report().
>>
>> Timestamp is prepended by -msg timstamp option with it.
>>
>> This is suggested by Luiz Capitulino.
>>
>> http://marc.info/?l=qemu-devel&m=137331442628866&w=2
>>
>> Signed-off-by: Seiji Aguchi <address@hidden>
>> ---
>> arch_init.c | 4 ++--
>> hw/char/serial.c | 5 +++--
>> hw/i386/pc.c | 6 +++---
>> qemu-char.c | 2 +-
>> target-i386/cpu.c | 2 +-
>> target-ppc/translate_init.c | 3 ++-
>> vl.c | 9 +++++----
>> 7 files changed, 17 insertions(+), 14 deletions(-)
>
> How is this related to error_get_pretty()?
>
> And does error_report() expect a trailing \n or not? I would assume no,
> but spotted some in this patch.
You're correct, and the patch needs fixing. See commits
312fd5f error: Strip trailing '\n' from error string arguments (again)
be62a2e Strip trailing '\n' from error_report()'s first argument (again)
6daf194 Strip trailing '\n' from error_report()'s first argument
[...]
- [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Seiji Aguchi, 2013/07/22
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Andreas Färber, 2013/07/22
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Luiz Capitulino, 2013/07/29
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Andreas Färber, 2013/07/29
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Luiz Capitulino, 2013/07/29
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Markus Armbruster, 2013/07/29
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Luiz Capitulino, 2013/07/29
- Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp, Seiji Aguchi, 2013/07/29