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

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