Re: [FFmpeg-devel] [PATCH v2] lavf/jacosubdec: compute subtitle duration correctly

2019-02-20 Thread Michael Niedermayer
On Tue, Feb 19, 2019 at 10:29:40AM +0100, Paul B Mahol wrote: > On 2/19/19, Adam Sampson wrote: > > When a JACOsub subtitle has two timestamps, they represent its start and > > end times (http://unicorn.us.com/jacosub/jscripts.html#l_times); the > > duration is the difference between the two, not

Re: [FFmpeg-devel] [PATCH v2] lavf/jacosubdec: compute subtitle duration correctly

2019-02-19 Thread Paul B Mahol
On 2/19/19, Adam Sampson wrote: > When a JACOsub subtitle has two timestamps, they represent its start and > end times (http://unicorn.us.com/jacosub/jscripts.html#l_times); the > duration is the difference between the two, not the sum of the two. > > The subtitle end times in the FATE test for th

[FFmpeg-devel] [PATCH v2] lavf/jacosubdec: compute subtitle duration correctly

2019-02-19 Thread Adam Sampson
When a JACOsub subtitle has two timestamps, they represent its start and end times (http://unicorn.us.com/jacosub/jscripts.html#l_times); the duration is the difference between the two, not the sum of the two. The subtitle end times in the FATE test for this were wrong as a result; fix them too. (

[FFmpeg-devel] [PATCH v2] lavf/jacosubdec: compute subtitle duration correctly

2019-02-13 Thread Adam Sampson
When a JACOsub subtitle has two timestamps, they represent its start and end times (http://unicorn.us.com/jacosub/jscripts.html#l_times); the duration is the difference between the two, not the sum of the two. The subtitle end times in the FATE test for this were wrong as a result; fix them too. (