From 73618d11d4108e8c3f9ada2cf85bc48352e70eee Mon Sep 17 00:00:00 2001 From: cigaes Date: Tue, 20 Nov 2012 19:43:53 +0000 Subject: subreader: do not skip the first char of ASS fields. Without this change, fields that can be sometimes empty and sometimes not, such as the Effect field, are counted in an inconsistent way. Since the number of fields is used to find where the text starts, it leads to internal field arriving on the video. Bug reported anonymously on the users mailing list. git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@35426 b3059339-0415-0410-9bf9-f77b7e298cf2 --- sub/subreader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'sub') diff --git a/sub/subreader.c b/sub/subreader.c index adb12724c4..826bc3577b 100644 --- a/sub/subreader.c +++ b/sub/subreader.c @@ -672,7 +672,7 @@ static subtitle *sub_read_line_ssa(stream_t *st,subtitle *current, tmp = line2; if(!(tmp=strchr(++tmp, ','))) break; if(brace && brace < tmp) break; // comma inside command - if(*(++tmp) == ' ') break; + if(tmp[1] == ' ') break; /* a space after a comma means we're already in a sentence */ line2 = tmp; } -- cgit v1.2.3