Re: [FFmpeg-devel] [PATCH 2/2] Add missing "const" all over the place.

2014-08-29 Thread Reimar Döffinger
On Thu, Aug 28, 2014 at 05:20:44PM -0700, Timothy Gu wrote: > On Aug 28, 2014 3:46 PM, "Reimar Döffinger" > wrote: > > > > There is a risk that one in the code that is not always > > compiled might break things. > > What do you mean by 'one in the code'? The intended grouping was (one) (in the c

Re: [FFmpeg-devel] [PATCH 2/2] Add missing "const" all over the place.

2014-08-29 Thread Carl Eugen Hoyos
Reimar Döffinger gmx.de> writes: > There is a risk that one in the code that is not > always compiled might break things. We will only find out if you push;-) Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman

Re: [FFmpeg-devel] [PATCH 2/2] Add missing "const" all over the place.

2014-08-28 Thread Michael Niedermayer
On Fri, Aug 29, 2014 at 12:45:34AM +0200, Reimar Döffinger wrote: > There is a risk that one in the code that is not always > compiled might break things. builds fine here [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Democracy is the form of government in wh

Re: [FFmpeg-devel] [PATCH 2/2] Add missing "const" all over the place.

2014-08-28 Thread Timothy Gu
On Aug 28, 2014 3:46 PM, "Reimar Döffinger" wrote: > > There is a risk that one in the code that is not always > compiled might break things. What do you mean by 'one in the code'? [...] Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org ht

[FFmpeg-devel] [PATCH 2/2] Add missing "const" all over the place.

2014-08-28 Thread Reimar Döffinger
There is a risk that one in the code that is not always compiled might break things. Signed-off-by: Reimar Döffinger --- ffserver.c | 2 +- libavcodec/aaccoder.c | 2 +- libavcodec/aic.c| 2 +- libavcodec/atrac3plus.c | 38 +++