[FFmpeg-devel] GSOC update on the CDXL demuxer

Reimar Döffinger Reimar.Doeffinger
Wed Apr 15 10:21:20 CEST 2009


On Tue, Apr 14, 2009 at 08:07:54PM +0200, Reimar D?ffinger wrote:
> On Tue, Apr 14, 2009 at 11:39:59AM -0500, Erion Omeri wrote:
> >     //cdxl_read_only_header(pb, cdxl);
> 
> Well, you will have to read at least "chunk size" and "sound size"
> values from the header and skip the rest, otherwise you'll be reading
> your audio data from the wrong place...

Btw. after thinking about it a bit, I think it is better to read only
the things you need like width and height from the header in
read_header and then seek back to the start of the file.
Then in read_packet you can just read the header first and continue from
there.
That means less data in the private context (possibly none at all?),
using the generic index/seeking support will be easier and it should
also work better for supporting video, since I think you will have to
copy (most of) the header into the video packet anyway.



More information about the ffmpeg-devel mailing list