]> git.pld-linux.org Git - packages/cairo.git/commitdiff
- Updated the lcd patch, taken from auto/th/cairo-1_6_4-2
authorqrczak <qrczak@pld-linux.org>
Fri, 13 Jun 2008 12:53:00 +0000 (12:53 +0000)
committercvs2git <feedback@pld-linux.org>
Sun, 24 Jun 2012 12:13:13 +0000 (12:13 +0000)
  http://bugs.freedesktop.org/show_bug.cgi?id=10301#c47
  It needs changes in ~/.fonts.conf (described there) to take effect,
  with lcddefault instead of lcdlegacy.
- Release 2.

Changed files:
    cairo-1.2.4-lcd-filter-1.patch -> 1.5
    cairo-lcd-filter.patch -> 1.1

cairo-1.2.4-lcd-filter-1.patch [deleted file]
cairo-lcd-filter.patch [new file with mode: 0644]

diff --git a/cairo-1.2.4-lcd-filter-1.patch b/cairo-1.2.4-lcd-filter-1.patch
deleted file mode 100644 (file)
index 5fffc6f..0000000
+++ /dev/null
@@ -1,710 +0,0 @@
---- cairo-1.4.12/src/cairo-ft-font.c.orig      2007-11-27 07:20:12.000000000 +0100
-+++ cairo-1.4.12/src/cairo-ft-font.c   2007-11-28 19:54:09.894901361 +0100
-@@ -55,6 +55,8 @@
- #include FT_SYNTHESIS_H
- #endif
-+#include FT_LCD_FILTER_H
-+
- #define DOUBLE_TO_26_6(d) ((FT_F26Dot6)((d) * 64.0))
- #define DOUBLE_FROM_26_6(t) ((double)(t) / 64.0)
- #define DOUBLE_TO_16_16(d) ((FT_Fixed)((d) * 65536.0))
-@@ -702,23 +704,300 @@
-     return CAIRO_STATUS_SUCCESS;
- }
--/* Empirically-derived subpixel filtering values thanks to Keith
-- * Packard and libXft. */
--static const int    filters[3][3] = {
--    /* red */
--#if 0
--    {    65538*4/7,65538*2/7,65538*1/7 },
--    /* green */
--    {    65536*1/4, 65536*2/4, 65537*1/4 },
--    /* blue */
--    {    65538*1/7,65538*2/7,65538*4/7 },
-+/* we sometimes need to convert the glyph bitmap in a FT_GlyphSlot
-+ * into a different format. For example, we want to convert a
-+ * FT_PIXEL_MODE_LCD or FT_PIXEL_MODE_LCD_V bitmap into a 32-bit
-+ * ARGB or ABGR bitmap.
-+ *
-+ * this function prepares a target descriptor for this operation.
-+ *
-+ * input :: target bitmap descriptor. The function will set its
-+ *          'width', 'rows' and 'pitch' fields, and only these
-+ *
-+ * slot  :: the glyph slot containing the source bitmap. this
-+ *          function assumes that slot->format == FT_GLYPH_FORMAT_BITMAP
-+ *
-+ * mode  :: the requested final rendering mode. supported values are
-+ *          MONO, NORMAL (i.e. gray), LCD and LCD_V
-+ *
-+ * the function returns the size in bytes of the corresponding buffer,
-+ * it's up to the caller to allocate the corresponding memory block
-+ * before calling _fill_xrender_bitmap
-+ *
-+ * it also returns -1 in case of error (e.g. incompatible arguments,
-+ * like trying to convert a gray bitmap into a monochrome one)
-+ */
-+static int
-+_compute_xrender_bitmap_size( FT_Bitmap*      target,
-+                              FT_GlyphSlot    slot,
-+                              FT_Render_Mode  mode )
-+{
-+    FT_Bitmap*  ftbit;
-+    int         width, height, pitch;
-+
-+    if ( slot->format != FT_GLYPH_FORMAT_BITMAP )
-+        return -1;
-+
-+    // compute the size of the final bitmap
-+    ftbit  = &slot->bitmap;
-+
-+    width  = ftbit->width;
-+    height = ftbit->rows;
-+    pitch  = (width+3) & ~3;
-+
-+    switch ( ftbit->pixel_mode )
-+    {
-+    case FT_PIXEL_MODE_MONO:
-+        if ( mode == FT_RENDER_MODE_MONO )
-+        {
-+          pitch = (((width+31) & ~31) >> 3);
-+          break;
-+        }
-+        /* fall-through */
-+
-+    case FT_PIXEL_MODE_GRAY:
-+        if ( mode == FT_RENDER_MODE_LCD   ||
-+            mode == FT_RENDER_MODE_LCD_V )
-+        {
-+            /* each pixel is replicated into a 32-bit ARGB value */
-+            pitch = width*4;
-+        }
-+        break;
-+
-+    case FT_PIXEL_MODE_LCD:
-+        if ( mode != FT_RENDER_MODE_LCD )
-+            return -1;
-+
-+      /* horz pixel triplets are packed into 32-bit ARGB values */
-+      width   /= 3;
-+      pitch    = width*4;
-+      break;
-+
-+    case FT_PIXEL_MODE_LCD_V:
-+        if ( mode != FT_RENDER_MODE_LCD_V )
-+            return -1;
-+
-+        /* vert pixel triplets are packed into 32-bit ARGB values */
-+        height  /= 3;
-+        pitch    = width*4;
-+        break;
-+
-+    default:  /* unsupported source format */
-+        return -1;
-+    }
-+
-+    target->width  = width;
-+    target->rows   = height;
-+    target->pitch  = pitch;
-+    target->buffer = NULL;
-+
-+    return pitch * height;
-+}
-+
-+/* this functions converts the glyph bitmap found in a FT_GlyphSlot
-+ * into a different format (see _compute_xrender_bitmap_size)
-+ *
-+ * you should call this function after _compute_xrender_bitmap_size
-+ *
-+ * target :: target bitmap descriptor. Note that its 'buffer' pointer
-+ *           must point to memory allocated by the caller
-+ *
-+ * slot   :: the glyph slot containing the source bitmap
-+ *
-+ * mode   :: the requested final rendering mode
-+ *
-+ * bgr    :: boolean, set if BGR or VBGR pixel ordering is needed
-+ */
-+static void
-+_fill_xrender_bitmap( FT_Bitmap*      target,
-+                      FT_GlyphSlot    slot,
-+                      FT_Render_Mode  mode,
-+                      int             bgr )
-+{
-+    FT_Bitmap*   ftbit = &slot->bitmap;
-+    unsigned char*   srcLine   = ftbit->buffer;
-+    unsigned char*   dstLine   = target->buffer;
-+    int              src_pitch = ftbit->pitch;
-+    int              width     = target->width;
-+    int              height    = target->rows;
-+    int              pitch     = target->pitch;
-+    int              subpixel;
-+    int              h;
-+
-+    subpixel = ( mode == FT_RENDER_MODE_LCD ||
-+        mode == FT_RENDER_MODE_LCD_V );
-+
-+    if ( src_pitch < 0 )
-+      srcLine -= src_pitch*(ftbit->rows-1);
-+
-+    target->pixel_mode = ftbit->pixel_mode;
-+
-+    switch ( ftbit->pixel_mode )
-+    {
-+    case FT_PIXEL_MODE_MONO:
-+        if ( subpixel )  /* convert mono to ARGB32 values */
-+        {
-+            for ( h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch )
-+            {
-+                int  x;
-+
-+                for ( x = 0; x < width; x++ )
-+                {
-+                    if ( srcLine[(x >> 3)] & (0x80 >> (x & 7)) )
-+                        ((unsigned int*)dstLine)[x] = 0xffffffffU;
-+                }
-+            }
-+            target->pixel_mode = FT_PIXEL_MODE_LCD;
-+        }
-+        else if ( mode == FT_RENDER_MODE_NORMAL )  /* convert mono to 8-bit gray */
-+        {
-+            for ( h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch )
-+            {
-+                int  x;
-+
-+                for ( x = 0; x < width; x++ )
-+                {
-+                    if ( srcLine[(x >> 3)] & (0x80 >> (x & 7)) )
-+                        dstLine[x] = 0xff;
-+                }
-+            }
-+            target->pixel_mode = FT_PIXEL_MODE_GRAY;
-+        }
-+        else  /* copy mono to mono */
-+        {
-+            int  bytes = (width+7) >> 3;
-+
-+            for ( h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch )
-+                memcpy( dstLine, srcLine, bytes );
-+        }
-+        break;
-+
-+    case FT_PIXEL_MODE_GRAY:
-+        if ( subpixel )  /* convert gray to ARGB32 values */
-+        {
-+            for ( h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch )
-+            {
-+                int            x;
-+                unsigned int*  dst = (unsigned int*)dstLine;
-+
-+                for ( x = 0; x < width; x++ )
-+                {
-+                    unsigned int  pix = srcLine[x];
-+
-+                    pix |= (pix << 8);
-+                    pix |= (pix << 16);
-+
-+                    dst[x] = pix;
-+                }
-+            }
-+            target->pixel_mode = FT_PIXEL_MODE_LCD;
-+        }
-+        else  /* copy gray into gray */
-+        {
-+            for ( h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch )
-+                memcpy( dstLine, srcLine, width );
-+        }
-+        break;
-+
-+    case FT_PIXEL_MODE_LCD:
-+        if ( !bgr )
-+        {
-+            /* convert horizontal RGB into ARGB32 */
-+            for ( h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch )
-+            {
-+                int            x;
-+                unsigned char* src = srcLine;
-+                unsigned int*  dst = (unsigned int*)dstLine;
-+
-+                for ( x = 0; x < width; x++, src += 3 )
-+                {
-+                    unsigned int  pix;
-+
-+                    pix = ((unsigned int)src[0] << 16) |
-+                          ((unsigned int)src[1] <<  8) |
-+                          ((unsigned int)src[2]      ) |
-+                          ((unsigned int)src[1] << 24) ;
-+
-+                    dst[x] = pix;
-+                }
-+            }
-+        }
-+        else
-+        {
-+          /* convert horizontal BGR into ARGB32 */
-+          for ( h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch )
-+          {
-+              int            x;
-+              unsigned char* src = srcLine;
-+              unsigned int*  dst = (unsigned int*)dstLine;
-+
-+              for ( x = 0; x < width; x++, src += 3 )
-+              {
-+                  unsigned int  pix;
-+
-+                  pix = ((unsigned int)src[2] << 16) |
-+                        ((unsigned int)src[1] <<  8) |
-+                        ((unsigned int)src[0]      ) |
-+                        ((unsigned int)src[1] << 24) ;
-+
-+                  dst[x] = pix;
-+              }
-+          }
-+        }
-+        break;
-+
-+    default:  /* FT_PIXEL_MODE_LCD_V */
-+        /* convert vertical RGB into ARGB32 */
-+        if ( !bgr )
-+        {
-+            for ( h = height; h > 0; h--, srcLine += 3*src_pitch, dstLine += pitch )
-+            {
-+                int            x;
-+                unsigned char* src = srcLine;
-+                unsigned int*  dst = (unsigned int*)dstLine;
-+
-+                for ( x = 0; x < width; x++, src += 1 )
-+                {
-+                    unsigned int  pix;
-+#if 1
-+                    pix = ((unsigned int)src[0]           << 16) |
-+                          ((unsigned int)src[src_pitch]   <<  8) |
-+                          ((unsigned int)src[src_pitch*2]      ) |
-+                          0xFF000000 ;
-+#else
-+                    pix = ((unsigned int)src[0]           << 16) |
-+                          ((unsigned int)src[src_pitch]   <<  8) |
-+                          ((unsigned int)src[src_pitch*2]      ) |
-+                          ((unsigned int)src[src_pitch]   << 24) ;
- #endif
--    {    65538*9/13,65538*3/13,65538*1/13 },
--    /* green */
--    {    65538*1/6, 65538*4/6, 65538*1/6 },
--    /* blue */
--    {    65538*1/13,65538*3/13,65538*9/13 },
--};
-+                    dst[x] = pix;
-+                }
-+            }
-+        }
-+        else
-+        {
-+            for ( h = height; h > 0; h--, srcLine += 3*src_pitch, dstLine += pitch )
-+            {
-+                int            x;
-+                unsigned char* src = srcLine;
-+                unsigned int*  dst = (unsigned int*)dstLine;
-+
-+                for ( x = 0; x < width; x++, src += 1 )
-+                {
-+                    unsigned int  pix;
-+
-+                    pix = ((unsigned int)src[src_pitch*2] << 16) |
-+                        ((unsigned int)src[src_pitch]   <<  8) |
-+                        ((unsigned int)src[0]                ) |
-+                        ((unsigned int)src[src_pitch]   << 24) ;
-+
-+                    dst[x] = pix;
-+                }
-+            }
-+        }
-+    }
-+}
-+
- /* Fills in val->image with an image surface created from @bitmap
-  */
-@@ -731,7 +1010,7 @@
-     int width, height, stride;
-     unsigned char *data;
-     int format = CAIRO_FORMAT_A8;
--    cairo_bool_t subpixel = FALSE;
-+    cairo_image_surface_t  *image;
-     width = bitmap->width;
-     height = bitmap->rows;
-@@ -772,7 +1051,6 @@
-               }
-           }
-       }
--
- #ifndef WORDS_BIGENDIAN
-       {
-           unsigned char   *d = data;
-@@ -784,17 +1062,15 @@
-           }
-       }
- #endif
-+
-       format = CAIRO_FORMAT_A1;
-       break;
-     case FT_PIXEL_MODE_LCD:
-     case FT_PIXEL_MODE_LCD_V:
-     case FT_PIXEL_MODE_GRAY:
--      switch (font_options->antialias) {
--      case CAIRO_ANTIALIAS_DEFAULT:
--      case CAIRO_ANTIALIAS_GRAY:
--      case CAIRO_ANTIALIAS_NONE:
--      default:
-+            if (font_options->antialias != CAIRO_ANTIALIAS_SUBPIXEL)
-+            {
-           stride = bitmap->pitch;
-           if (own_buffer) {
-               data = bitmap->buffer;
-@@ -807,105 +1083,16 @@
-               memcpy (data, bitmap->buffer, stride * height);
-           }
-           format = CAIRO_FORMAT_A8;
--          break;
--      case CAIRO_ANTIALIAS_SUBPIXEL: {
--          int             x, y;
--          unsigned char   *in_line, *out_line, *in;
--          unsigned int    *out;
--          unsigned int    red, green, blue;
--          int             rf, gf, bf;
--          int             s;
--          int             o, os;
--          unsigned char   *data_rgba;
--          unsigned int    width_rgba, stride_rgba;
--          int             vmul = 1;
--          int             hmul = 1;
--
--          switch (font_options->subpixel_order) {
--          case CAIRO_SUBPIXEL_ORDER_DEFAULT:
--          case CAIRO_SUBPIXEL_ORDER_RGB:
--          case CAIRO_SUBPIXEL_ORDER_BGR:
--          default:
--              width /= 3;
--              hmul = 3;
--              break;
--          case CAIRO_SUBPIXEL_ORDER_VRGB:
--          case CAIRO_SUBPIXEL_ORDER_VBGR:
--              vmul = 3;
--              height /= 3;
--              break;
--          }
--          /*
--           * Filter the glyph to soften the color fringes
--           */
--          width_rgba = width;
--          stride = bitmap->pitch;
--          stride_rgba = (width_rgba * 4 + 3) & ~3;
--          data_rgba = calloc (stride_rgba, height);
--          if (data_rgba == NULL) {
--              if (own_buffer)
--                  free (bitmap->buffer);
--              _cairo_error (CAIRO_STATUS_NO_MEMORY);
--              return CAIRO_STATUS_NO_MEMORY;
--          }
-+            } else {
-+                // if we get there, the  data from the source bitmap
-+                // really comes from _fill_xrender_bitmap, and is
-+                // made of 32-bit ARGB or ABGR values
-+                assert(own_buffer != 0);
-+                assert(bitmap->pixel_mode != FT_PIXEL_MODE_GRAY);
--          os = 1;
--          switch (font_options->subpixel_order) {
--          case CAIRO_SUBPIXEL_ORDER_VRGB:
--              os = stride;
--          case CAIRO_SUBPIXEL_ORDER_DEFAULT:
--          case CAIRO_SUBPIXEL_ORDER_RGB:
--          default:
--              rf = 0;
--              gf = 1;
--              bf = 2;
--              break;
--          case CAIRO_SUBPIXEL_ORDER_VBGR:
--              os = stride;
--          case CAIRO_SUBPIXEL_ORDER_BGR:
--              bf = 0;
--              gf = 1;
--              rf = 2;
--              break;
--          }
--          in_line = bitmap->buffer;
--          out_line = data_rgba;
--          for (y = 0; y < height; y++)
--          {
--              in = in_line;
--              out = (unsigned int *) out_line;
--              in_line += stride * vmul;
--              out_line += stride_rgba;
--              for (x = 0; x < width * hmul; x += hmul)
--              {
--                  red = green = blue = 0;
--                  o = 0;
--                  for (s = 0; s < 3; s++)
--                  {
--                      red += filters[rf][s]*in[x+o];
--                      green += filters[gf][s]*in[x+o];
--                      blue += filters[bf][s]*in[x+o];
--                      o += os;
--                  }
--                  red = red / 65536;
--                  green = green / 65536;
--                  blue = blue / 65536;
--                  *out++ = (green << 24) | (red << 16) | (green << 8) | blue;
--              }
--          }
--
--          /* Images here are stored in native format. The
--           * backend must convert to its own format as needed
--           */
--
--          if (own_buffer)
--              free (bitmap->buffer);
--          data = data_rgba;
--          stride = stride_rgba;
-+                data   = bitmap->buffer;
-+          stride = bitmap->pitch;
-           format = CAIRO_FORMAT_ARGB32;
--          subpixel = TRUE;
--          break;
--      }
-       }
-       break;
-     case FT_PIXEL_MODE_GRAY2:
-@@ -918,19 +1105,20 @@
-       return CAIRO_STATUS_NO_MEMORY;
-     }
--    *surface = (cairo_image_surface_t *)
-+        /* XXX */
-+        *surface = image = (cairo_image_surface_t *)
-       cairo_image_surface_create_for_data (data,
-                                            format,
-                                            width, height, stride);
--    if ((*surface)->base.status) {
-+        if (image->base.status) {
-       free (data);
-       return CAIRO_STATUS_NO_MEMORY;
-     }
--    if (subpixel)
--      pixman_image_set_component_alpha ((*surface)->pixman_image, TRUE);
-+        if (font_options->antialias == CAIRO_ANTIALIAS_SUBPIXEL)
-+            pixman_image_set_component_alpha (image->pixman_image, TRUE);
--    _cairo_image_surface_assume_ownership_of_data ((*surface));
-+        _cairo_image_surface_assume_ownership_of_data (image);
-     return CAIRO_STATUS_SUCCESS;
- }
-@@ -955,16 +1143,44 @@
-                      cairo_font_options_t      *font_options,
-                      cairo_image_surface_t    **surface)
- {
-+    int rgba = FC_RGBA_UNKNOWN;
-     FT_GlyphSlot glyphslot = face->glyph;
-     FT_Outline *outline = &glyphslot->outline;
-     FT_Bitmap bitmap;
-     FT_BBox cbox;
--    FT_Matrix matrix;
--    int hmul = 1;
--    int vmul = 1;
-     unsigned int width, height, stride;
--    cairo_bool_t subpixel = FALSE;
-+    cairo_format_t format;
-     cairo_status_t status;
-+    FT_Error  fterror;
-+    FT_Library  library = glyphslot->library;
-+    FT_Render_Mode  render_mode = FT_RENDER_MODE_NORMAL;
-+
-+    switch (font_options->antialias)
-+    {
-+    case CAIRO_ANTIALIAS_NONE:
-+        render_mode = FT_RENDER_MODE_MONO;
-+        break;
-+
-+    case CAIRO_ANTIALIAS_SUBPIXEL:
-+        switch (font_options->subpixel_order)
-+        {
-+            case CAIRO_SUBPIXEL_ORDER_DEFAULT:
-+            case CAIRO_SUBPIXEL_ORDER_RGB:
-+            case CAIRO_SUBPIXEL_ORDER_BGR:
-+                render_mode = FT_RENDER_MODE_LCD;
-+                break;
-+
-+            case CAIRO_SUBPIXEL_ORDER_VRGB:
-+            case CAIRO_SUBPIXEL_ORDER_VBGR:
-+                render_mode = FT_RENDER_MODE_LCD_V;
-+                break;
-+        }
-+        break;
-+
-+    case CAIRO_ANTIALIAS_DEFAULT:
-+    case CAIRO_ANTIALIAS_GRAY:
-+        render_mode = FT_RENDER_MODE_NORMAL;
-+    }
-     FT_Outline_Get_CBox (outline, &cbox);
-@@ -975,20 +1191,19 @@
-     width = (unsigned int) ((cbox.xMax - cbox.xMin) >> 6);
-     height = (unsigned int) ((cbox.yMax - cbox.yMin) >> 6);
--    stride = (width * hmul + 3) & ~3;
-+    stride = (width + 3) & ~3;
-     if (width * height == 0) {
--      cairo_format_t format;
-       /* Looks like fb handles zero-sized images just fine */
--      switch (font_options->antialias) {
--      case CAIRO_ANTIALIAS_NONE:
-+        switch (render_mode)
-+        {
-+          case FT_RENDER_MODE_MONO:
-           format = CAIRO_FORMAT_A1;
-           break;
--      case CAIRO_ANTIALIAS_SUBPIXEL:
--          format= CAIRO_FORMAT_ARGB32;
-+          case FT_RENDER_MODE_LCD:
-+          case FT_RENDER_MODE_LCD_V:
-+            format = CAIRO_FORMAT_ARGB32;
-           break;
--      case CAIRO_ANTIALIAS_DEFAULT:
--      case CAIRO_ANTIALIAS_GRAY:
-       default:
-           format = CAIRO_FORMAT_A8;
-           break;
-@@ -1000,75 +1215,70 @@
-           return CAIRO_STATUS_NO_MEMORY;
-     } else  {
--      matrix.xx = matrix.yy = 0x10000L;
--      matrix.xy = matrix.yx = 0;
-+        int  bitmap_size;
--      switch (font_options->antialias) {
--      case CAIRO_ANTIALIAS_NONE:
--          bitmap.pixel_mode = FT_PIXEL_MODE_MONO;
--          bitmap.num_grays  = 1;
--          stride = ((width + 31) & -32) >> 3;
-+        switch (render_mode)
-+        {
-+        case FT_RENDER_MODE_LCD:
-+            if (font_options->subpixel_order == CAIRO_SUBPIXEL_ORDER_BGR  ) {
-+                rgba = FC_RGBA_BGR;
-+            } else {
-+                rgba = FC_RGBA_RGB;
-+            }
-           break;
--      case CAIRO_ANTIALIAS_DEFAULT:
--      case CAIRO_ANTIALIAS_GRAY:
--          bitmap.pixel_mode = FT_PIXEL_MODE_GRAY;
--          bitmap.num_grays  = 256;
--          stride = (width + 3) & -4;
-+        case FT_RENDER_MODE_LCD_V:
-+            if (font_options->subpixel_order == CAIRO_SUBPIXEL_ORDER_VBGR ) {
-+                rgba = FC_RGBA_VBGR;
-+            } else {
-+                rgba = FC_RGBA_VRGB;
-+            }
-           break;
--      case CAIRO_ANTIALIAS_SUBPIXEL:
--          switch (font_options->subpixel_order) {
--          case CAIRO_SUBPIXEL_ORDER_RGB:
--          case CAIRO_SUBPIXEL_ORDER_BGR:
--          case CAIRO_SUBPIXEL_ORDER_DEFAULT:
-           default:
--              matrix.xx *= 3;
--              hmul = 3;
--              subpixel = TRUE;
-               break;
--          case CAIRO_SUBPIXEL_ORDER_VRGB:
--          case CAIRO_SUBPIXEL_ORDER_VBGR:
--              matrix.yy *= 3;
--              vmul = 3;
--              subpixel = TRUE;
--              break;
--          }
--          FT_Outline_Transform (outline, &matrix);
--
--          bitmap.pixel_mode = FT_PIXEL_MODE_GRAY;
--          bitmap.num_grays  = 256;
--          stride = (width * hmul + 3) & -4;
-       }
--      bitmap.pitch = stride;
--      bitmap.width = width * hmul;
--      bitmap.rows = height * vmul;
--      bitmap.buffer = calloc (stride, bitmap.rows);
-+        FT_Library_SetLcdFilter( library, FT_LCD_FILTER_DEFAULT );
-+
-+        fterror = FT_Render_Glyph( face->glyph, render_mode );
--      if (bitmap.buffer == NULL) {
-+        FT_Library_SetLcdFilter( library, FT_LCD_FILTER_NONE );
-+
-+        if (fterror != 0) {
-           _cairo_error (CAIRO_STATUS_NO_MEMORY);
-           return CAIRO_STATUS_NO_MEMORY;
-       }
--      FT_Outline_Translate (outline, -cbox.xMin*hmul, -cbox.yMin*vmul);
-+        bitmap_size = _compute_xrender_bitmap_size( &bitmap,
-+                                                    face->glyph,
-+                                                    render_mode );
-+        if ( bitmap_size < 0 ) {
-+          _cairo_error (CAIRO_STATUS_NO_MEMORY);
-+            return CAIRO_STATUS_NO_MEMORY;
-+      }
--      if (FT_Outline_Get_Bitmap (glyphslot->library, outline, &bitmap) != 0) {
-+        bitmap.buffer = calloc(1, bitmap_size);
-+        if (bitmap.buffer == NULL) {
-           free (bitmap.buffer);
-           _cairo_error (CAIRO_STATUS_NO_MEMORY);
-           return CAIRO_STATUS_NO_MEMORY;
-       }
-+        _fill_xrender_bitmap( &bitmap, face->glyph, render_mode,
-+                              (rgba == FC_RGBA_BGR || rgba == FC_RGBA_VBGR) );
-+
-+        // NOTE: _get_bitmap_surface will free bitmap.buffer if there is an error
-       status = _get_bitmap_surface (&bitmap, TRUE, font_options, surface);
-       if (status)
-           return status;
--    }
-     /*
-      * Note: the font's coordinate system is upside down from ours, so the
-      * Y coordinate of the control box needs to be negated.
-      */
-     cairo_surface_set_device_offset (&(*surface)->base,
--                                   floor ((double) cbox.xMin / 64.0),
--                                   floor (-(double) cbox.yMax / 64.0));
-+                                         (double) glyphslot->bitmap_left,
-+                                         (double)-glyphslot->bitmap_top);
-+    }
-     return CAIRO_STATUS_SUCCESS;
- }
-@@ -1442,11 +1652,11 @@
-               case CAIRO_SUBPIXEL_ORDER_DEFAULT:
-               case CAIRO_SUBPIXEL_ORDER_RGB:
-               case CAIRO_SUBPIXEL_ORDER_BGR:
--                  load_target |= FT_LOAD_TARGET_LCD;
-+                  load_target = FT_LOAD_TARGET_LCD;
-                   break;
-               case CAIRO_SUBPIXEL_ORDER_VRGB:
-               case CAIRO_SUBPIXEL_ORDER_VBGR:
--                  load_target |= FT_LOAD_TARGET_LCD_V;
-+                  load_target = FT_LOAD_TARGET_LCD_V;
-               break;
-               }
-           }
diff --git a/cairo-lcd-filter.patch b/cairo-lcd-filter.patch
new file mode 100644 (file)
index 0000000..32e1fb6
--- /dev/null
@@ -0,0 +1,1050 @@
+diff --git a/doc/public/cairo-sections.txt b/doc/public/cairo-sections.txt
+index 98cc974..d40790b 100644
+--- a/doc/public/cairo-sections.txt
++++ b/doc/public/cairo-sections.txt
+@@ -304,6 +304,9 @@ cairo_font_options_get_antialias
+ cairo_subpixel_order_t
+ cairo_font_options_set_subpixel_order
+ cairo_font_options_get_subpixel_order
++cairo_lcd_filter_t
++cairo_font_options_set_lcd_filter
++cairo_font_options_get_lcd_filter
+ cairo_hint_style_t
+ cairo_font_options_set_hint_style
+ cairo_font_options_get_hint_style
+diff --git a/src/cairo-font-options.c b/src/cairo-font-options.c
+index dedd337..9e47a9c 100644
+--- a/src/cairo-font-options.c
++++ b/src/cairo-font-options.c
+@@ -39,6 +39,7 @@
+ static const cairo_font_options_t _cairo_font_options_nil = {
+     CAIRO_ANTIALIAS_DEFAULT,
+     CAIRO_SUBPIXEL_ORDER_DEFAULT,
++    CAIRO_LCD_FILTER_DEFAULT,
+     CAIRO_HINT_STYLE_DEFAULT,
+     CAIRO_HINT_METRICS_DEFAULT
+ };
+@@ -54,6 +55,7 @@ _cairo_font_options_init_default (cairo_font_options_t *options)
+ {
+     options->antialias = CAIRO_ANTIALIAS_DEFAULT;
+     options->subpixel_order = CAIRO_SUBPIXEL_ORDER_DEFAULT;
++    options->lcd_filter = CAIRO_LCD_FILTER_DEFAULT;
+     options->hint_style = CAIRO_HINT_STYLE_DEFAULT;
+     options->hint_metrics = CAIRO_HINT_METRICS_DEFAULT;
+ }
+@@ -64,6 +66,7 @@ _cairo_font_options_init_copy (cairo_font_options_t          *options,
+ {
+     options->antialias = other->antialias;
+     options->subpixel_order = other->subpixel_order;
++    options->lcd_filter = other->lcd_filter;
+     options->hint_style = other->hint_style;
+     options->hint_metrics = other->hint_metrics;
+ }
+@@ -189,6 +192,8 @@ cairo_font_options_merge (cairo_font_options_t       *options,
+       options->antialias = other->antialias;
+     if (other->subpixel_order != CAIRO_SUBPIXEL_ORDER_DEFAULT)
+       options->subpixel_order = other->subpixel_order;
++    if (other->lcd_filter != CAIRO_LCD_FILTER_DEFAULT)
++      options->lcd_filter = other->lcd_filter;
+     if (other->hint_style != CAIRO_HINT_STYLE_DEFAULT)
+       options->hint_style = other->hint_style;
+     if (other->hint_metrics != CAIRO_HINT_METRICS_DEFAULT)
+@@ -221,6 +226,7 @@ cairo_font_options_equal (const cairo_font_options_t *options,
+     return (options->antialias == other->antialias &&
+           options->subpixel_order == other->subpixel_order &&
++          options->lcd_filter == other->lcd_filter &&
+           options->hint_style == other->hint_style &&
+           options->hint_metrics == other->hint_metrics);
+ }
+@@ -246,7 +252,8 @@ cairo_font_options_hash (const cairo_font_options_t *options)
+     return ((options->antialias) |
+           (options->subpixel_order << 4) |
+-          (options->hint_style << 8) |
++          (options->lcd_filter << 8) |
++          (options->hint_style << 12) |
+           (options->hint_metrics << 16));
+ }
+ slim_hidden_def (cairo_font_options_hash);
+@@ -328,6 +335,44 @@ cairo_font_options_get_subpixel_order (const cairo_font_options_t *options)
+ }
+ /**
++ * cairo_font_options_set_lcd_filter:
++ * @options: a #cairo_font_options_t
++ * @lcd_filter: the new LCD filter
++ *
++ * Sets the LCD filter for the font options object. The LCD filter
++ * specifies how pixels are filtered when rendered with an antialiasing
++ * mode of %CAIRO_ANTIALIAS_SUBPIXEL. See the documentation for
++ * #cairo_lcd_filter_t for full details.
++ **/
++void
++cairo_font_options_set_lcd_filter (cairo_font_options_t *options,
++                                 cairo_lcd_filter_t    lcd_filter)
++{
++    if (cairo_font_options_status (options))
++      return;
++
++    options->lcd_filter = lcd_filter;
++}
++
++/**
++ * cairo_font_options_get_lcd_filter:
++ * @options: a #cairo_font_options_t
++ *
++ * Gets the LCD filter for the font options object.
++ * See the documentation for #cairo_lcd_filter_t for full details.
++ *
++ * Return value: the LCD filter for the font options object
++ **/
++cairo_lcd_filter_t
++cairo_font_options_get_lcd_filter (const cairo_font_options_t *options)
++{
++    if (cairo_font_options_status ((cairo_font_options_t *) options))
++      return CAIRO_LCD_FILTER_DEFAULT;
++
++    return options->lcd_filter;
++}
++
++/**
+  * cairo_font_options_set_hint_style:
+  * @options: a #cairo_font_options_t
+  * @hint_style: the new hint style
+diff --git a/src/cairo-ft-font.c b/src/cairo-ft-font.c
+index 95da10b..eb35f67 100644
+--- a/src/cairo-ft-font.c
++++ b/src/cairo-ft-font.c
+@@ -56,6 +56,8 @@
+ #include FT_SYNTHESIS_H
+ #endif
++#include FT_LCD_FILTER_H
++
+ #define DOUBLE_TO_26_6(d) ((FT_F26Dot6)((d) * 64.0))
+ #define DOUBLE_FROM_26_6(t) ((double)(t) / 64.0)
+ #define DOUBLE_TO_16_16(d) ((FT_Fixed)((d) * 65536.0))
+@@ -724,23 +726,286 @@ _cairo_ft_unscaled_font_set_scale (cairo_ft_unscaled_font_t *unscaled,
+     return CAIRO_STATUS_SUCCESS;
+ }
+-/* Empirically-derived subpixel filtering values thanks to Keith
+- * Packard and libXft. */
+-static const int    filters[3][3] = {
+-    /* red */
+-#if 0
+-    {    65538*4/7,65538*2/7,65538*1/7 },
+-    /* green */
+-    {    65536*1/4, 65536*2/4, 65537*1/4 },
+-    /* blue */
+-    {    65538*1/7,65538*2/7,65538*4/7 },
++/* we sometimes need to convert the glyph bitmap in a FT_GlyphSlot
++ * into a different format. For example, we want to convert a
++ * FT_PIXEL_MODE_LCD or FT_PIXEL_MODE_LCD_V bitmap into a 32-bit
++ * ARGB or ABGR bitmap.
++ *
++ * this function prepares a target descriptor for this operation.
++ *
++ * input :: target bitmap descriptor. The function will set its
++ *          'width', 'rows' and 'pitch' fields, and only these
++ *
++ * slot  :: the glyph slot containing the source bitmap. this
++ *          function assumes that slot->format == FT_GLYPH_FORMAT_BITMAP
++ *
++ * mode  :: the requested final rendering mode. supported values are
++ *          MONO, NORMAL (i.e. gray), LCD and LCD_V
++ *
++ * the function returns the size in bytes of the corresponding buffer,
++ * it's up to the caller to allocate the corresponding memory block
++ * before calling _fill_xrender_bitmap
++ *
++ * it also returns -1 in case of error (e.g. incompatible arguments,
++ * like trying to convert a gray bitmap into a monochrome one)
++ */
++static int
++_compute_xrender_bitmap_size(FT_Bitmap      *target,
++                           FT_GlyphSlot    slot,
++                           FT_Render_Mode  mode)
++{
++    FT_Bitmap *ftbit;
++    int width, height, pitch;
++
++    if (slot->format != FT_GLYPH_FORMAT_BITMAP)
++      return -1;
++
++    // compute the size of the final bitmap
++    ftbit = &slot->bitmap;
++
++    width = ftbit->width;
++    height = ftbit->rows;
++    pitch = (width + 3) & ~3;
++
++    switch (ftbit->pixel_mode) {
++    case FT_PIXEL_MODE_MONO:
++      if (mode == FT_RENDER_MODE_MONO) {
++          pitch = (((width + 31) & ~31) >> 3);
++          break;
++      }
++      /* fall-through */
++
++    case FT_PIXEL_MODE_GRAY:
++      if (mode == FT_RENDER_MODE_LCD ||
++          mode == FT_RENDER_MODE_LCD_V)
++      {
++          /* each pixel is replicated into a 32-bit ARGB value */
++          pitch = width * 4;
++      }
++      break;
++
++    case FT_PIXEL_MODE_LCD:
++      if (mode != FT_RENDER_MODE_LCD)
++          return -1;
++
++      /* horz pixel triplets are packed into 32-bit ARGB values */
++      width /= 3;
++      pitch = width * 4;
++      break;
++
++    case FT_PIXEL_MODE_LCD_V:
++      if (mode != FT_RENDER_MODE_LCD_V)
++          return -1;
++
++      /* vert pixel triplets are packed into 32-bit ARGB values */
++      height /= 3;
++      pitch = width * 4;
++      break;
++
++    default:  /* unsupported source format */
++      return -1;
++    }
++
++    target->width = width;
++    target->rows = height;
++    target->pitch = pitch;
++    target->buffer = NULL;
++
++    return pitch * height;
++}
++
++/* this functions converts the glyph bitmap found in a FT_GlyphSlot
++ * into a different format (see _compute_xrender_bitmap_size)
++ *
++ * you should call this function after _compute_xrender_bitmap_size
++ *
++ * target :: target bitmap descriptor. Note that its 'buffer' pointer
++ *           must point to memory allocated by the caller
++ *
++ * slot   :: the glyph slot containing the source bitmap
++ *
++ * mode   :: the requested final rendering mode
++ *
++ * bgr    :: boolean, set if BGR or VBGR pixel ordering is needed
++ */
++static void
++_fill_xrender_bitmap(FT_Bitmap      *target,
++                   FT_GlyphSlot    slot,
++                   FT_Render_Mode  mode,
++                   int             bgr)
++{
++    FT_Bitmap *ftbit = &slot->bitmap;
++    unsigned char *srcLine = ftbit->buffer;
++    unsigned char *dstLine = target->buffer;
++    int src_pitch = ftbit->pitch;
++    int width = target->width;
++    int height = target->rows;
++    int pitch = target->pitch;
++    int subpixel;
++    int h;
++
++    subpixel = (mode == FT_RENDER_MODE_LCD ||
++              mode == FT_RENDER_MODE_LCD_V);
++
++    if (src_pitch < 0)
++      srcLine -= src_pitch * (ftbit->rows - 1);
++
++    target->pixel_mode = ftbit->pixel_mode;
++
++    switch (ftbit->pixel_mode) {
++    case FT_PIXEL_MODE_MONO:
++      if (subpixel) {
++          /* convert mono to ARGB32 values */
++
++          for (h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch) {
++              int x;
++
++              for (x = 0; x < width; x++) {
++                  if (srcLine[(x >> 3)] & (0x80 >> (x & 7)))
++                      ((unsigned int *) dstLine)[x] = 0xffffffffU;
++              }
++          }
++          target->pixel_mode = FT_PIXEL_MODE_LCD;
++
++      } else if (mode == FT_RENDER_MODE_NORMAL) {
++          /* convert mono to 8-bit gray */
++
++          for (h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch) {
++              int x;
++
++              for (x = 0; x < width; x++) {
++                  if (srcLine[(x >> 3)] & (0x80 >> (x & 7)))
++                      dstLine[x] = 0xff;
++              }
++          }
++          target->pixel_mode = FT_PIXEL_MODE_GRAY;
++
++      } else {
++          /* copy mono to mono */
++
++          int  bytes = (width + 7) >> 3;
++
++          for (h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch)
++              memcpy (dstLine, srcLine, bytes);
++      }
++      break;
++
++    case FT_PIXEL_MODE_GRAY:
++      if (subpixel) {
++          /* convert gray to ARGB32 values */
++
++          for (h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch) {
++              int x;
++              unsigned int *dst = (unsigned int *) dstLine;
++
++              for (x = 0; x < width; x++) {
++                  unsigned int pix = srcLine[x];
++
++                  pix |= (pix << 8);
++                  pix |= (pix << 16);
++
++                  dst[x] = pix;
++              }
++          }
++          target->pixel_mode = FT_PIXEL_MODE_LCD;
++        } else {
++            /* copy gray into gray */
++
++            for (h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch)
++                memcpy (dstLine, srcLine, width);
++        }
++        break;
++
++    case FT_PIXEL_MODE_LCD:
++      if (!bgr) {
++          /* convert horizontal RGB into ARGB32 */
++
++          for (h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch) {
++              int x;
++              unsigned char *src = srcLine;
++              unsigned int *dst = (unsigned int *) dstLine;
++
++              for (x = 0; x < width; x++, src += 3) {
++                  unsigned int  pix;
++
++                  pix = ((unsigned int)src[0] << 16) |
++                        ((unsigned int)src[1] <<  8) |
++                        ((unsigned int)src[2]      ) |
++                        ((unsigned int)src[1] << 24) ;
++
++                  dst[x] = pix;
++              }
++          }
++      } else {
++          /* convert horizontal BGR into ARGB32 */
++
++          for (h = height; h > 0; h--, srcLine += src_pitch, dstLine += pitch) {
++
++              int x;
++              unsigned char *src = srcLine;
++              unsigned int *dst = (unsigned int *) dstLine;
++
++              for (x = 0; x < width; x++, src += 3) {
++                  unsigned int  pix;
++
++                  pix = ((unsigned int)src[2] << 16) |
++                        ((unsigned int)src[1] <<  8) |
++                        ((unsigned int)src[0]      ) |
++                        ((unsigned int)src[1] << 24) ;
++
++                  dst[x] = pix;
++              }
++          }
++      }
++      break;
++
++    default:  /* FT_PIXEL_MODE_LCD_V */
++      /* convert vertical RGB into ARGB32 */
++      if (!bgr) {
++
++          for (h = height; h > 0; h--, srcLine += 3 * src_pitch, dstLine += pitch) {
++              int x;
++              unsigned char* src = srcLine;
++              unsigned int*  dst = (unsigned int *) dstLine;
++
++              for (x = 0; x < width; x++, src += 1) {
++                  unsigned int pix;
++#if 1
++                  pix = ((unsigned int)src[0]           << 16) |
++                        ((unsigned int)src[src_pitch]   <<  8) |
++                        ((unsigned int)src[src_pitch*2]      ) |
++                        0xFF000000 ;
++#else
++                  pix = ((unsigned int)src[0]           << 16) |
++                        ((unsigned int)src[src_pitch]   <<  8) |
++                        ((unsigned int)src[src_pitch*2]      ) |
++                        ((unsigned int)src[src_pitch]   << 24) ;
+ #endif
+-    {    65538*9/13,65538*3/13,65538*1/13 },
+-    /* green */
+-    {    65538*1/6, 65538*4/6, 65538*1/6 },
+-    /* blue */
+-    {    65538*1/13,65538*3/13,65538*9/13 },
+-};
++                  dst[x] = pix;
++              }
++          }
++      } else {
++
++          for (h = height; h > 0; h--, srcLine += 3*src_pitch, dstLine += pitch) {
++              int x;
++              unsigned char *src = srcLine;
++              unsigned int *dst = (unsigned int *) dstLine;
++
++              for (x = 0; x < width; x++, src += 1) {
++                  unsigned int  pix;
++
++                  pix = ((unsigned int)src[src_pitch * 2] << 16) |
++                        ((unsigned int)src[src_pitch]     <<  8) |
++                        ((unsigned int)src[0]                  ) |
++                        ((unsigned int)src[src_pitch]     << 24) ;
++
++                  dst[x] = pix;
++              }
++          }
++      }
++    }
++}
++
+ /* Fills in val->image with an image surface created from @bitmap
+  */
+@@ -753,7 +1018,7 @@ _get_bitmap_surface (FT_Bitmap                 *bitmap,
+     int width, height, stride;
+     unsigned char *data;
+     int format = CAIRO_FORMAT_A8;
+-    cairo_bool_t subpixel = FALSE;
++    cairo_image_surface_t *image;
+     width = bitmap->width;
+     height = bitmap->rows;
+@@ -810,11 +1075,7 @@ _get_bitmap_surface (FT_Bitmap                *bitmap,
+     case FT_PIXEL_MODE_LCD:
+     case FT_PIXEL_MODE_LCD_V:
+     case FT_PIXEL_MODE_GRAY:
+-      switch (font_options->antialias) {
+-      case CAIRO_ANTIALIAS_DEFAULT:
+-      case CAIRO_ANTIALIAS_GRAY:
+-      case CAIRO_ANTIALIAS_NONE:
+-      default:
++        if (font_options->antialias != CAIRO_ANTIALIAS_SUBPIXEL) {
+           stride = bitmap->pitch;
+           if (own_buffer) {
+               data = bitmap->buffer;
+@@ -826,104 +1087,16 @@ _get_bitmap_surface (FT_Bitmap              *bitmap,
+               memcpy (data, bitmap->buffer, stride * height);
+           }
+           format = CAIRO_FORMAT_A8;
+-          break;
+-      case CAIRO_ANTIALIAS_SUBPIXEL: {
+-          int             x, y;
+-          unsigned char   *in_line, *out_line, *in;
+-          unsigned int    *out;
+-          unsigned int    red, green, blue;
+-          int             rf, gf, bf;
+-          int             s;
+-          int             o, os;
+-          unsigned char   *data_rgba;
+-          unsigned int    width_rgba, stride_rgba;
+-          int             vmul = 1;
+-          int             hmul = 1;
+-
+-          switch (font_options->subpixel_order) {
+-          case CAIRO_SUBPIXEL_ORDER_DEFAULT:
+-          case CAIRO_SUBPIXEL_ORDER_RGB:
+-          case CAIRO_SUBPIXEL_ORDER_BGR:
+-          default:
+-              width /= 3;
+-              hmul = 3;
+-              break;
+-          case CAIRO_SUBPIXEL_ORDER_VRGB:
+-          case CAIRO_SUBPIXEL_ORDER_VBGR:
+-              vmul = 3;
+-              height /= 3;
+-              break;
+-          }
+-          /*
+-           * Filter the glyph to soften the color fringes
+-           */
+-          width_rgba = width;
++      } else {
++          // if we get there, the  data from the source bitmap
++          // really comes from _fill_xrender_bitmap, and is
++          // made of 32-bit ARGB or ABGR values
++          assert (own_buffer != 0);
++          assert (bitmap->pixel_mode != FT_PIXEL_MODE_GRAY);
++  
++          data = bitmap->buffer;
+           stride = bitmap->pitch;
+-          stride_rgba = (width_rgba * 4 + 3) & ~3;
+-          data_rgba = calloc (stride_rgba, height);
+-          if (data_rgba == NULL) {
+-              if (own_buffer)
+-                  free (bitmap->buffer);
+-              return _cairo_error (CAIRO_STATUS_NO_MEMORY);
+-          }
+-
+-          os = 1;
+-          switch (font_options->subpixel_order) {
+-          case CAIRO_SUBPIXEL_ORDER_VRGB:
+-              os = stride;
+-          case CAIRO_SUBPIXEL_ORDER_DEFAULT:
+-          case CAIRO_SUBPIXEL_ORDER_RGB:
+-          default:
+-              rf = 0;
+-              gf = 1;
+-              bf = 2;
+-              break;
+-          case CAIRO_SUBPIXEL_ORDER_VBGR:
+-              os = stride;
+-          case CAIRO_SUBPIXEL_ORDER_BGR:
+-              bf = 0;
+-              gf = 1;
+-              rf = 2;
+-              break;
+-          }
+-          in_line = bitmap->buffer;
+-          out_line = data_rgba;
+-          for (y = 0; y < height; y++)
+-          {
+-              in = in_line;
+-              out = (unsigned int *) out_line;
+-              in_line += stride * vmul;
+-              out_line += stride_rgba;
+-              for (x = 0; x < width * hmul; x += hmul)
+-              {
+-                  red = green = blue = 0;
+-                  o = 0;
+-                  for (s = 0; s < 3; s++)
+-                  {
+-                      red += filters[rf][s]*in[x+o];
+-                      green += filters[gf][s]*in[x+o];
+-                      blue += filters[bf][s]*in[x+o];
+-                      o += os;
+-                  }
+-                  red = red / 65536;
+-                  green = green / 65536;
+-                  blue = blue / 65536;
+-                  *out++ = (green << 24) | (red << 16) | (green << 8) | blue;
+-              }
+-          }
+-
+-          /* Images here are stored in native format. The
+-           * backend must convert to its own format as needed
+-           */
+-
+-          if (own_buffer)
+-              free (bitmap->buffer);
+-          data = data_rgba;
+-          stride = stride_rgba;
+           format = CAIRO_FORMAT_ARGB32;
+-          subpixel = TRUE;
+-          break;
+-      }
+       }
+       break;
+     case FT_PIXEL_MODE_GRAY2:
+@@ -935,19 +1108,20 @@ _get_bitmap_surface (FT_Bitmap               *bitmap,
+       return _cairo_error (CAIRO_STATUS_NO_MEMORY);
+     }
+-    *surface = (cairo_image_surface_t *)
++    /* XXX */
++    *surface = image = (cairo_image_surface_t *)
+       cairo_image_surface_create_for_data (data,
+                                            format,
+                                            width, height, stride);
+-    if ((*surface)->base.status) {
++    if (image->base.status) {
+       free (data);
+       return (*surface)->base.status;
+     }
+-    if (subpixel)
+-      pixman_image_set_component_alpha ((*surface)->pixman_image, TRUE);
++    if (font_options->antialias == CAIRO_ANTIALIAS_SUBPIXEL)
++      pixman_image_set_component_alpha (image->pixman_image, TRUE);
+-    _cairo_image_surface_assume_ownership_of_data ((*surface));
++    _cairo_image_surface_assume_ownership_of_data (image);
+     return CAIRO_STATUS_SUCCESS;
+ }
+@@ -972,16 +1146,59 @@ _render_glyph_outline (FT_Face                    face,
+                      cairo_font_options_t      *font_options,
+                      cairo_image_surface_t    **surface)
+ {
++    int rgba = FC_RGBA_UNKNOWN;
++    int lcd_filter = FT_LCD_FILTER_LEGACY;
+     FT_GlyphSlot glyphslot = face->glyph;
+     FT_Outline *outline = &glyphslot->outline;
+     FT_Bitmap bitmap;
+     FT_BBox cbox;
+-    FT_Matrix matrix;
+-    int hmul = 1;
+-    int vmul = 1;
+     unsigned int width, height, stride;
+-    cairo_bool_t subpixel = FALSE;
+     cairo_status_t status;
++    FT_Error fterror;
++    FT_Library library = glyphslot->library;
++    FT_Render_Mode render_mode = FT_RENDER_MODE_NORMAL;
++
++    switch (font_options->antialias) {
++    case CAIRO_ANTIALIAS_NONE:
++      render_mode = FT_RENDER_MODE_MONO;
++      break;
++
++    case CAIRO_ANTIALIAS_SUBPIXEL:
++      switch (font_options->subpixel_order) {
++          case CAIRO_SUBPIXEL_ORDER_DEFAULT:
++          case CAIRO_SUBPIXEL_ORDER_RGB:
++          case CAIRO_SUBPIXEL_ORDER_BGR:
++              render_mode = FT_RENDER_MODE_LCD;
++              break;
++
++          case CAIRO_SUBPIXEL_ORDER_VRGB:
++          case CAIRO_SUBPIXEL_ORDER_VBGR:
++              render_mode = FT_RENDER_MODE_LCD_V;
++              break;
++      }
++
++      switch (font_options->lcd_filter) {
++      case CAIRO_LCD_FILTER_NONE:
++          lcd_filter = FT_LCD_FILTER_NONE;
++          break;
++      case CAIRO_LCD_FILTER_DEFAULT:
++      case CAIRO_LCD_FILTER_INTRA_PIXEL:
++          lcd_filter = FT_LCD_FILTER_LEGACY;
++          break;
++      case CAIRO_LCD_FILTER_FIR3:
++          lcd_filter = FT_LCD_FILTER_LIGHT;
++          break;
++      case CAIRO_LCD_FILTER_FIR5:
++          lcd_filter = FT_LCD_FILTER_DEFAULT;
++          break;
++      }
++
++      break;
++
++    case CAIRO_ANTIALIAS_DEFAULT:
++    case CAIRO_ANTIALIAS_GRAY:
++      render_mode = FT_RENDER_MODE_NORMAL;
++    }
+     FT_Outline_Get_CBox (outline, &cbox);
+@@ -992,20 +1209,22 @@ _render_glyph_outline (FT_Face                    face,
+     width = (unsigned int) ((cbox.xMax - cbox.xMin) >> 6);
+     height = (unsigned int) ((cbox.yMax - cbox.yMin) >> 6);
+-    stride = (width * hmul + 3) & ~3;
++    stride = (width + 3) & ~3;
+     if (width * height == 0) {
+       cairo_format_t format;
+       /* Looks like fb handles zero-sized images just fine */
+-      switch (font_options->antialias) {
+-      case CAIRO_ANTIALIAS_NONE:
++      switch (render_mode) {
++      case FT_RENDER_MODE_MONO:
+           format = CAIRO_FORMAT_A1;
+           break;
+-      case CAIRO_ANTIALIAS_SUBPIXEL:
++      case FT_RENDER_MODE_LCD:
++      case FT_RENDER_MODE_LCD_V:
+           format= CAIRO_FORMAT_ARGB32;
+           break;
+-      case CAIRO_ANTIALIAS_DEFAULT:
+-      case CAIRO_ANTIALIAS_GRAY:
++      case FT_RENDER_MODE_LIGHT:
++      case FT_RENDER_MODE_NORMAL:
++      case FT_RENDER_MODE_MAX:
+       default:
+           format = CAIRO_FORMAT_A8;
+           break;
+@@ -1017,74 +1236,69 @@ _render_glyph_outline (FT_Face                    face,
+           return (*surface)->base.status;
+     } else  {
+-      matrix.xx = matrix.yy = 0x10000L;
+-      matrix.xy = matrix.yx = 0;
++      int bitmap_size;
+-      switch (font_options->antialias) {
+-      case CAIRO_ANTIALIAS_NONE:
+-          bitmap.pixel_mode = FT_PIXEL_MODE_MONO;
+-          bitmap.num_grays  = 1;
+-          stride = ((width + 31) & -32) >> 3;
+-          break;
+-      case CAIRO_ANTIALIAS_DEFAULT:
+-      case CAIRO_ANTIALIAS_GRAY:
+-          bitmap.pixel_mode = FT_PIXEL_MODE_GRAY;
+-          bitmap.num_grays  = 256;
+-          stride = (width + 3) & -4;
++      switch (render_mode) {
++      case FT_RENDER_MODE_LCD:
++          if (font_options->subpixel_order == CAIRO_SUBPIXEL_ORDER_BGR) {
++              rgba = FC_RGBA_BGR;
++          } else {
++              rgba = FC_RGBA_RGB;
++          }
+           break;
+-      case CAIRO_ANTIALIAS_SUBPIXEL:
+-          switch (font_options->subpixel_order) {
+-          case CAIRO_SUBPIXEL_ORDER_RGB:
+-          case CAIRO_SUBPIXEL_ORDER_BGR:
+-          case CAIRO_SUBPIXEL_ORDER_DEFAULT:
+-          default:
+-              matrix.xx *= 3;
+-              hmul = 3;
+-              subpixel = TRUE;
+-              break;
+-          case CAIRO_SUBPIXEL_ORDER_VRGB:
+-          case CAIRO_SUBPIXEL_ORDER_VBGR:
+-              matrix.yy *= 3;
+-              vmul = 3;
+-              subpixel = TRUE;
+-              break;
++      case FT_RENDER_MODE_LCD_V:
++          if (font_options->subpixel_order == CAIRO_SUBPIXEL_ORDER_VBGR) {
++              rgba = FC_RGBA_VBGR;
++          } else {
++              rgba = FC_RGBA_VRGB;
+           }
+-          FT_Outline_Transform (outline, &matrix);
+-
+-          bitmap.pixel_mode = FT_PIXEL_MODE_GRAY;
+-          bitmap.num_grays  = 256;
+-          stride = (width * hmul + 3) & -4;
++          break;
++      case FT_RENDER_MODE_MONO:
++      case FT_RENDER_MODE_LIGHT:
++      case FT_RENDER_MODE_NORMAL:
++      case FT_RENDER_MODE_MAX:
++      default:
++          break;
+       }
+-      bitmap.pitch = stride;
+-      bitmap.width = width * hmul;
+-      bitmap.rows = height * vmul;
+-      bitmap.buffer = calloc (stride, bitmap.rows);
+-      if (bitmap.buffer == NULL)
++      FT_Library_SetLcdFilter (library, lcd_filter);
++
++      fterror = FT_Render_Glyph (face->glyph, render_mode);
++
++      FT_Library_SetLcdFilter (library, FT_LCD_FILTER_NONE);
++
++      if (fterror != 0)
+           return _cairo_error (CAIRO_STATUS_NO_MEMORY);
+-      FT_Outline_Translate (outline, -cbox.xMin*hmul, -cbox.yMin*vmul);
++      bitmap_size = _compute_xrender_bitmap_size (&bitmap,
++                                                  face->glyph,
++                                                  render_mode);
++      if (bitmap_size < 0)
++          return _cairo_error (CAIRO_STATUS_NO_MEMORY);
+-      if (FT_Outline_Get_Bitmap (glyphslot->library, outline, &bitmap) != 0) {
+-          free (bitmap.buffer);
++      bitmap.buffer = calloc (1, bitmap_size);
++      if (bitmap.buffer == NULL)
+           return _cairo_error (CAIRO_STATUS_NO_MEMORY);
+-      }
++      _fill_xrender_bitmap (&bitmap, face->glyph, render_mode,
++                            (rgba == FC_RGBA_BGR || rgba == FC_RGBA_VBGR));
++
++      // NOTE: _get_bitmap_surface will free bitmap.buffer if there is an error
+       status = _get_bitmap_surface (&bitmap, TRUE, font_options, surface);
+       if (status)
+           return status;
+-    }
+-
+-    /*
+-     * Note: the font's coordinate system is upside down from ours, so the
+-     * Y coordinate of the control box needs to be negated.  Moreover, device
+-     * offsets are position of glyph origin relative to top left while xMin
+-     * and yMax are offsets of top left relative to origin.  Another negation.
+-     */
+-    cairo_surface_set_device_offset (&(*surface)->base,
+-                                   floor (-(double) cbox.xMin / 64.0),
+-                                   floor (+(double) cbox.yMax / 64.0));
++      /*
++       * Note: the font's coordinate system is upside down from ours, so the
++       * Y coordinate of the control box needs to be negated.  Moreover, device
++       * offsets are position of glyph origin relative to top left while xMin
++       * and yMax are offsets of top left relative to origin.  Another negation.
++       */
++      cairo_surface_set_device_offset (&(*surface)->base,
++                                       (double)-glyphslot->bitmap_left,
++                                       (double)+glyphslot->bitmap_top);
++    }
++    
+     return CAIRO_STATUS_SUCCESS;
+ }
+@@ -1270,6 +1484,15 @@ typedef struct _cairo_ft_scaled_font {
+ const cairo_scaled_font_backend_t cairo_ft_scaled_font_backend;
++/* Fontconfig version older than 2.6 didn't have these options */
++#ifndef FC_LCD_FILTER
++#define FC_LCD_FILTER "lcdfilter"
++#define FC_LCD_NONE   0
++#define FC_LCD_DEFAULT        1
++#define FC_LCD_LIGHT  2
++#define FC_LCD_LEGACY 3
++#endif
++
+ /* The load flags passed to FT_Load_Glyph control aspects like hinting and
+  * antialiasing. Here we compute them from the fields of a FcPattern.
+  */
+@@ -1303,6 +1526,7 @@ _get_pattern_ft_options (FcPattern *pattern, cairo_ft_options_t *ret)
+     
+     if (antialias) {
+       cairo_subpixel_order_t subpixel_order;
++      int lcd_filter;
+       /* disable hinting if requested */
+       if (FcPatternGetBool (pattern,
+@@ -1338,6 +1562,25 @@ _get_pattern_ft_options (FcPattern *pattern, cairo_ft_options_t *ret)
+           ft_options.base.antialias = CAIRO_ANTIALIAS_SUBPIXEL;
+       }
++      if (FcPatternGetInteger (pattern,
++                               FC_LCD_FILTER, 0, &lcd_filter) == FcResultMatch)
++      {
++          switch (lcd_filter) {
++          case FC_LCD_NONE:
++              ft_options.base.lcd_filter = CAIRO_LCD_FILTER_NONE;
++              break;
++          case FC_LCD_DEFAULT:
++              ft_options.base.lcd_filter = CAIRO_LCD_FILTER_FIR5;
++              break;
++          case FC_LCD_LIGHT:
++              ft_options.base.lcd_filter = CAIRO_LCD_FILTER_FIR3;
++              break;
++          case FC_LCD_LEGACY:
++              ft_options.base.lcd_filter = CAIRO_LCD_FILTER_INTRA_PIXEL;
++              break;
++          }
++      }
++
+ #ifdef FC_HINT_STYLE    
+       if (FcPatternGetInteger (pattern, 
+                                FC_HINT_STYLE, 0, &hintstyle) != FcResultMatch)
+@@ -1438,6 +1681,12 @@ _cairo_ft_options_merge (cairo_ft_options_t *options,
+     if (other->base.hint_style == CAIRO_HINT_STYLE_NONE)
+       options->base.hint_style = CAIRO_HINT_STYLE_NONE;
++    if (options->base.lcd_filter == CAIRO_LCD_FILTER_DEFAULT)
++      options->base.lcd_filter = other->base.lcd_filter;
++
++    if (other->base.lcd_filter == CAIRO_LCD_FILTER_NONE)
++      options->base.lcd_filter = CAIRO_LCD_FILTER_NONE;
++
+     if (options->base.antialias == CAIRO_ANTIALIAS_NONE) {
+       if (options->base.hint_style == CAIRO_HINT_STYLE_NONE)
+           load_flags |= FT_LOAD_NO_HINTING;
+@@ -1461,11 +1710,11 @@ _cairo_ft_options_merge (cairo_ft_options_t *options,
+               case CAIRO_SUBPIXEL_ORDER_DEFAULT:
+               case CAIRO_SUBPIXEL_ORDER_RGB:
+               case CAIRO_SUBPIXEL_ORDER_BGR:
+-                  load_target |= FT_LOAD_TARGET_LCD;
++                  load_target = FT_LOAD_TARGET_LCD;
+                   break;
+               case CAIRO_SUBPIXEL_ORDER_VRGB:
+               case CAIRO_SUBPIXEL_ORDER_VBGR:
+-                  load_target |= FT_LOAD_TARGET_LCD_V;
++                  load_target = FT_LOAD_TARGET_LCD_V;
+               break;
+               }
+           }
+@@ -2404,6 +2653,33 @@ _cairo_ft_font_options_substitute (const cairo_font_options_t *options,
+       }
+     }
++    if (options->lcd_filter != CAIRO_LCD_FILTER_DEFAULT)
++    {
++      if (FcPatternGet (pattern, FC_LCD_FILTER, 0, &v) == FcResultNoMatch)
++      {
++          int lcd_filter;
++
++          switch (options->lcd_filter) {
++          case CAIRO_LCD_FILTER_NONE:
++              lcd_filter = FT_LCD_FILTER_NONE;
++              break;
++          case CAIRO_LCD_FILTER_DEFAULT:
++          case CAIRO_LCD_FILTER_INTRA_PIXEL:
++              lcd_filter = FT_LCD_FILTER_LEGACY;
++              break;
++          case CAIRO_LCD_FILTER_FIR3:
++              lcd_filter = FT_LCD_FILTER_LIGHT;
++              break;
++          case CAIRO_LCD_FILTER_FIR5:
++              lcd_filter = FT_LCD_FILTER_DEFAULT;
++              break;
++          }
++
++          if (! FcPatternAddInteger (pattern, FC_LCD_FILTER, lcd_filter))
++              return _cairo_error (CAIRO_STATUS_NO_MEMORY);
++      }
++    }
++
+     if (options->hint_style != CAIRO_HINT_STYLE_DEFAULT)
+     {
+       if (FcPatternGet (pattern, FC_HINTING, 0, &v) == FcResultNoMatch)
+diff --git a/src/cairo-types-private.h b/src/cairo-types-private.h
+index 30f7476..35409c7 100644
+--- a/src/cairo-types-private.h
++++ b/src/cairo-types-private.h
+@@ -108,6 +108,7 @@ struct _cairo_array {
+ struct _cairo_font_options {
+     cairo_antialias_t antialias;
+     cairo_subpixel_order_t subpixel_order;
++    cairo_lcd_filter_t lcd_filter;
+     cairo_hint_style_t hint_style;
+     cairo_hint_metrics_t hint_metrics;
+ };
+diff --git a/src/cairo-xlib-screen.c b/src/cairo-xlib-screen.c
+index 60421cd..0fd4d8e 100644
+--- a/src/cairo-xlib-screen.c
++++ b/src/cairo-xlib-screen.c
+@@ -130,6 +130,15 @@ get_integer_default (Display    *dpy,
+ #define FC_HINT_FULL        3
+ #endif
++/* Fontconfig version older than 2.6 didn't have these options */
++#ifndef FC_LCD_FILTER
++#define FC_LCD_FILTER "lcdfilter"
++#define FC_LCD_NONE   0
++#define FC_LCD_DEFAULT        1
++#define FC_LCD_LIGHT  2
++#define FC_LCD_LEGACY 3
++#endif
++
+ static void
+ _cairo_xlib_init_screen_font_options (Display *dpy, cairo_xlib_screen_info_t *info)
+ {
+@@ -137,12 +146,21 @@ _cairo_xlib_init_screen_font_options (Display *dpy, cairo_xlib_screen_info_t *in
+     cairo_bool_t xft_antialias;
+     int xft_hintstyle;
+     int xft_rgba;
++    int xft_lcdfilter;
+     cairo_antialias_t antialias;
+     cairo_subpixel_order_t subpixel_order;
++    cairo_lcd_filter_t lcd_filter;
+     cairo_hint_style_t hint_style;
+     if (!get_boolean_default (dpy, "antialias", &xft_antialias))
+       xft_antialias = TRUE;
++ 
++    if (!get_integer_default (dpy, "lcdfilter", &xft_lcdfilter)) {
++      /* -1 is an non-existant Fontconfig constant used to differentiate
++       * the case when no lcdfilter property is available.
++       */
++      xft_lcdfilter = -1;
++    }
+     if (!get_boolean_default (dpy, "hinting", &xft_hinting))
+       xft_hinting = TRUE;
+@@ -226,6 +244,24 @@ _cairo_xlib_init_screen_font_options (Display *dpy, cairo_xlib_screen_info_t *in
+       subpixel_order = CAIRO_SUBPIXEL_ORDER_DEFAULT;
+     }
++    switch (xft_lcdfilter) {
++    case FC_LCD_NONE:
++      lcd_filter = CAIRO_LCD_FILTER_NONE;
++      break;
++    case FC_LCD_DEFAULT:
++      lcd_filter = CAIRO_LCD_FILTER_FIR5;
++      break;
++    case FC_LCD_LIGHT:
++      lcd_filter = CAIRO_LCD_FILTER_FIR3;
++      break;
++    case FC_LCD_LEGACY:
++      lcd_filter = CAIRO_LCD_FILTER_INTRA_PIXEL;
++      break;
++    default:
++      lcd_filter = CAIRO_LCD_FILTER_DEFAULT;
++      break;
++    }
++
+     if (xft_antialias) {
+       if (subpixel_order == CAIRO_SUBPIXEL_ORDER_DEFAULT)
+           antialias = CAIRO_ANTIALIAS_GRAY;
+@@ -238,6 +274,7 @@ _cairo_xlib_init_screen_font_options (Display *dpy, cairo_xlib_screen_info_t *in
+     cairo_font_options_set_hint_style (&info->font_options, hint_style);
+     cairo_font_options_set_antialias (&info->font_options, antialias);
+     cairo_font_options_set_subpixel_order (&info->font_options, subpixel_order);
++    cairo_font_options_set_lcd_filter (&info->font_options, lcd_filter);
+     cairo_font_options_set_hint_metrics (&info->font_options, CAIRO_HINT_METRICS_ON);
+ }
+diff --git a/src/cairo.h b/src/cairo.h
+index 4a3a773..6c25a60 100644
+--- a/src/cairo.h
++++ b/src/cairo.h
+@@ -953,6 +953,26 @@ typedef enum _cairo_subpixel_order {
+ } cairo_subpixel_order_t;
+ /**
++ * cairo_lcd_filter_t:
++ * @CAIRO_LCD_FILTER_DEFAULT: Use the default LCD filter for
++ *   font backend and target device
++ * @CAIRO_LCD_FILTER_NONE: Do not perform LCD filtering
++ * @CAIRO_LCD_FILTER_INTRA_PIXEL: Intra-pixel filter
++ * @CAIRO_LCD_FILTER_FIR3: FIR filter with a 3x3 kernel
++ * @CAIRO_LCD_FILTER_FIR5: FIR filter with a 5x5 kernel
++ *
++ * The LCD filter specifies the low-pass filter applied to LCD-optimized
++ * bitmaps generated with an antialiasing mode of %CAIRO_ANTIALIAS_SUBPIXEL.
++ **/
++typedef enum _cairo_lcd_filter {
++    CAIRO_LCD_FILTER_DEFAULT,
++    CAIRO_LCD_FILTER_NONE,
++    CAIRO_LCD_FILTER_INTRA_PIXEL,
++    CAIRO_LCD_FILTER_FIR3,
++    CAIRO_LCD_FILTER_FIR5
++} cairo_lcd_filter_t;
++
++/**
+  * cairo_hint_style_t:
+  * @CAIRO_HINT_STYLE_DEFAULT: Use the default hint style for
+  *   font backend and target device
+@@ -1058,6 +1078,12 @@ cairo_public cairo_subpixel_order_t
+ cairo_font_options_get_subpixel_order (const cairo_font_options_t *options);
+ cairo_public void
++cairo_font_options_set_lcd_filter (cairo_font_options_t   *options,
++                                 cairo_lcd_filter_t  lcd_filter);
++cairo_public cairo_lcd_filter_t
++cairo_font_options_get_lcd_filter (const cairo_font_options_t *options);
++
++cairo_public void
+ cairo_font_options_set_hint_style (cairo_font_options_t *options,
+                                  cairo_hint_style_t     hint_style);
+ cairo_public cairo_hint_style_t
This page took 0.248432 seconds and 4 git commands to generate.