[cairo-commit] 4 commits - src/cairo-image-surface.c test/a8-clear.c test/a8-clear.ref.png test/Makefile.am test/Makefile.sources

Chris Wilson ickle at kemper.freedesktop.org
Wed Sep 29 05:58:06 PDT 2010


 src/cairo-image-surface.c |   11 +++++--
 test/Makefile.am          |    1 
 test/Makefile.sources     |    1 
 test/a8-clear.c           |   64 ++++++++++++++++++++++++++++++++++++++++++++++
 test/a8-clear.ref.png     |binary
 5 files changed, 74 insertions(+), 3 deletions(-)

New commits:
commit 877bc14eaf188ddf5ca747a0b7c808cccd2d1c5f
Merge: 7b308fd... 66ace8c...
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Wed Sep 29 13:57:36 2010 +0100

    Merge branch '1.10'

commit 66ace8cd33c0ebe88972f0c4d5c0effeacb1cea3
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Wed Sep 29 13:46:02 2010 +0100

    image: The a8 in-place span compositing is only valid for SOURCE
    
    Fixes test/a8-clear.
    
    Reported-by: Jeremy Moles <jeremy at emperorlinux.com>
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/src/cairo-image-surface.c b/src/cairo-image-surface.c
index 2a1c818..491859f 100644
--- a/src/cairo-image-surface.c
+++ b/src/cairo-image-surface.c
@@ -3447,10 +3447,15 @@ _composite_spans (void                          *closure,
     /* TODO: support rendering to A1 surfaces (or: go add span
      * compositing to pixman.) */
 
-    if (pattern == NULL && dst_format == PIXMAN_a8) {
+    if (pattern == NULL &&
+	dst_format == PIXMAN_a8 &&
+	op == CAIRO_OPERATOR_SOURCE)
+    {
 	mask = dst;
 	dst = NULL;
-    } else {
+    }
+    else
+    {
 	int stride = CAIRO_STRIDE_FOR_WIDTH_BPP (extents->width, 8);
 	uint8_t *data = mask_buf;
 
commit 05faebd5845c68005f5b96428952faae5a67208f
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Wed Sep 29 13:23:24 2010 +0100

    test: Add a8-clear
    
    Reported-by: Jeremy Moles <jeremy at emperorlinux.com>
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/test/Makefile.am b/test/Makefile.am
index 53b03c7..a3c1f7a 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -138,6 +138,7 @@ REFERENCE_IMAGES = \
 	a1-rasterisation-triangles.ref.png \
 	a1-traps-sample.ref.png \
 	a1-traps-sample.quartz.xfail.png \
+	a8-clear.ref.png \
 	a8-mask.ref.png \
 	aliasing.ref.png \
 	aliasing.image16.ref.png \
diff --git a/test/Makefile.sources b/test/Makefile.sources
index e497cd7..a13aea8 100644
--- a/test/Makefile.sources
+++ b/test/Makefile.sources
@@ -5,6 +5,7 @@ test_sources = \
 	a1-mask-sample.c 				\
 	a1-traps-sample.c				\
 	a1-rasterisation.c				\
+	a8-clear.c					\
 	a8-mask.c					\
 	aliasing.c					\
 	alpha-similar.c					\
diff --git a/test/a8-clear.c b/test/a8-clear.c
new file mode 100644
index 0000000..1459f83
--- /dev/null
+++ b/test/a8-clear.c
@@ -0,0 +1,64 @@
+/*
+ * Copyright © 2010 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person
+ * obtaining a copy of this software and associated documentation
+ * files (the "Software"), to deal in the Software without
+ * restriction, including without limitation the rights to use, copy,
+ * modify, merge, publish, distribute, sublicense, and/or sell copies
+ * of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be
+ * included in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+ * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+ * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+ * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ *
+ * Author: Chris Wilson <chris at chris-wilson.co.uk>
+ *
+ * Based on a bug snippet by Jeremy Moles <jeremy at emperorlinux.com>
+ */
+
+#include "cairo-test.h"
+
+static cairo_test_status_t
+draw (cairo_t *cr, int width, int height)
+{
+    cairo_pattern_t *mask;
+
+    cairo_set_source_rgb (cr, 1, 0, 0);
+    cairo_paint (cr);
+
+    cairo_push_group_with_content (cr, CAIRO_CONTENT_ALPHA);
+    {
+	cairo_set_source_rgb (cr, 1, 1, 1);
+	cairo_paint (cr);
+
+	cairo_move_to (cr, 0, 0);
+	cairo_line_to (cr, width, height);
+	cairo_set_operator (cr, CAIRO_OPERATOR_CLEAR);
+	cairo_set_line_width (cr, 10);
+	cairo_stroke (cr);
+    }
+    mask = cairo_pop_group (cr);
+    cairo_set_source_rgb (cr, 1, 1, 1);
+    cairo_mask (cr, mask);
+    cairo_pattern_destroy (mask);
+
+    return CAIRO_TEST_SUCCESS;
+}
+
+CAIRO_TEST (a8_clear,
+	    "Test clear on an a8 surface",
+	    "a8, clear", /* keywords */
+	    NULL, /* requirements */
+	    40, 40,
+	    NULL, draw)
+
diff --git a/test/a8-clear.ref.png b/test/a8-clear.ref.png
new file mode 100644
index 0000000..98a624e
Binary files /dev/null and b/test/a8-clear.ref.png differ
commit ac6fd51fe4e500973e0939983d00aa81c8acf556
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Wed Sep 29 13:25:08 2010 +0100

    image: Silence a compile warning
    
    cairo-image-surface.c: In function ‘_cairo_image_reset_static_data’:
    cairo-image-surface.c:1006: warning: old-style function definition
    
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/src/cairo-image-surface.c b/src/cairo-image-surface.c
index 4d65ee6..2a1c818 100644
--- a/src/cairo-image-surface.c
+++ b/src/cairo-image-surface.c
@@ -1003,7 +1003,7 @@ static struct {
 static int n_cached;
 
 void
-_cairo_image_reset_static_data ()
+_cairo_image_reset_static_data (void)
 {
     while (n_cached)
 	pixman_image_unref (cache[--n_cached].image);


More information about the cairo-commit mailing list