[FFmpeg-cvslog] r13899 - in trunk/libavformat: isom.c mov.c

[Michael Niedermayer]

On Mon, Jun 23, 2008 at 10:53:07AM +0200, Benoit Fouet wrote:
> Benoit Fouet wrote:
> > Baptiste Coudurier wrote:
> >   
> >> Hi,
> >>
> >> bcoudurier wrote:
> >>
> >>     
> >>> [...]
> >>>
> >>>  static const MOVParseTableEntry mov_default_parse_table[] = {
> >>> +{ MKTAG('a','v','s','s'), mov_read_extradata },
> >>>  { MKTAG('c','o','6','4'), mov_read_stco },
> >>>  { MKTAG('c','t','t','s'), mov_read_ctts }, /* composition time to sample */
> >>>  { MKTAG('d','i','n','f'), mov_read_default },
> >>>
> >>>       
> >> Btw this is really annoying to have to add specific atom parsing to
> >> retrieve extradata, applications cannot decode file with lavf, even if
> >> demuxer currently ouput correctly all samples.
> >>
> >> A generic way for exporting this is needed. Atm Im thinking of putting
> >> everything after 'stsd' generic fields in extradata. What do you guys
> >> think ?
> >>
> >>
> >>     
> >
> > well, that will need some rework in decoders, but would save a lot of
> > work in the demuxer
> >
> >   
> 
> well, on second thought... this will also break compatibility with other
> containers

yes


>, no ? and also break decoders people would have written using
> lavf...

yes

it would also break stream copy

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- 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-cvslog/attachments/20080623/a6835afc/attachment.pgp>