Fontconfig

  1. Bug #164640
  2. Comment #63

Comment 63 for bug 164640

Revision history for this message
In , Freedesktop (freedesktop) wrote :

- In cairo_lcd_filter_t docs, both FIR's are documented as 5x5.

- Maybe put them in this order: DEFAULT, NONE, INTRA_PIXEL, FIR3, FIR5.

+ /* XXXsyp is this needed? */
+ if (other->base.lcd_filter == CAIRO_LCD_FILTER_NONE)
+ options->base.lcd_filter = CAIRO_LCD_FILTER_NONE;

Well, this is one of the controversial areas in the current cairo-ft code. I'm not sure, but following neighboring code, seems like this is correct. Remove the XXX.

Looks good to me. Can be merged! Just fixed the above as a followup patch in your tree and I'll fetch and push!