Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-08-01 Thread Blue Swirl
On Wed, Aug 1, 2012 at 6:45 AM, Paolo Bonzini wrote: > Il 31/07/2012 22:38, Blue Swirl ha scritto: >>> > It's a whole lot of churn though, and clobbers the history for most QMP >>> > functions. It also seems like a strange thing for clang to complain >>> > about... >> Not really, it's just C99: >

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Paolo Bonzini
Il 31/07/2012 22:38, Blue Swirl ha scritto: >> > It's a whole lot of churn though, and clobbers the history for most QMP >> > functions. It also seems like a strange thing for clang to complain >> > about... > Not really, it's just C99: > http://en.wikipedia.org/wiki/Restrict > > Prefixing would

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Anthony Liguori
Paolo Bonzini writes: > Il 30/07/2012 18:04, blauwir...@gmail.com ha scritto: >> From: Blue Swirl >> >> 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 chang

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Michael Roth
On Tue, Jul 31, 2012 at 08:38:45PM +, Blue Swirl wrote: > On Tue, Jul 31, 2012 at 6:55 PM, Michael Roth > wrote: > > On Tue, Jul 31, 2012 at 04:56:50PM +, Blue Swirl wrote: > >> On Tue, Jul 31, 2012 at 12:58 PM, Luiz Capitulino > >> wrote: > >> > On Tue, 31 Jul 2012 09:28:43 +0200 > >> >

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Blue Swirl
On Tue, Jul 31, 2012 at 6:55 PM, Michael Roth wrote: > On Tue, Jul 31, 2012 at 04:56:50PM +, Blue Swirl wrote: >> On Tue, Jul 31, 2012 at 12:58 PM, Luiz Capitulino >> wrote: >> > On Tue, 31 Jul 2012 09:28:43 +0200 >> > Paolo Bonzini wrote: >> > >> >> Il 30/07/2012 18:04, blauwir...@gmail.com

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Michael Roth
On Tue, Jul 31, 2012 at 04:56:50PM +, Blue Swirl wrote: > On Tue, Jul 31, 2012 at 12:58 PM, Luiz Capitulino > wrote: > > On Tue, 31 Jul 2012 09:28:43 +0200 > > Paolo Bonzini wrote: > > > >> Il 30/07/2012 18:04, blauwir...@gmail.com ha scritto: > >> > From: Blue Swirl > >> > > >> > Clang comp

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Blue Swirl
On Tue, Jul 31, 2012 at 12:58 PM, Luiz Capitulino wrote: > On Tue, 31 Jul 2012 09:28:43 +0200 > Paolo Bonzini wrote: > >> Il 30/07/2012 18:04, blauwir...@gmail.com ha scritto: >> > From: Blue Swirl >> > >> > Clang compiler complained about use of reserved word 'restrict' in SLIRP >> > and QAPI.

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Luiz Capitulino
On Tue, 31 Jul 2012 09:28:43 +0200 Paolo Bonzini wrote: > Il 30/07/2012 18:04, blauwir...@gmail.com ha scritto: > > From: Blue Swirl > > > > Clang compiler complained about use of reserved word 'restrict' in SLIRP > > and QAPI. > > > > Rename 'restrict' to 'restricted' which also matches other

Re: [Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-31 Thread Paolo Bonzini
Il 30/07/2012 18:04, blauwir...@gmail.com ha scritto: > From: Blue Swirl > > 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,

[Qemu-devel] [PATCH 3/5] qapi: avoid reserved word restrict

2012-07-30 Thread blauwirbel
From: Blue Swirl Clang compiler complained about use of reserved word 'restrict' in SLIRP and QAPI. Rename 'restrict' to 'restricted' which also matches other SLIRP code. Signed-off-by: Blue Swirl --- net/slirp.c |6 +++--- qapi-schema.json |4 ++-- 2 files changed, 5 insertions(