[FFmpeg-devel] [PATCH] Escape 124 (RPL) decoder rev5

Reimar Döffinger Reimar.Doeffinger
Mon Mar 31 19:59:42 CEST 2008


Hello,
On Mon, Mar 31, 2008 at 06:57:30PM +0200, Michael Niedermayer wrote:
> > > It does not work for Sun CC, so I guess it is not legal C...
> > 
> > This will be a problem when we get Sun CC up and running for FATE. Then
> > again, was FFmpeg previously known to work with Sun CC?
> 
> If its not valid C (can someone confirm this?) then ill change it to [1]

I would prefer something like
#if defined(__GNUC__) && __GNUC__ < 3
#define AV_VAR_ARRAYSZ 0
#else
#define AV_VAR_ARRAYSZ
#endif
in libavutil, then I could get rid of similar mess in MPlayer without
extra work as well :-).
Does this sound like a good idea to you?
A better name would be nice though, preferably one that explains the
trickery to anyone less versed in C...

Greetings,
Reimar D?ffinger




More information about the ffmpeg-devel mailing list