[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict |
Date: |
Tue, 31 Jul 2012 16:56:50 +0000 |
On Tue, Jul 31, 2012 at 12:58 PM, Luiz Capitulino
<address@hidden> wrote:
> On Tue, 31 Jul 2012 09:28:43 +0200
> Paolo Bonzini <address@hidden> wrote:
>
>> Il 30/07/2012 18:04, address@hidden ha scritto:
>> > From: Blue Swirl <address@hidden>
>> >
>> > Clang compiler complained about use of reserved word 'restrict' in SLIRP
>> > and QAPI.
>> >
>> > Rename 'restrict' to 'restricted' which also matches other SLIRP code.
>>
>> Can't do it, this changes the command-line option.
>>
>> Luiz, Michael, any ideas?
>
> I'm not sure how complicated it would be to implement this, but we could add
> a 'bind' keyword to the type dict to control mapping between protocol names
> and generated variable names. Like this:
>
> { 'type': 'NetdevUserOptions',
> 'data': {
> '*hostname': 'str',
> '*restrict': 'bool',
> ...
> '*hostfwd': ['String'],
> '*guestfwd': ['String'] },
>
> 'bind': { 'restrict': 'restricted' } }
How about prefixing all json-generated field names with for example
'json_'? Should be a simple mechanical change.
In addition to 'restrict', there may also be problems with 'if'
(-drive, HMP drive_add) and maybe also 'auto' as value (several
command line options, HMP pci_add) in the future.
- [Qemu-devel] [PATCH 2/5] sparc: fix expression with uninitialized initial value, (continued)
[Qemu-devel] [PATCH 4/5] user: fix accidental AREG0 use, blauwirbel, 2012/07/30
[Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict, blauwirbel, 2012/07/30
Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict, Anthony Liguori, 2012/07/31
[Qemu-devel] [PATCH 1/5] sparc: fix floppy TC line setup, blauwirbel, 2012/07/30
[Qemu-devel] [PATCH 5/5] configure: disable a few Clang compiler warnings, blauwirbel, 2012/07/30