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; }