Re: [Qemu-devel] [PATCH 01/12] qapi/qmp-event.c: Don't manually include os-win32.h/os-posix.h

2015-07-31 Thread Daniel P. Berrange
On Wed, Jul 22, 2015 at 05:44:00PM +0100, Peter Maydell wrote: > qmp-event.c already includes qemu-common.h, so manually including > os-win32.h/os-posix.h is unnecessary (and potentially fragile, > since it's duplicating the #ifdef logic that chooses which of the > two we need). Remove the unnecess

[Qemu-devel] [PATCH 01/12] qapi/qmp-event.c: Don't manually include os-win32.h/os-posix.h

2015-07-22 Thread Peter Maydell
qmp-event.c already includes qemu-common.h, so manually including os-win32.h/os-posix.h is unnecessary (and potentially fragile, since it's duplicating the #ifdef logic that chooses which of the two we need). Remove the unnecessary include logic. Signed-off-by: Peter Maydell --- qapi/qmp-event.c