[cairo-commit] test/ft-show-glyphs-table.c test/ft-show-glyphs-table-ref.png test/.gitignore test/Makefile.am
Carl Worth
cworth at kemper.freedesktop.org
Thu Apr 3 17:12:31 PDT 2008
test/.gitignore | 1
test/Makefile.am | 2
test/ft-show-glyphs-table-ref.png |binary
test/ft-show-glyphs-table.c | 116 ++++++++++++++++++++++++++++++++++++++
4 files changed, 119 insertions(+)
New commits:
commit 8a78760f15c29c72e6a945b2157fd214e8045e1e
Author: Carl Worth <cworth at cworth.org>
Date: Thu Apr 3 17:12:14 2008 -0700
Add new ft-show-glyphs-table test to exercise recent glyph positioning bug fix
Interestingly, this test case does demonstrate that cairo-pdf
is fixed, (where without commit f6509933a4e0 the Y positions
of the glyphs were inverted); however, cairo-ps is failing
with this test, (all the glyphs are ending up on top of each
other).
diff --git a/test/.gitignore b/test/.gitignore
index e153c7a..268afca 100644
--- a/test/.gitignore
+++ b/test/.gitignore
@@ -67,6 +67,7 @@ finer-grained-fallbacks
ft-text-antialias-none
ft-font-create-for-ft-face
ft-show-glyphs-positioning
+ft-show-glyphs-table
ft-text-vertical-layout-type1
ft-text-vertical-layout-type3
font-face-get-type
diff --git a/test/Makefile.am b/test/Makefile.am
index ef1d757..e2987fa 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -200,6 +200,7 @@ if CAIRO_HAS_FT_FONT
TESTS += bitmap-font$(EXEEXT)
TESTS += ft-font-create-for-ft-face$(EXEEXT)
TESTS += ft-show-glyphs-positioning$(EXEEXT)
+TESTS += ft-show-glyphs-table$(EXEEXT)
TESTS += ft-text-vertical-layout-type1$(EXEEXT)
TESTS += ft-text-vertical-layout-type3$(EXEEXT)
TESTS += ft-text-antialias-none$(EXEEXT)
@@ -389,6 +390,7 @@ REFERENCE_IMAGES = \
ft-show-glyphs-positioning-pdf-ref.png \
ft-show-glyphs-positioning-ps-ref.png \
ft-show-glyphs-positioning-svg-ref.png \
+ ft-show-glyphs-table-ref.png \
ft-text-vertical-layout-type1-ps-ref.png \
ft-text-vertical-layout-type1-ref.png \
ft-text-vertical-layout-type1-svg-ref.png \
diff --git a/test/ft-show-glyphs-table-ref.png b/test/ft-show-glyphs-table-ref.png
new file mode 100644
index 0000000..92a3527
Binary files /dev/null and b/test/ft-show-glyphs-table-ref.png differ
diff --git a/test/ft-show-glyphs-table.c b/test/ft-show-glyphs-table.c
new file mode 100644
index 0000000..a7654bc
--- /dev/null
+++ b/test/ft-show-glyphs-table.c
@@ -0,0 +1,116 @@
+/*
+ * Copyright © 2008 Red Hat, Inc.
+ *
+ * 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.
+ *
+ * Authors: Eugeniy Meshcheryakov <eugen at debian.org>
+ * Adrian Johnson <ajohnson at redneon.com>
+ * Carl Worth <cworth at cworth.org>
+ */
+
+#include "cairo-test.h"
+#include <cairo-ft.h>
+
+#define TEXT_SIZE 20
+#define PAD 10
+#define GRID_SIZE 30
+#define GRID_ROWS 10
+#define GRID_COLS 4
+#define NUM_GLYPHS (GRID_ROWS * GRID_COLS)
+#define WIDTH (PAD + GRID_COLS * GRID_SIZE + PAD)
+#define HEIGHT (PAD + GRID_ROWS * GRID_SIZE + PAD)
+
+/* This test was originally inspired by this bug report:
+ *
+ * Error when creating pdf charts for new FreeSerifItalic.ttf
+ * http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=%23474136
+ *
+ * The original assertion failure was fairly boring, but the later
+ * glyph mispositiing was quite interesting. And it turns out that the
+ * _cairo_pdf_operators_show_glyphs code is fairly convoluted with a
+ * code path that wasn't being exercised at all by the test suite.
+ *
+ * So this is an attempt to exercise that code path. Apparently laying
+ * glyphs out vertically in a table like this, (so that there's a
+ * large change in Y position from one glyph to the next), exercises
+ * the code well.
+ */
+
+static cairo_test_draw_function_t draw;
+
+cairo_test_t test = {
+ "ft-show-glyphs-table",
+ "Test cairo_show_glyphs with cairo-ft backend and glyphs laid out in a table",
+ WIDTH, HEIGHT,
+ draw
+};
+
+static cairo_test_status_t
+draw (cairo_t *cr, int width, int height)
+{
+ cairo_font_options_t *font_options;
+ cairo_scaled_font_t *scaled_font;
+ FT_Face face;
+ FT_ULong charcode;
+ FT_UInt idx;
+ int i = 0;
+ cairo_glyph_t glyphs[NUM_GLYPHS];
+
+ /* paint white so we don't need separate ref images for
+ * RGB24 and ARGB32 */
+ cairo_set_source_rgb (cr, 1.0, 1.0, 1.0);
+ cairo_paint (cr);
+
+ cairo_select_font_face (cr, "Bitstream Vera Sans",
+ CAIRO_FONT_SLANT_NORMAL,
+ CAIRO_FONT_WEIGHT_NORMAL);
+ cairo_set_font_size (cr, TEXT_SIZE);
+
+ font_options = cairo_font_options_create ();
+ cairo_get_font_options (cr, font_options);
+ cairo_font_options_set_hint_metrics (font_options, CAIRO_HINT_METRICS_OFF);
+ cairo_set_font_options (cr, font_options);
+ cairo_font_options_destroy (font_options);
+
+ cairo_set_source_rgb (cr, 0.0, 0.0, 0.0);
+
+ scaled_font = cairo_get_scaled_font (cr);
+ face = cairo_ft_scaled_font_lock_face (scaled_font);
+ {
+ charcode = FT_Get_First_Char(face, &idx);
+ while (idx && (i < NUM_GLYPHS)) {
+ glyphs[i] = (cairo_glyph_t) {idx, PAD + GRID_SIZE * (i/GRID_ROWS), PAD + TEXT_SIZE + GRID_SIZE * (i%GRID_ROWS)};
+ i++;
+ charcode = FT_Get_Next_Char(face, charcode, &idx);
+ }
+ }
+ cairo_ft_scaled_font_unlock_face (scaled_font);
+
+ cairo_show_glyphs(cr, glyphs, i);
+
+ return CAIRO_TEST_SUCCESS;
+}
+
+int
+main (void)
+{
+ return cairo_test (&test);
+}
More information about the cairo-commit
mailing list