[MPlayer-cvslog] r35356 - in trunk/libass: ass.c ass.h ass_drawing.c ass_parse.c ass_parse.h ass_render.c ass_render.h ass_render_api.c ass_shaper.c ass_utils.c ass_utils.h

[Reimar Döffinger]

On Tue, Nov 06, 2012 at 06:41:14AM +0100, SubJunk wrote:
> @@ -639,27 +664,15 @@ void ass_shaper_shape(ASS_Shaper *shaper
>  #ifdef CONFIG_HARFBUZZ
>      switch (shaper->shaping_level) {
>      case ASS_SHAPING_SIMPLE:
> -        shape_fribidi(shaper, glyphs, text_info->length);
> +        ass_shaper_skip_characters(text_info);
>          break;
>      case ASS_SHAPING_COMPLEX:
>          shape_harfbuzz(shaper, glyphs, text_info->length);
>          break;
>      }
>  #else
> -        shape_fribidi(shaper, glyphs, text_info->length);
> +        ass_shaper_skip_characters(text_info);
>  #endif

I think you broke something here, now shape_fribidi is not used at all.
As a result I guess that right-to-left languages no longer work at all
with MPlayer.