Re: [Qemu-devel] [PATCH 0/2] Clean up includes of monitor.h

2012-06-06 Thread Luiz Capitulino
On Wed, 6 Jun 2012 08:08:42 +0200 Paolo Bonzini wrote: > There is no reason why QAPI clients should use monitor.h at all, > qerror.h suffices. > > qmp-commands.h uses the Monitor typedef, but it is only included > from files that already get the typedef from qemu-common.h. Could you please reb

[Qemu-devel] [PATCH 0/2] Clean up includes of monitor.h

2012-06-05 Thread Paolo Bonzini
There is no reason why QAPI clients should use monitor.h at all, qerror.h suffices. qmp-commands.h uses the Monitor typedef, but it is only included from files that already get the typedef from qemu-common.h. Paolo Bonzini (2): kvm: add missing include files qmp: do not include monitor.h from