From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:54811) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SbsA4-0004nV-Sc for qemu-devel@nongnu.org; Tue, 05 Jun 2012 07:43:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Sbs9y-0007HU-MO for qemu-devel@nongnu.org; Tue, 05 Jun 2012 07:42:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:9072) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Sbs9y-0007HG-Ex for qemu-devel@nongnu.org; Tue, 05 Jun 2012 07:42:50 -0400 Message-ID: <4FCDF0B2.6030707@redhat.com> Date: Tue, 05 Jun 2012 13:42:42 +0200 From: Kevin Wolf MIME-Version: 1.0 References: <1338858018-17189-1-git-send-email-mdroth@linux.vnet.ibm.com> <1338858018-17189-3-git-send-email-mdroth@linux.vnet.ibm.com> <4FCDE0F4.2010202@siemens.com> <4FCDE9A8.7060301@us.ibm.com> <4FCDECEB.5040708@siemens.com> In-Reply-To: <4FCDECEB.5040708@siemens.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 02/17] qidl: add qc definitions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: Anthony Liguori , "owasserm@redhat.com" , "quintela@redhat.com" , "qemu-devel@nongnu.org" , Michael Roth , "yamahata@valinux.co.jp" , "pbonzini@redhat.com" , "akong@redhat.com" , "afaerber@suse.de" Am 05.06.2012 13:26, schrieb Jan Kiszka: > On 2012-06-05 13:12, Anthony Liguori wrote: >> On 06/05/2012 06:35 PM, Jan Kiszka wrote: >>> On 2012-06-05 03:00, Michael Roth wrote: >>>> Define away the annotations so we can still compile. >>>> >>>> Signed-off-by: Michael Roth >>>> --- >>>> qapi/qc.h | 11 +++++++++++ >>>> 1 files changed, 11 insertions(+), 0 deletions(-) >>>> create mode 100644 qapi/qc.h >>>> >>>> diff --git a/qapi/qc.h b/qapi/qc.h >>>> new file mode 100644 >>>> index 0000000..3b3a8b9 >>>> --- /dev/null >>>> +++ b/qapi/qc.h >>>> @@ -0,0 +1,11 @@ >>>> +#ifndef QC_H >>>> +#define QC_H >>>> + >>>> +#define qc_declaration >>>> +#define _immutable >>>> +#define _derived >>>> +#define _broken >>>> +#define _version(x) >>>> +#define _size_is(x) >>>> + >>>> +#endif >>> >>> These tags require a different prefix than the reserved '_'. >> >> The rationale is that QIDL is part of the "compiler/library implementation" that >> this namespace is reserved for. > > It's a QEMU-internal thing, and we want to be portable. Not sure if it's > worth to risk collisions in some distant corner. I agree, this isn't really convincing. If QIDL was part of the system environment, it wouldn't be in the qemu source tree. After all these rules aren't there just for fun but in order to avoid naming conflicts, and conflicts between qemu and qemu are much less likely than between some exotic libc and qemu. Kevin