[MPlayer-dev-eng] [PATCH] SSA/ASS subtitles support

[Evgeniy Stepanov]

On Monday 03 April 2006 07:24, Alexander Strange wrote:
> The current patch actually ignores the border width in the SSA and
> sets it to 1. If I change that, it looks better at large sizes but
> still looks funny at small ones. I suspect this is partially because
> VSFilter uses a better method for subsampling chroma in 4:2:0 (but
> macosx displays in 4:2:2 anyway, so if the renderer supports that it
> would improve). The rest might be freetype's imperfect hinting; I
> think you have to actually enable the auto-hinter manually?

Can you try enabling the auto-hinter then ? With increased border width, it 
looks almost perfect to me already.
Must be FT_LOAD_FORCE_AUTOHINT option.

{\bordN} override value is used, style value isn't. I'll fix it.

> Also, why does it scale the font size by .6?

The text is really huge without it :)
With .6 it is about VSFilter's size.
Maybe the resolution values are wrong ? It could also be the reason for weird 
hinting.

> I disabled the cache and only had a noticeable performance loss in
> one movie, so having a correct-but-slower mode would be fine with me.
> It would also be useful in mencoder.

Try ftp://toro.imec.msu.ru/pub/users/eugeni/output.mkv with and without cache.

> Some other things:
> * I went and rebuilt mplayer with fontconfig on, and ass_fontconfig.c
> never matches any fonts, all of which fc-match worked with. I copied
> and pasted the fontconfig matching code from font_load_ft and it
> worked (if you add in an FC_INDEX query, since the old one ignored
> it). I can look into this further if you want.

Sure, that would be great.

> * Japanese subtitles clip out the side because it doesn't
> automatically linebreak them, since they have no whitespace. I think
> fribidi might handle this sort of thing, but I'm not sure about that.
> * This file asserts out: http://raspberryheaven.net/~adtrw/anime/
> Makoto%20Shinkai%20-%20Other%20Worlds.mkv
> ass_render.c:1200: failed assertion `frame_context.dmpi->stride[1] ==
> frame_context.dmpi->chroma_width'
> (The mistimed subtitles are my fault, not yours.)