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

[Benoit Fouet]

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, no ? and also break decoders people would have written using
lavf...

-- 
Benoit Fouet
Purple Labs S.A.
www.purplelabs.com