# HG changeset patch # User cigaes # Date 1353440633 0 # Node ID d6b9570722978dc0058813c3b5aec94152947419 # Parent 2bc3693278cba80c1a5b0d24b8fad0d848f9941f 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. diff -r 2bc3693278cb -r d6b957072297 sub/subreader.c --- a/sub/subreader.c Mon Nov 19 19:37:31 2012 +0000 +++ b/sub/subreader.c Tue Nov 20 19:43:53 2012 +0000 @@ -677,7 +677,7 @@ 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; }