[FFmpeg-devel] [PATCH] movenc: Fix muxing of Apple Quicktime chapters.

Philip Langdale philipl at overt.org
Thu May 17 08:08:03 CEST 2012


On May 16, 2012, at 9:37 PM, Yusuke Nakamura <muken.the.vfrmaniac at gmail.com> wrote:

> 2012/5/14 Philip Langdale <philipl at overt.org>
> 
>> On Mon, 14 May 2012 09:48:06 +0900
>> Yusuke Nakamura <muken.the.vfrmaniac at gmail.com> wrote:
>> 
>>> 
>>> For 3GPP timed text, atom type tag of the sample description is
>>> 'tx3g', not 'text'.
>> 
>> Your statement is correct in isolation, but in the context of Apple's
>> chapter format, they use and expect 'text'. This isn't a real subtitle
>> track.
>> 
>> --phil
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> 
> 
> If you want to use 'text' tag, don't use 3gpp timed text structure.
> 
> I'm OK if it is undocumented and probably wrong.
> I'm not OK if it is documented and obviously wrong.
> 
> Now. the first patch is better than the second for me.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 

This is a little silly. We don't get to choose how Apple defined
their chapter format. It is what it is. Yes, it's a timed text track
with a 'text' tag and a timed text header that all known players
ignore, but at least some of which (the apple ones) still require.

Nothing I have included in this patch is discretionary, except for
correcting the field definitions in the header. If the design offends
your sensibilities, I sympathize; but we don't have a say if we
want to support the feature.

--phil


More information about the ffmpeg-devel mailing list