[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 3/3] qmp: do not include monitor.h from qapi-
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 3/3] qmp: do not include monitor.h from qapi-types-core.h |
Date: |
Thu, 07 Jun 2012 07:35:23 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 |
Il 07/06/2012 06:55, Anthony Liguori ha scritto:
> On 06/07/2012 10:02 AM, Paolo Bonzini wrote:
>> The comment is stale, monitor.h is not needed anymore (only qerror.h
>> is, because it contains the schema for errors).
>>
>> Signed-off-by: Paolo Bonzini<address@hidden>
>> ---
>> qapi/qapi-types-core.h | 4 +---
>> 1 files changed, 1 insertions(+), 3 deletions(-)
>>
>> diff --git a/qapi/qapi-types-core.h b/qapi/qapi-types-core.h
>> index 27e6be0..f781fc3 100644
>> --- a/qapi/qapi-types-core.h
>> +++ b/qapi/qapi-types-core.h
>> @@ -16,8 +16,6 @@
>>
>> #include "qemu-common.h"
>> #include "error.h"
>> -
>> -/* FIXME this is temporary until we remove middle mode */
>> -#include "monitor.h"
>> +#include "qerror.h"
>
> Middle mode generates signatures that contain a 'Monitor *'.
> qmp-commands.h includes qmp-types which only includes qmp-types-core.h
>
> Without having a #include "monitor.h" here, qmp-commands.h is relying on
> something else including monitor.h before including qmp-commands.h
Everything should be including qemu-common.h that has the "typedef
struct Monitor Monitor". Note that this is not really just a cleanup.
Right now I cannot include qapi-types.h from block_int.h, because
monitor.h depends on block_int.h.
Paolo