[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: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict |
Date: |
Tue, 31 Jul 2012 19:01:35 -0500 |
User-agent: |
Notmuch/0.13.2+93~ged93d79 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Paolo Bonzini <address@hidden> writes:
> 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?
Just change c_var() in scripts/qapi.py to have:
diff --git a/scripts/qapi.py b/scripts/qapi.py
index 8082af3..34bfbf6 100644
--- a/scripts/qapi.py
+++ b/scripts/qapi.py
@@ -131,6 +131,8 @@ def camel_case(name):
return new_name
def c_var(name):
+ if name in ['restrict']:
+ return 'q_restrict'
return name.replace('-', '_').lstrip("*")
def c_fun(name):
Then fixup and build failures (make sure to do a clean
first--dependencies seem to be off for qapi).
Regards,
Anthony Liguori
>
> Paolo
>
>> Signed-off-by: Blue Swirl <address@hidden>
>> ---
>> net/slirp.c | 6 +++---
>> qapi-schema.json | 4 ++--
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/net/slirp.c b/net/slirp.c
>> index 5c2e6b2..8c42b53 100644
>> --- a/net/slirp.c
>> +++ b/net/slirp.c
>> @@ -722,9 +722,9 @@ int net_init_slirp(const NetClientOptions *opts, const
>> char *name,
>> net_init_slirp_configs(user->hostfwd, SLIRP_CFG_HOSTFWD);
>> net_init_slirp_configs(user->guestfwd, 0);
>>
>> - ret = net_slirp_init(vlan, "user", name, user->restrict, vnet,
>> user->host,
>> - user->hostname, user->tftp, user->bootfile,
>> - user->dhcpstart, user->dns, user->smb,
>> + ret = net_slirp_init(vlan, "user", name, user->restricted, vnet,
>> + user->host, user->hostname, user->tftp,
>> + user->bootfile, user->dhcpstart, user->dns,
>> user->smb,
>> user->smbserver);
>>
>> while (slirp_configs) {
>> diff --git a/qapi-schema.json b/qapi-schema.json
>> index bc55ed2..3912430 100644
>> --- a/qapi-schema.json
>> +++ b/qapi-schema.json
>> @@ -1925,7 +1925,7 @@
>> #
>> # @hostname: #optional client hostname reported by the builtin DHCP server
>> #
>> -# @restrict: #optional isolate the guest from the host
>> +# @restricted: #optional isolate the guest from the host
>> #
>> # @ip: #optional legacy parameter, use net= instead
>> #
>> @@ -1956,7 +1956,7 @@
>> { 'type': 'NetdevUserOptions',
>> 'data': {
>> '*hostname': 'str',
>> - '*restrict': 'bool',
>> + '*restricted':'bool',
>> '*ip': 'str',
>> '*net': 'str',
>> '*host': 'str',
>>
- Re: [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, Paolo Bonzini, 2012/07/31
- Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict, Luiz Capitulino, 2012/07/31
- Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict, Blue Swirl, 2012/07/31
- Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict, Michael Roth, 2012/07/31
- Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict, Blue Swirl, 2012/07/31
- Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict, Michael Roth, 2012/07/31
Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict,
Anthony Liguori <=
[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