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

Ronald S. Bultje rsbultje
Tue Mar 10 00:17:53 CET 2009


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




More information about the ffmpeg-devel mailing list