On 30.09.2017 02:15, Alistair Francis wrote: > Replace a large number of the fprintf(stderr, "*\n" calls with > error_report(). The functions were renamed with these commands and then > compiler issues where manually fixed. [...] > diff --git a/hw/input/pckbd.c b/hw/input/pckbd.c > index c479f827b6..ff4f03e4c3 100644 > --- a/hw/input/pckbd.c > +++ b/hw/input/pckbd.c > @@ -22,6 +22,7 @@ > * THE SOFTWARE. > */ > #include "qemu/osdep.h" > +#include "qemu/error-report.h" > #include "hw/hw.h" > #include "hw/isa/isa.h" > #include "hw/i386/pc.h" > @@ -307,7 +308,7 @@ static void kbd_write_command(void *opaque, hwaddr addr, > /* ignore that */ > break; > default: > - fprintf(stderr, "qemu: unsupported keyboard cmd=0x%02x\n", (int)val); > + error_report("qemu: unsupported keyboard cmd=0x%02x", (int)val); > break;
Please drop the "qemu:" prefix from the string now here, too. Thomas