[MPlayer-dev-eng] Re: [PATCH] S_TEXT/SSA support for Matroska files.

[Tristan]

ChristianHJW wrote:
 > Just curious, what differences did you encounter ? We normally mux them
 > as they are, just the timecodes are stripped and put into matroska
 > blockgroup, together with block duration, IIRC ??

  That stripping is precisly what is different (I'm following your
documentation after all :P). Besides the fact that sub_read_line_ssa in
subreader.c needs a file and not a string, it relies on the fact that
the sscanf for the timestamps in SSA will not fail (and it shouldn't
with valid SSA ^.^). It just makes sense to me to write a new decoder
since all I'm after is the text. All the special SSA commands are
unsupported with MPlayer G1 anyways.

On a sidenote, I found a bug in my MVK/SSA decoder. Subs with the same
timestamp will conflict with each other, but SSA subs can have
conflicting timestamps as subs can be instructed to be displayed at
different places (see http://ld-anime.faireal.net/guide/mkv-eng). I'm
trying to work out a sensible solution as I write this.

- Tristan Willy