[MPlayer-cvslog] r19649 - in trunk: libass/ass_mp.c libass/ass_mp.h libmpdemux/demux_mkv.c

[Reimar Döffinger]

Hello,
On Fri, Dec 22, 2006 at 12:31:14AM +0300, Evgeniy Stepanov wrote:
> handle_subtitles() joins multiple subtitles in mkv_d->subs, which is correct 
> for the old subtitles engine. On the other hand, libass wants each subtitle 
> separately, that's what clear_subtitles() is for. I admit this is ugly. 
> mkv_d->subs should not be used in ASS mode and subtitle parsing should be 
> reimplemented in libass. For now, lets put this line under "if 
> (ass_enabled)".

Maybe my subtitle cleanup patch can improve the situation (though in the
current version it probably breaks this (I did not yet test)).

Greetings,
Reimar Döffinger