[cairo-commit] test/bug-40410.c test/bug-40410.ref.png test/Makefile.refs test/Makefile.sources

[Chris Wilson]

 test/Makefile.refs     |    1 
 test/Makefile.sources  |    1 
 test/bug-40410.c       |   72 +++++++++++++++++++++++++++++++++++++++++++++++++
 test/bug-40410.ref.png |binary
 4 files changed, 74 insertions(+)

New commits:
commit 8d91de49261662bc920739bbb91d22a49016eb81
Author: Krzysztof Kosiński <tweenk.pl at gmail.com>
Date:   Fri Aug 26 22:59:01 2011 +0100

    test: Add test case from bug-40410
    
    Exercises a case where we failed to fill a rectangle...

diff --git a/test/Makefile.refs b/test/Makefile.refs
index 1dd8835..0db6208 100644
--- a/test/Makefile.refs
+++ b/test/Makefile.refs
@@ -54,6 +54,7 @@ REFERENCE_IMAGES = \
 	bilevel-image.ref.png \
 	bitmap-font.ref.png \
 	bitmap-font.rgb24.ref.png \
+	bug-40410.ref.png \
 	bug-bo-rectangular.image16.ref.png \
 	bug-bo-rectangular.ps.xfail.png \
 	bug-bo-rectangular.ref.png \
diff --git a/test/Makefile.sources b/test/Makefile.sources
index 2c0f056..7289615 100644
--- a/test/Makefile.sources
+++ b/test/Makefile.sources
@@ -18,6 +18,7 @@ test_sources = \
 	big-little-box.c				\
 	big-trap.c					\
 	bilevel-image.c					\
+	bug-40410.c					\
 	bug-bo-rectangular.c				\
 	bug-extents.c					\
 	bug-seams.c					\
diff --git a/test/bug-40410.c b/test/bug-40410.c
new file mode 100644
index 0000000..9235cf9
--- /dev/null
+++ b/test/bug-40410.c
@@ -0,0 +1,72 @@
+/*
+ * Copyright © 2011 Krzysztof Kosiński
+ *
+ * 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: Krzysztof Kosiński <tweenk.pl at gmail.com>
+ */
+
+#include "cairo-test.h"
+
+#define WIDTH	300
+#define HEIGHT	100
+
+/*
+ * The bug appears to be triggered if:
+ * 1. There is more than one subpath
+ * 2. All subpaths are axis-aligned rectangles
+ * 3. Only one of the subpaths is within surface bounds
+ *
+ * Tweaking any of the coordinates so that there is at least one
+ * non-axis-aligned segment or removing the second subpath (the one that is
+ * outside the surface bounds) causes the bug to disappear.
+ */
+
+static cairo_test_status_t
+draw (cairo_t *cr, int width, int height)
+{
+    cairo_set_source_rgb (cr, 1,1,1);
+    cairo_paint (cr);
+
+    cairo_move_to (cr, 10.3, 10.6);
+    cairo_line_to (cr, 10.3, 150.2);
+    cairo_line_to (cr, 290.1, 150.2);
+    cairo_line_to (cr, 290.1, 10.6);
+    cairo_close_path (cr);
+
+    cairo_move_to (cr, 10.3, 180.7);
+    cairo_line_to (cr, 10.3, 230.2);
+    cairo_line_to (cr, 290.1, 230.2);
+    cairo_line_to (cr, 290.1, 180.7);
+    cairo_close_path (cr);
+
+    cairo_set_source_rgb (cr, 1, 0, 0);
+    cairo_fill (cr);
+
+    return CAIRO_TEST_SUCCESS;
+}
+
+CAIRO_TEST (bug_40410,
+	    "Exercises a bug found in 1.10.2 (and never again!)",
+	    "fill", /* keywords */
+	    NULL, /* requirements */
+	    WIDTH, HEIGHT,
+	    NULL, draw)
diff --git a/test/bug-40410.ref.png b/test/bug-40410.ref.png
new file mode 100644
index 0000000..a315935
Binary files /dev/null and b/test/bug-40410.ref.png differ