[MPlayer-dev-eng] [patch] RT subtitle timestamp parsing buggy

Clément Bœsch u at pkh.me
Sun May 25 03:08:08 CEST 2014


On Sat, May 24, 2014 at 08:05:00PM +0200, Reimar Döffinger wrote:
> On Fri, Mar 21, 2014 at 05:16:35PM +0200, hhaamu at gmail.com wrote:
> > 1) sscanf sets a1, a2 etc even when the whole line does not match. The
> > lines that try to parse begin/end tags will partially match and set
> > those variables, leading to faulty timestamps. The variables should be
> > reset to zero after each match attempt.
> 
> That whole code is seriously horrible and scary.

RT is supported in FFmpeg if you want to replace it. I think mpv switched
to them somehow.

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <https://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20140525/c16deb96/attachment.asc>


More information about the MPlayer-dev-eng mailing list