[FFmpeg-devel] MOV Muxer fix to respect the standard

[Måns Rullgård]

Larbi Joubala <larbi.joubala at resonate-mp4.com> writes:

> Reimar D?ffinger a ?crit :
>> On Fri, May 22, 2009 at 04:59:55PM +0200, Larbi Joubala wrote:
>>
>>> Index: movenc.c
>>> ===================================================================
>>> --- movenc.c	(revision 18893)
>>> +++ movenc.c	(working copy)
>>> @@ -1013,8 +1013,11 @@
>>>      put_be32(pb ,0); /* reserved */
>>>      put_be32(pb ,0); /* reserved */
>>>      put_be32(pb ,0); /* reserved */
>>> -    put_byte(pb, strlen(descr)); /* string counter */
>>> +    if (track && track->mode == MODE_MOV || !track)
>>> +        put_byte(pb, strlen(descr)); /* string counter */
>>>
>>
>> if (!track || track->mode == MODE_MOV)
>>
>> but anyway I doubt this should depend on some per-track property.
>
>>
>>
> You've got some reference about this?
> I didn't see it in the QT MOV standard or ISO 14496-12 standard.

Eh?  It's in the description of the 'hdlr' box/atom.  Why did you
suggest the change if you couldn't find it in the specs?

-- 
M?ns Rullg?rd
mans at mansr.com