[vlc-devel] [PATCH] Freetype: fix RTL

Thomas Guillem thomas at gllm.fr
Wed Apr 11 09:16:07 CEST 2018


Looks good to me (according to fribidi_reorder_line documentation).

On Wed, Apr 11, 2018, at 08:42, Salah-Eddin Shaban wrote:
> ---
>  modules/text_renderer/freetype/text_layout.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/modules/text_renderer/freetype/text_layout.c b/modules/
> text_renderer/freetype/text_layout.c
> index 1a28786d09..cf80c52810 100644
> --- a/modules/text_renderer/freetype/text_layout.c
> +++ b/modules/text_renderer/freetype/text_layout.c
> @@ -1194,7 +1194,7 @@ static int LayoutLine( filter_t *p_filter,
>      int i_ul_thickness = 0;
>  
>  #ifdef HAVE_FRIBIDI
> -    bool b_reordered = ( 0 ==
> +    bool b_reordered = ( 0 !=
>          fribidi_reorder_line( 0, &p_paragraph->p_types[i_first_char],
>                            1 + i_last_char - i_first_char,
>                            0, p_paragraph->paragraph_type,
> -- 
> 2.13.6
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list