Re: [Qemu-devel] [PATCH] Convert stderr message calling error_get_pretty() to error_report() to prepend timestamp

[Luiz Capitulino]

On Mon, 29 Jul 2013 23:23:32 +0200
Andreas Färber <address@hidden> wrote:

> Am 29.07.2013 23:20, schrieb Luiz Capitulino:
> > On Mon, 22 Jul 2013 23:23:29 +0200
> > Andreas Färber <address@hidden> wrote:
> >> Am 22.07.2013 23:03, schrieb Seiji Aguchi:
> >>> Convert stderr messages calling error_get_pretty()
> >>> to error_report().
> >>
> >> How is this related to error_get_pretty()?
> > 
> > Yeah, we're converting fprintf(stderr,) calls to error_report() so that
> > error messages get a timestamp.
> 
> Want to add that to my http://wiki.qemu.org/DeveloperNews so that people
> reading it stop adding new ones? :)

Big IMHO here, but honestly speaking I'm not a huge fan of error_report()
because I think that random code shouldn't be allowed to print to the
monitor (only HMP code should).

But it's widespread and it's where the timestamp lives, so calling fprintf()
instead of error_report() will probably start hurting soon.