qemu-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]