diff options
| author | Seiji Aguchi <seiji.aguchi@hds.com> | 2013-08-05 15:40:44 -0400 |
|---|---|---|
| committer | Luiz Capitulino <lcapitulino@redhat.com> | 2013-08-20 11:51:59 -0400 |
| commit | 4a44d85e28bd282f53ccf0fa933dd71b8744a229 (patch) | |
| tree | 50836ead288ef58a3b217dfb9b8ec91004a9f23a /arch_init.c | |
| parent | 9176e8fb8f78206bd4039f001aca0d931a47d663 (diff) | |
| download | qemu-4a44d85e28bd282f53ccf0fa933dd71b8744a229.zip qemu-4a44d85e28bd282f53ccf0fa933dd71b8744a229.tar.gz qemu-4a44d85e28bd282f53ccf0fa933dd71b8744a229.tar.bz2 | |
Convert stderr message calling error_get_pretty() to error_report()
Convert stderr messages calling error_get_pretty()
to error_report().
Timestamp is prepended by -msg timstamp option with it.
Per Markus's comment below, A conversion from fprintf() to
error_report() is always an improvement, regardless of
error_get_pretty().
http://marc.info/?l=qemu-devel&m=137513283408601&w=2
But, it is not reasonable to convert them at one time
because fprintf() is used everwhere in qemu.
So, it should be done step by step with avoiding regression.
Signed-off-by: Seiji Aguchi <seiji.aguchi@hds.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'arch_init.c')
| -rw-r--r-- | arch_init.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch_init.c b/arch_init.c index 68a7ab7..94d45e1 100644 --- a/arch_init.c +++ b/arch_init.c @@ -1125,8 +1125,8 @@ void do_acpitable_option(const QemuOpts *opts) acpi_table_add(opts, &err); if (err) { - fprintf(stderr, "Wrong acpi table provided: %s\n", - error_get_pretty(err)); + error_report("Wrong acpi table provided: %s", + error_get_pretty(err)); error_free(err); exit(1); } |