Re: [Inkscape-devel] Re: [ft-devel] inkscape fails with FT221

[Wolfram Quester]

Hi David!

your patch to fix inkscape and FT221 contains the following hunk:

> @@ -474,12 +474,12 @@
>  
>      theFace=pango_ft2_font_get_face(pFont);
>  
> -     if ( theFace->units_per_EM == 0 ) return false; // bitmap font
> -     
>       if ( pFont == NULL ) return false;
>       
>       if ( theFace == NULL ) return false;
>  
> +    if ( !FT_IS_SCALABLE(theFace) ) return false;  // bitmap font
> +
>      TT_HoriHeader *hhea = (TT_HoriHeader*)FT_Get_Sfnt_Table(theFace, 
> ft_sfnt_hhea);
>      if (hhea == NULL) return false;
>      run = hhea->caret_Slope_Run;

In this file there are two more occurences of "theFace->units_per_EM == 0" 
to test if the font is a bitmap font. Should these be replaced by
"FT_IS_SCALABLE" too?

Thanks,

Wolfi

signature.asc
Description: Digital signature