[FFmpeg-devel] [PATCH 1/2] matroskaenc: Add CuePoints for subtitle tracks

James Almer jamrial at gmail.com
Fri Sep 20 23:02:34 CEST 2013


On 20/09/13 5:49 PM, Michael Niedermayer wrote:
> On Thu, Sep 19, 2013 at 08:21:16PM -0300, James Almer wrote:
>> On 19/09/13 7:11 PM, Michael Niedermayer wrote:
>>> On Sat, Sep 14, 2013 at 05:59:21AM -0300, James Almer wrote:
>>>>
>>>> Signed-off-by: James Almer <jamrial at gmail.com>
>>>> ---
>>>>  libavformat/matroskaenc.c | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> Do these 2 patches fix/improve playback/seeking with some software?
>>> If so please put this information in the commit message
>>> and also explain how the issue and fix can be reproduced if you
>>> have a testcase
>>
>> I don't know if any software currently reads and uses cues for subtitle tracks, 
>> and considering how new the cueduration element is i doubt any software reads 
>> them either.
>> I'm essentially replicating mkvtoolnix muxing practices with this. Afaik, with 
>> proper demuxer/splitter support, the idea of having a cueduration entry in 
>> subtitle cue points is to improve the display of subtitles right after seeking 
>> to a given point in the stream.
> 
> if you say this matches existing practice then its ok
> 
> first patch applied
> second seems not to apply anymore

The fate test changed after the muxingapp patch from a few days ago.
I'll resubmit it later.

Regards.


More information about the ffmpeg-devel mailing list