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

[Michael Niedermayer]

On Mon, Mar 31, 2008 at 07:59:42PM +0200, Reimar D?ffinger wrote:
> 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?

As the only spot where this occurs is being fixed by elis patch, i dont see
any need for that mess in avutil :)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080401/8b8855af/attachment.pgp>