[MPlayer-cvslog] r35426 - trunk/sub/subreader.c

cigaes subversion at mplayerhq.hu
Tue Nov 20 20:43:53 CET 2012


Author: cigaes
Date: Tue Nov 20 20:43:53 2012
New Revision: 35426

Log:
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.

Modified:
   trunk/sub/subreader.c

Modified: trunk/sub/subreader.c
==============================================================================
--- trunk/sub/subreader.c	Mon Nov 19 20:37:31 2012	(r35425)
+++ trunk/sub/subreader.c	Tue Nov 20 20:43:53 2012	(r35426)
@@ -677,7 +677,7 @@ static subtitle *sub_read_line_ssa(strea
             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;
           }


More information about the MPlayer-cvslog mailing list