[libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Ronald S. Bultje
From: "Ronald S. Bultje" Defining restrict results - for some compilers - in changing other uses of the restrict keyword also, e.g. __declspec(restrict) gets changed to __declspec(__restrict) on MSVC. This causes compilation failures. Therefore, using a private namespace macro instead is more rel

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Måns Rullgård
"Ronald S. Bultje" writes: > From: "Ronald S. Bultje" > > Defining restrict results - for some compilers - in changing other > uses of the restrict keyword also, e.g. __declspec(restrict) gets > changed to __declspec(__restrict) on MSVC. This causes compilation > failures. Therefore, using a pri

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Ronald S. Bultje
Hi, On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: > "Ronald S. Bultje" writes: > >> From: "Ronald S. Bultje" >> >> Defining restrict results - for some compilers - in changing other >> uses of the restrict keyword also, e.g. __declspec(restrict) gets >> changed to __declspec(__restrict)

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Kostya Shishkov
On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: > Hi, > > On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: > > "Ronald S. Bultje" writes: > > > >> From: "Ronald S. Bultje" > >> > >> Defining restrict results - for some compilers - in changing other > >> uses of the restri

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Ronald S. Bultje
Hi, On Sat, Jun 16, 2012 at 12:39 AM, Kostya Shishkov wrote: > On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: >> On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: >> > "Ronald S. Bultje" writes: >> > >> >> From: "Ronald S. Bultje" >> >> >> >> Defining restrict results -

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Kostya Shishkov
On Sat, Jun 16, 2012 at 12:41:35AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Jun 16, 2012 at 12:39 AM, Kostya Shishkov > wrote: > > On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: > >> On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: > >> > "Ronald S. Bultje" write

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Ronald S. Bultje
Hi, On Sat, Jun 16, 2012 at 12:58 AM, Kostya Shishkov wrote: > On Sat, Jun 16, 2012 at 12:41:35AM -0400, Ronald S. Bultje wrote: >> On Sat, Jun 16, 2012 at 12:39 AM, Kostya Shishkov >> wrote: >> > On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: >> >> On Fri, Jun 15, 2012 at 5:4

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Kostya Shishkov
On Sat, Jun 16, 2012 at 01:10:15AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Jun 16, 2012 at 12:58 AM, Kostya Shishkov > wrote: > > On Sat, Jun 16, 2012 at 12:41:35AM -0400, Ronald S. Bultje wrote: > >> On Sat, Jun 16, 2012 at 12:39 AM, Kostya Shishkov > >> wrote: > >> > On Fri, Jun 15, 2

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Ronald S. Bultje
Hi, On Sat, Jun 16, 2012 at 1:17 AM, Kostya Shishkov wrote: > On Sat, Jun 16, 2012 at 01:10:15AM -0400, Ronald S. Bultje wrote: >> On Sat, Jun 16, 2012 at 12:58 AM, Kostya Shishkov >> wrote: >> > On Sat, Jun 16, 2012 at 12:41:35AM -0400, Ronald S. Bultje wrote: >> >> On Sat, Jun 16, 2012 at 12:3

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Kostya Shishkov
On Sat, Jun 16, 2012 at 01:22:26AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Jun 16, 2012 at 1:17 AM, Kostya Shishkov > wrote: > > On Sat, Jun 16, 2012 at 01:10:15AM -0400, Ronald S. Bultje wrote: > >> On Sat, Jun 16, 2012 at 12:58 AM, Kostya Shishkov > >> wrote: > >> > On Sat, Jun 16, 20

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Ronald S. Bultje
Hi, On Sat, Jun 16, 2012 at 1:32 AM, Kostya Shishkov wrote: > On Sat, Jun 16, 2012 at 01:22:26AM -0400, Ronald S. Bultje wrote: >> On Sat, Jun 16, 2012 at 1:17 AM, Kostya Shishkov >> wrote: >> > On Sat, Jun 16, 2012 at 01:10:15AM -0400, Ronald S. Bultje wrote: >> >> On Sat, Jun 16, 2012 at 12:58

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Kostya Shishkov
On Sat, Jun 16, 2012 at 01:35:39AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Jun 16, 2012 at 1:32 AM, Kostya Shishkov > wrote: > > On Sat, Jun 16, 2012 at 01:22:26AM -0400, Ronald S. Bultje wrote: > >> On Sat, Jun 16, 2012 at 1:17 AM, Kostya Shishkov > >> wrote: > >> > On Sat, Jun 16, 201

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-15 Thread Luca Barbato
On 06/16/2012 03:02 AM, Ronald S. Bultje wrote: > I wish to support a compiler that does not support c99. Shall we work > together and make that happen or shall we just cry like little zealous > babies while the real world passes by? Reality is that either you add 10 lines in the preprocessor that

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-16 Thread Måns Rullgård
Kostya Shishkov writes: > On Sat, Jun 16, 2012 at 01:35:39AM -0400, Ronald S. Bultje wrote: >> Hi, >> >> On Sat, Jun 16, 2012 at 1:32 AM, Kostya Shishkov >> wrote: >> > On Sat, Jun 16, 2012 at 01:22:26AM -0400, Ronald S. Bultje wrote: >> >> On Sat, Jun 16, 2012 at 1:17 AM, Kostya Shishkov >> >>

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-16 Thread Ronald S. Bultje
Hi, On Sat, Jun 16, 2012 at 2:21 AM, Luca Barbato wrote: > On 06/16/2012 03:02 AM, Ronald S. Bultje wrote: >> I wish to support a compiler that does not support c99. Shall we work >> together and make that happen or shall we just cry like little zealous >> babies while the real world passes by? >

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-16 Thread Måns Rullgård
"Ronald S. Bultje" writes: > Hi, > > On Sat, Jun 16, 2012 at 2:21 AM, Luca Barbato wrote: >> On 06/16/2012 03:02 AM, Ronald S. Bultje wrote: >>> I wish to support a compiler that does not support c99. Shall we work >>> together and make that happen or shall we just cry like little zealous >>> ba

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-16 Thread Luca Barbato
On 06/16/2012 06:33 PM, Ronald S. Bultje wrote: > Hi, > > On Sat, Jun 16, 2012 at 2:21 AM, Luca Barbato wrote: >> On 06/16/2012 03:02 AM, Ronald S. Bultje wrote: >>> I wish to support a compiler that does not support c99. Shall we work >>> together and make that happen or shall we just cry like l

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-16 Thread Måns Rullgård
Luca Barbato writes: > On 06/16/2012 06:33 PM, Ronald S. Bultje wrote: >> Hi, >> >> On Sat, Jun 16, 2012 at 2:21 AM, Luca Barbato wrote: >>> On 06/16/2012 03:02 AM, Ronald S. Bultje wrote: I wish to support a compiler that does not support c99. Shall we work together and make that hap

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-16 Thread Måns Rullgård
Måns Rullgård writes: > Luca Barbato writes: > >> On 06/16/2012 06:33 PM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Sat, Jun 16, 2012 at 2:21 AM, Luca Barbato wrote: On 06/16/2012 03:02 AM, Ronald S. Bultje wrote: > I wish to support a compiler that does not support c99. Shall we wor

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-18 Thread Diego Biurrun
On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: > On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: > > "Ronald S. Bultje" writes: > >> From: "Ronald S. Bultje" > >> > >> Defining restrict results - for some compilers - in changing other > >> uses of the restrict keyword al

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-18 Thread Reinhard Tartler
On Mon, Jun 18, 2012 at 3:26 PM, Diego Biurrun wrote: >> I wish to support a compiler that does not support c99. Shall we work >> together and make that happen or shall we just cry like little zealous >> babies while the real world passes by? > > Comments like this are not exactly helping, but yo

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-19 Thread Diego Biurrun
On Mon, Jun 18, 2012 at 07:05:22PM +0200, Reinhard Tartler wrote: > On Mon, Jun 18, 2012 at 3:26 PM, Diego Biurrun wrote: > > >> I wish to support a compiler that does not support c99. Shall we work > >> together and make that happen or shall we just cry like little zealous > >> babies while the

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-28 Thread Alex Converse
On Mon, Jun 18, 2012 at 6:26 AM, Diego Biurrun wrote: > On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: >> On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: >> > "Ronald S. Bultje" writes: >> >> From: "Ronald S. Bultje" >> >> >> >> Defining restrict results - for some comp

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-28 Thread Måns Rullgård
Alex Converse writes: > On Mon, Jun 18, 2012 at 6:26 AM, Diego Biurrun wrote: >> On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: >>> On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: >>> > "Ronald S. Bultje" writes: >>> >> From: "Ronald S. Bultje" >>> >> >>> >> Defining

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-28 Thread Martin Storsjö
On Thu, 28 Jun 2012, Måns Rullgård wrote: Replacing restrict with av_restrict fundamentally doesn't seem worse then any of these other restrictions. This is fundamentally different because it changes the allowed syntax from C99 to something slightly different. Such a change puts an additional

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-28 Thread Måns Rullgård
Martin Storsjö writes: > On Thu, 28 Jun 2012, Måns Rullgård wrote: > >>> Replacing restrict with av_restrict fundamentally doesn't seem worse >>> then any of these other restrictions. >> >> This is fundamentally different because it changes the allowed syntax >> from C99 to something slightly dif

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-28 Thread Martin Storsjö
On Thu, 28 Jun 2012, Måns Rullgård wrote: Martin Storsjö writes: On Thu, 28 Jun 2012, Måns Rullgård wrote: Replacing restrict with av_restrict fundamentally doesn't seem worse then any of these other restrictions. This is fundamentally different because it changes the allowed syntax from

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-28 Thread Måns Rullgård
Martin Storsjö writes: > On Thu, 28 Jun 2012, Måns Rullgård wrote: > >> Martin Storsjö writes: >> >>> On Thu, 28 Jun 2012, Måns Rullgård wrote: >>> > Replacing restrict with av_restrict fundamentally doesn't seem worse > then any of these other restrictions. This is fundamental

Re: [libav-devel] [PATCH] Change all uses of restrict to use av_restrict instead.

2012-06-28 Thread Diego Biurrun
On Thu, Jun 28, 2012 at 11:23:48AM -0700, Alex Converse wrote: > On Mon, Jun 18, 2012 at 6:26 AM, Diego Biurrun wrote: > > On Fri, Jun 15, 2012 at 06:02:11PM -0700, Ronald S. Bultje wrote: > >> On Fri, Jun 15, 2012 at 5:41 PM, Måns Rullgård wrote: > >> > "Ronald S. Bultje" writes: > >> >> From: