[FFmpeg-devel] [PATCH] mov.c: factorize ff_mov_read_stsd_entries()

[Michael Niedermayer]

On Mon, Mar 09, 2009 at 07:17:53PM -0400, Ronald S. Bultje wrote:
> Hi Baptiste,
> 
> On Mon, Mar 9, 2009 at 6:57 PM, Baptiste Coudurier
> <baptiste.coudurier at gmail.com> wrote:
> > I think Palette must be in MOVStsd anyway, and palette stuff is
> > deprecated ...
> > Demuxer will gather palette from MOVStsd if it needs too, and update
> > MOVContext if needed.
> 

> I like this. Ideally, whatever we choose, it'd automatically update
> that structure (AVCodecContext?) that needs the palette for ffplay to

as a saftey reminder to everyone:
The decoder and demuxer can (and do for ffplay) run in seperate threads
updating fields in AVCodecContext (like the palette) during playback is
NOT possible the threads with a AVPacket que between them will entangle

all changes during playback must go through AVPackets, this is why all
the palette stuff has been deprecated.
palettes can either be passed through
* extradata (this implicates they never change during the file)
OR
* AVPackets

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is not what we do, but why we do it that matters.
-------------- 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/20090310/5edb922b/attachment.pgp>