Update to cairo-1.6.4.

Major changes between the 1.4 and 1.6 branch:
- Now uses external "pixman" library instead of a bundled one.
- Improvements to PDF, SVG and PostScript output
- New support for arbitrary X server visuals
  (the 8-bit display support patch (patch-ae in pkgsrc) is integrated)
- rendering backend improvements
- various new API additions, while maintaining ABI compatibility
- New Quartz backend
- bug fixes

7 files changed, 34 insertions, 372 deletions

diff --git a/graphics/cairo/Makefile b/graphics/cairo/Makefile
index 6c317c2be99..35418db29cd 100644
--- a/graphics/cairo/Makefile
+++ b/graphics/cairo/Makefile
@@ -1,6 +1,6 @@
-# $NetBSD: Makefile,v 1.60 2008/04/12 22:43:01 jlam Exp $
+# $NetBSD: Makefile,v 1.61 2008/04/21 11:29:33 tnn Exp $
 
-DISTNAME=	cairo-1.4.14
+DISTNAME=	cairo-1.6.4
 CATEGORIES=	graphics
 MASTER_SITES=	http://cairographics.org/releases/
 
@@ -31,6 +31,7 @@ PLIST.carbon=		yes
 .include "options.mk"
 
 BUILDLINK_API_DEPENDS.glitz+=	glitz>=0.5.1
+BUILDLINK_API_DEPENDS.pixman+=	pixman>=0.10.0
 
 # For snprintf()
 CPPFLAGS.SunOS+=		-D__EXTENSIONS__
@@ -43,4 +44,5 @@ TEST_TARGET=			check
 .include "../../graphics/glitz/buildlink3.mk"
 .include "../../graphics/png/buildlink3.mk"
 .include "../../mk/pthread.buildlink3.mk"
+.include "../../x11/pixman/buildlink3.mk"
 .include "../../mk/bsd.pkg.mk"
diff --git a/graphics/cairo/PLIST b/graphics/cairo/PLIST
index aee90f7eafc..984456eae79 100644
--- a/graphics/cairo/PLIST
+++ b/graphics/cairo/PLIST
@@ -1,4 +1,4 @@
-@comment $NetBSD: PLIST,v 1.18 2008/04/12 22:43:01 jlam Exp $
+@comment $NetBSD: PLIST,v 1.19 2008/04/21 11:29:33 tnn Exp $
 ${PLIST.carbon}include/cairo/cairo-atsui.h
 include/cairo/cairo-deprecated.h
 include/cairo/cairo-features.h
@@ -45,6 +45,8 @@ share/gtk-doc/html/cairo/cairo-PNG-Support.html
 share/gtk-doc/html/cairo/cairo-Paths.html
 share/gtk-doc/html/cairo/cairo-Patterns.html
 share/gtk-doc/html/cairo/cairo-PostScript-Surfaces.html
+share/gtk-doc/html/cairo/cairo-Quartz-Fonts.html
+share/gtk-doc/html/cairo/cairo-Quartz-Surfaces.html
 share/gtk-doc/html/cairo/cairo-SVG-Surfaces.html
 share/gtk-doc/html/cairo/cairo-Scaled-Fonts.html
 share/gtk-doc/html/cairo/cairo-Text.html
@@ -65,11 +67,10 @@ share/gtk-doc/html/cairo/index.html
 share/gtk-doc/html/cairo/index.sgml
 share/gtk-doc/html/cairo/index-1.2.html
 share/gtk-doc/html/cairo/index-1.4.html
+share/gtk-doc/html/cairo/index-1.6.html
 share/gtk-doc/html/cairo/ix01.html
 share/gtk-doc/html/cairo/language-bindings.html
 share/gtk-doc/html/cairo/left.png
-share/gtk-doc/html/cairo/pt01.html
-share/gtk-doc/html/cairo/pt02.html
 share/gtk-doc/html/cairo/right.png
 share/gtk-doc/html/cairo/style.css
 share/gtk-doc/html/cairo/up.png
diff --git a/graphics/cairo/buildlink3.mk b/graphics/cairo/buildlink3.mk
index 907128057ad..3f5e027f255 100644
--- a/graphics/cairo/buildlink3.mk
+++ b/graphics/cairo/buildlink3.mk
@@ -1,4 +1,4 @@
-# $NetBSD: buildlink3.mk,v 1.26 2007/05/30 08:54:30 rillig Exp $
+# $NetBSD: buildlink3.mk,v 1.27 2008/04/21 11:29:33 tnn Exp $
 
 BUILDLINK_DEPTH:=	${BUILDLINK_DEPTH}+
 CAIRO_BUILDLINK3_MK:=	${CAIRO_BUILDLINK3_MK}+
@@ -12,7 +12,7 @@ BUILDLINK_PACKAGES+=	cairo
 BUILDLINK_ORDER:=	${BUILDLINK_ORDER} ${BUILDLINK_DEPTH}cairo
 
 .if ${CAIRO_BUILDLINK3_MK} == "+"
-BUILDLINK_API_DEPENDS.cairo+=	cairo>=1.0.0nb2
+BUILDLINK_API_DEPENDS.cairo+=	cairo>=1.6.4
 BUILDLINK_ABI_DEPENDS.cairo?=	cairo>=1.0.4nb1
 BUILDLINK_PKGSRCDIR.cairo?=	../../graphics/cairo
 
@@ -32,5 +32,6 @@ pkgbase := cairo
 .include "../../graphics/freetype2/buildlink3.mk"
 .include "../../graphics/glitz/buildlink3.mk"
 .include "../../graphics/png/buildlink3.mk"
+.include "../../x11/pixman/buildlink3.mk"
 
 BUILDLINK_DEPTH:=	${BUILDLINK_DEPTH:S/+$//}
diff --git a/graphics/cairo/distinfo b/graphics/cairo/distinfo
index 2052838cf8f..b91367fb474 100644
--- a/graphics/cairo/distinfo
+++ b/graphics/cairo/distinfo
@@ -1,8 +1,7 @@
-$NetBSD: distinfo,v 1.39 2008/03/12 20:48:11 drochner Exp $
+$NetBSD: distinfo,v 1.40 2008/04/21 11:29:33 tnn Exp $
 
-SHA1 (cairo-1.4.14.tar.gz) = 63310243ba8af949060b06c01fc28ee5471ed5a4
-RMD160 (cairo-1.4.14.tar.gz) = 9e7c87a292c889b06b29e9622be0a7daa3ddd6fd
-Size (cairo-1.4.14.tar.gz) = 3283608 bytes
-SHA1 (patch-aa) = 79c04674fde56522e1f09e5896fcef4c6b6a0d36
-SHA1 (patch-ab) = c6b98417270f7165aa3499d92d0ca9c19bdd4318
-SHA1 (patch-ae) = 0b5c35720f9b675f132ca4da8c7e6e2b69c2f1a1
+SHA1 (cairo-1.6.4.tar.gz) = 9d990fe39a125ceb07221623c237cd7015855d5c
+RMD160 (cairo-1.6.4.tar.gz) = ccae736a40d985b72bda7c2f0a98a0838020658e
+Size (cairo-1.6.4.tar.gz) = 5042623 bytes
+SHA1 (patch-aa) = a6e95dc17dd316236dd7197f5e743c733e57587d
+SHA1 (patch-ab) = 2a0a99846fab962b552efa79c9261927de792dfd
diff --git a/graphics/cairo/patches/patch-aa b/graphics/cairo/patches/patch-aa
index 52e3b00e7ba..82855799afd 100644
--- a/graphics/cairo/patches/patch-aa
+++ b/graphics/cairo/patches/patch-aa
@@ -1,19 +1,13 @@
-$NetBSD: patch-aa,v 1.10 2007/06/08 07:20:53 wiz Exp $
+$NetBSD: patch-aa,v 1.11 2008/04/21 11:29:33 tnn Exp $
 
---- src/Makefile.in.orig	2007-05-01 20:25:14.000000000 +0000
+--- src/Makefile.in.orig	2008-04-11 23:33:41.000000000 +0200
 +++ src/Makefile.in
-@@ -244,7 +244,7 @@ CAIRO_CAN_TEST_PDF_SURFACE_FALSE = @CAIR
- CAIRO_CAN_TEST_PDF_SURFACE_TRUE = @CAIRO_CAN_TEST_PDF_SURFACE_TRUE@
- CAIRO_CAN_TEST_SVG_SURFACE_FALSE = @CAIRO_CAN_TEST_SVG_SURFACE_FALSE@
- CAIRO_CAN_TEST_SVG_SURFACE_TRUE = @CAIRO_CAN_TEST_SVG_SURFACE_TRUE@
+@@ -288,9 +288,9 @@ AUTOHEADER = @AUTOHEADER@
+ AUTOMAKE = @AUTOMAKE@
+ AWK = @AWK@
+ CAIROPERF_LIBS = @CAIROPERF_LIBS@
 -CAIRO_CFLAGS = @CAIRO_CFLAGS@
 +CAIRO_CFLAGS = @CAIRO_CFLAGS@ ${PTHREAD_CFLAGS}
- CAIRO_HAS_ATSUI_FONT_FALSE = @CAIRO_HAS_ATSUI_FONT_FALSE@
- CAIRO_HAS_ATSUI_FONT_TRUE = @CAIRO_HAS_ATSUI_FONT_TRUE@
- CAIRO_HAS_BEOS_SURFACE_FALSE = @CAIRO_HAS_BEOS_SURFACE_FALSE@
-@@ -282,7 +282,7 @@ CAIRO_HAS_XLIB_SURFACE_TRUE = @CAIRO_HAS
- CAIRO_HAS_XLIB_XRENDER_SURFACE_FALSE = @CAIRO_HAS_XLIB_XRENDER_SURFACE_FALSE@
- CAIRO_HAS_XLIB_XRENDER_SURFACE_TRUE = @CAIRO_HAS_XLIB_XRENDER_SURFACE_TRUE@
  CAIRO_LDADD = @CAIRO_LDADD@
 -CAIRO_LIBS = @CAIRO_LIBS@
 +CAIRO_LIBS = @CAIRO_LIBS@ ${PTHREAD_LDFLAGS} ${PTHREAD_LIBS}
diff --git a/graphics/cairo/patches/patch-ab b/graphics/cairo/patches/patch-ab
index 54616fc41c8..dfd4ae77091 100644
--- a/graphics/cairo/patches/patch-ab
+++ b/graphics/cairo/patches/patch-ab
@@ -1,13 +1,13 @@
-$NetBSD: patch-ab,v 1.8 2007/07/06 18:19:43 drochner Exp $
+$NetBSD: patch-ab,v 1.9 2008/04/21 11:29:33 tnn Exp $
 
---- test/Makefile.in.orig	2007-06-27 23:06:21.000000000 +0200
+--- test/Makefile.in.orig	2008-04-11 23:33:42.000000000 +0200
 +++ test/Makefile.in
-@@ -54,7 +54,7 @@ check_PROGRAMS = imagediff$(EXEEXT) png-
- 	$(am__EXEEXT_13) $(am__EXEEXT_14)
- EXTRA_PROGRAMS = $(am__EXEEXT_1) $(am__EXEEXT_2) $(am__EXEEXT_11) \
- 	$(am__EXEEXT_12)
--@HAVE_PTHREAD_TRUE@am__append_9 = -lpthread
-+@HAVE_PTHREAD_TRUE@am__append_9 = ${PTHREAD_LDFLAGS} ${PTHREAD_LIBS}
- @CAIRO_CAN_TEST_PDF_SURFACE_TRUE@am__append_10 = pdf2png
- @CAIRO_CAN_TEST_PDF_SURFACE_TRUE@@CAIRO_HAS_SVG_SURFACE_TRUE@am__append_11 = pdf2svg$(EXEEXT)
- @CAIRO_CAN_TEST_SVG_SURFACE_TRUE@am__append_12 = svg2png
+@@ -59,7 +59,7 @@ host_triplet = @host@
+ @CAIRO_HAS_PDF_SURFACE_TRUE@@CAIRO_HAS_PS_SURFACE_TRUE@@CAIRO_HAS_SVG_SURFACE_TRUE@am__append_10 = fallback-resolution$(EXEEXT)
+ check_PROGRAMS = imagediff$(EXEEXT) png-flatten$(EXEEXT) \
+ 	$(am__EXEEXT_1) $(am__EXEEXT_2)
+-@HAVE_PTHREAD_TRUE@am__append_11 = -lpthread
++@HAVE_PTHREAD_TRUE@am__append_11 = ${PTHREAD_LDFLAGS} ${PTHREAD_LIBS}
+ @CAIRO_CAN_TEST_PDF_SURFACE_TRUE@am__append_12 = pdf2png
+ @CAIRO_CAN_TEST_SVG_SURFACE_TRUE@am__append_13 = svg2png
+ subdir = test
diff --git a/graphics/cairo/patches/patch-ae b/graphics/cairo/patches/patch-ae
deleted file mode 100644
index 22771265ff8..00000000000
--- a/graphics/cairo/patches/patch-ae
+++ /dev/null
@@ -1,335 +0,0 @@
-$NetBSD: patch-ae,v 1.9 2007/11/30 20:49:25 drochner Exp $
-
-Fixes cairo on 8-bit pseudo color and other 8-bit displays.
-See https://bugs.freedesktop.org/show_bug.cgi?id=4945
-
---- src/cairo-xlib-surface-private.h.orig	2007-11-27 07:20:12.000000000 +0100
-+++ src/cairo-xlib-surface-private.h
-@@ -39,6 +39,14 @@
- 
- typedef struct _cairo_xlib_surface cairo_xlib_surface_t;
- 
-+struct clut_r3g3b2 {
-+    struct clut_r3g3b2 *next;
-+    Display            *dpy;
-+    Colormap           cmap;
-+    uint32_t           clut[256];
-+    unsigned char      ilut[256];
-+};
-+
- struct _cairo_xlib_surface {
-     cairo_surface_t base;
- 
-@@ -89,6 +97,9 @@ struct _cairo_xlib_surface {
-     cairo_filter_t filter;
-     int repeat;
-     XTransform xtransform;
-+
-+    struct clut_r3g3b2 *clut;
-+    int workaround;
- };
- 
- enum {
---- src/cairo-xlib-surface.c.orig	2007-11-27 07:20:12.000000000 +0100
-+++ src/cairo-xlib-surface.c
-@@ -108,6 +108,10 @@ static const XTransform identity = { {
- #define CAIRO_SURFACE_RENDER_HAS_PICTURE_TRANSFORM(surface)	CAIRO_SURFACE_RENDER_AT_LEAST((surface), 0, 6)
- #define CAIRO_SURFACE_RENDER_HAS_FILTERS(surface)	CAIRO_SURFACE_RENDER_AT_LEAST((surface), 0, 6)
- 
-+#define WORKAROUND_NONE 0
-+#define WORKAROUND_8BIT_PALETTE 1
-+#define WORKAROUND_8BIT_DIRECT 2
-+
- static int
- _CAIRO_FORMAT_DEPTH (cairo_format_t format)
- {
-@@ -494,6 +498,74 @@ _swap_ximage_to_native (XImage *ximage)
-     }
- }
- 
-+static struct clut_r3g3b2 * _get_clut_r3g3b2(Display *dpy, Colormap cmap) {
-+    static struct clut_r3g3b2 *first = NULL;
-+    int i,j, min, d;
-+    struct clut_r3g3b2 *clut;
-+    unsigned char r,g,b, r2,g2,b2;
-+    
-+    clut = first;
-+    while(clut) {
-+	if ( clut->dpy == dpy && clut->cmap == cmap )
-+	    return clut;
-+	clut = clut->next;
-+    }
-+    
-+    clut = calloc(1, sizeof(*clut));
-+    if(clut == NULL)
-+	return NULL;
-+    
-+    clut->next = first;
-+    clut->dpy = dpy;
-+    clut->cmap = cmap;
-+    first = clut;
-+
-+    /* Construct the clut from Colormap */
-+    for (i = 0; i < 256; i++) {
-+	XColor xcol;
-+	xcol.pixel = i;
-+	XQueryColor(dpy, cmap, &xcol);
-+	clut->clut[i] = ( ( ((uint32_t)xcol.red   & 0xff00 ) << 8) |
-+			  ( ((uint32_t)xcol.green & 0xff00 ) ) |
-+			  ( ((uint32_t)xcol.blue  & 0xff00 ) >> 8) );
-+    }
-+    /*
-+      
-+    Find the best matching color in the colormap for all r3g3b2
-+    values. The distance is maybe not perceptively valid, but it
-+    should not be too bad.
-+    
-+    */
-+    for (i = 0; i < 256; i++) {
-+	r = i >> 5;
-+	g = (i >> 2) & 0x7;
-+	b = (i << 1) & 0x7;
-+	min = 255;
-+	for(j = 0; j < 256; j++) {
-+	    r2 = (clut->clut[j] & 0xff0000) >> 21;
-+	    g2 = (clut->clut[j] & 0x00ff00) >> 13;
-+	    b2 = (clut->clut[j] & 0x0000ff) >> 5;
-+	    if ( r2 == r && g2 == g && (b2 & 0x6) == b ) {
-+		clut->ilut[i] = j;
-+		break;
-+	    }
-+	    /*
-+	      Squares make higher bits much more important than lower
-+	      ones.
-+	    */
-+	    d  = (r2 ^ r) * (r2 ^ r);
-+	    d += (g2 ^ g) * (g2 ^ g);
-+	    d += (b2 ^ b) * (b2 ^ b);
-+	    if(d < min) {
-+		clut->ilut[i] = j;
-+		min = d;
-+	    }
-+	}
-+    }
-+    
-+    return clut;
-+}
-+
- static cairo_status_t
- _get_image_surface (cairo_xlib_surface_t    *surface,
- 		    cairo_rectangle_int16_t *interest_rect,
-@@ -655,18 +727,77 @@ _get_image_surface (cairo_xlib_surface_t
-     }
-     else
-     {
-+        if ((surface->clut != NULL) && (surface->workaround == WORKAROUND_8BIT_PALETTE)) {
-+	    
-+      	    /*
-+	     * Otherwise, we construct a buffer containing RGB24 data
-+	     * using the specified workaround.
-+	     */
-+	    uint32_t *data, *dst, *clut;
-+	    uint8_t  *src8;
-+	    int i,j;
-+
-+	    data = (uint32_t*)malloc(ximage->height * ximage->width * 4);
-+	    if (data == NULL) {
-+		_cairo_error(CAIRO_STATUS_NO_MEMORY);
-+		goto FAIL;
-+	    }
-+
-+	    clut = surface->clut->clut;
-+	    src8 = (uint8_t*) ximage->data;
-+	    dst = data;
-+	    for (j = 0; j < ximage->height; j++) {
-+		for (i = 0; i < ximage->width; i++)
-+		    *dst++ = clut[src8[i]];
-+		src8 += ximage->bytes_per_line;
-+	    }
-+	    free(ximage->data);
-+	    image = (cairo_image_surface_t*)
-+		cairo_image_surface_create_for_data ((unsigned char *)data,
-+                   CAIRO_FORMAT_RGB24, ximage->width, ximage->height,
-+                   ximage->width*4);
-+        } else if (surface->workaround == WORKAROUND_8BIT_DIRECT) {
-+
-+	    uint32_t *data, *dst;
-+	    uint8_t  *src8;
-+	    int i,j;
-+
-+	    data = (uint32_t*)malloc(ximage->height * ximage->width * 4);
-+	    if (data == NULL) {
-+		_cairo_error(CAIRO_STATUS_NO_MEMORY);
-+		goto FAIL;
-+	    }
-+
-+	    src8 = (uint8_t*) ximage->data;
-+	    dst = data;
-+	    for (j = 0; j < ximage->height; j++) {
-+		for (i = 0; i < ximage->width; i++)
-+	      		    *dst++ = (src8[i] & masks.red_mask << 21) | 
-+			             (src8[i] & masks.green_mask << 10 ) | 
-+			             (src8[i] & masks.blue_mask );
-+		 
-+		src8 += ximage->bytes_per_line;
-+	    }
-+	    free(ximage->data);
-+	    image = (cairo_image_surface_t*)
-+		cairo_image_surface_create_for_data ((unsigned char *)data,
-+                   CAIRO_FORMAT_RGB24, ximage->width, ximage->height,
-+                   ximage->width*4);
-+	}else if (surface->workaround == WORKAROUND_NONE){
- 	/*
- 	 * XXX This can't work.  We must convert the data to one of the
- 	 * supported pixman formats.  Pixman needs another function
- 	 * which takes data in an arbitrary format and converts it
- 	 * to something supported by that library.
- 	 */
--	image = (cairo_image_surface_t*)
--	    _cairo_image_surface_create_with_masks ((unsigned char *) ximage->data,
-+            image = (cairo_image_surface_t*)
-+	      _cairo_image_surface_create_with_masks ((unsigned char *) ximage->data,
- 						    &masks,
- 						    ximage->width,
- 						    ximage->height,
- 						    ximage->bytes_per_line);
-+	}
-+	
- 	if (image->base.status)
- 	    goto FAIL;
-     }
-@@ -770,6 +901,31 @@ _cairo_xlib_surface_ensure_gc (cairo_xli
-     return CAIRO_STATUS_SUCCESS;
- }
- 
-+static int
-+_make_space_for(unsigned char ** buf, int *size, int *stride, int width, int height, int Bpp)
-+{
-+    unsigned char * data;
-+    int l;
-+
-+    *stride = width * Bpp;
-+    if(*stride%4)
-+      *stride += 4 - *stride % 4;
-+    l = (*stride * height);
-+    if (*size < l) {
-+      if(*buf)
-+          data = realloc(*buf, l);
-+      else
-+          data = malloc(l);
-+      if(data) {
-+          *buf = data;
-+          *size = l;
-+      } else {
-+          return -1;
-+      }
-+    }
-+    return 0;
-+}
-+
- static cairo_status_t
- _draw_image_surface (cairo_xlib_surface_t   *surface,
- 		     cairo_image_surface_t  *image,
-@@ -782,22 +938,78 @@ _draw_image_surface (cairo_xlib_surface_
- {
-     XImage ximage;
-     unsigned int bpp, alpha, red, green, blue;
-+    unsigned int depth = image->depth;
-+    unsigned int stride = image->stride;
-     int native_byte_order = _native_byte_order_lsb () ? LSBFirst : MSBFirst;
-     cairo_status_t status;
- 
-     pixman_format_get_masks (pixman_image_get_format (image->pixman_image),
- 			     &bpp, &alpha, &red, &green, &blue);
- 
-+    if ((surface->clut != NULL) && (surface->workaround == WORKAROUND_8BIT_PALETTE)) {
-+      static unsigned char *buf = NULL;
-+      static int size = 0;
-+      int i, j;
-+      unsigned char *data, *ilut;
-+      uint32_t *src;
-+      uint8_t *dst8;
-+
-+      if (_make_space_for(&buf, &size, &stride, image->width, image->height, 1))
-+            return CAIRO_STATUS_NO_MEMORY;
-+      data = buf;
-+      src = (uint32_t*)image->data;
-+      ilut = surface->clut->ilut;
-+      for (j=0;j<image->height;j++) {
-+            dst8 = data + j * stride;
-+            for (i=0;i<image->width;i++) {
-+              dst8[i] = ilut[ ((*src >> 16) & 0xe0) |
-+                              ((*src >> 11) & 0x1c) |
-+                              ((*src >> 6)  & 0x03) ];
-+              src++;
-+            }
-+      }
-+      alpha = red = green = blue = 0;
-+      depth = bpp = 8;
-+              ximage.data = data;
-+
-+    }else if (surface->workaround == WORKAROUND_8BIT_DIRECT){
-+      static unsigned char *buf = NULL;
-+      static int size = 0;
-+      int i, j;
-+      unsigned char *data;
-+      uint32_t *src;
-+      uint8_t *dst8;
-+
-+      if (_make_space_for(&buf, &size, &stride, image->width, image->height, 1))
-+            return CAIRO_STATUS_NO_MEMORY;
-+      data = buf;
-+      src = (uint32_t*)image->data;
-+      for (j=0;j<image->height;j++) {
-+            dst8 = data + j * stride;
-+            for (i=0;i<image->width;i++) {
-+              dst8[i] = ((*src >> 16) & 0xe0) |
-+                        ((*src >> 11) & 0x1c) |
-+                        ((*src >> 6)  & 0x03) ;
-+              src++;
-+            }
-+      }
-+      alpha = red = green = blue = 0;
-+      depth = bpp = 8;
-+              ximage.data = data;
-+    } else if( surface->workaround == WORKAROUND_NONE){
-+      ximage.data = (char *)image->data;
-+    }
-+
-     ximage.width = image->width;
-     ximage.height = image->height;
-     ximage.format = ZPixmap;
--    ximage.data = (char *)image->data;
-+    // ximage.data is assigned above
-     ximage.byte_order = native_byte_order;
-     ximage.bitmap_unit = 32;	/* always for libpixman */
-     ximage.bitmap_bit_order = native_byte_order;
-     ximage.bitmap_pad = 32;	/* always for libpixman */
--    ximage.depth = image->depth;
--    ximage.bytes_per_line = image->stride;
-+    ximage.depth = depth;
-+    ximage.bytes_per_line = stride;
-     ximage.bits_per_pixel = bpp;
-     ximage.red_mask = red;
-     ximage.green_mask = green;
-@@ -2043,6 +2255,20 @@ _cairo_xlib_surface_create_internal (Dis
-     surface->have_clip_rects = FALSE;
-     surface->clip_rects = surface->embedded_clip_rects;
-     surface->num_clip_rects = 0;
-+    surface->clut = NULL;
-+
-+    if (xrender_format == NULL &&
-+       (visual==NULL?FALSE:(visual->class == PseudoColor || visual->class == StaticColor))) {
-+        surface->clut = _get_clut_r3g3b2(dpy,
-+            DefaultColormapOfScreen(surface->screen));
-+    }else if (xrender_format == NULL &&
-+	      (visual==NULL?FALSE:((visual->class == TrueColor) 
-+				   && (surface->visual->red_mask == 0x07)
-+				   && (surface->visual->green_mask == 0x38)
-+				   && (surface->visual->blue_mask == 0xc0))))
-+        surface->workaround = WORKAROUND_8BIT_DIRECT;
-+    else 
-+        surface->workaround = WORKAROUND_NONE;
- 
-     return (cairo_surface_t *) surface;
- }