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

[Ronald S. Bultje]

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
work, because that'd mean no code is needed in rtsp_qt.c relating to
any media-specific handling. so whatever "demuxer" does in
"MOVContext" should be complete unnecessary for the whole thing to
work, if that makes any sense.

Ronald