Attachment 38454 Details for Bug 29092 – Remove hidden qt_draw_glyphs usage until qt provide public version

[patch] Remove hidden qt_draw_glyphs usage until qt provide public version

29092 (text/plain), 2.90 KB, created by Oleg Romashin on 2010-09-05 08:48:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Oleg Romashin

Created: 2010-09-05 08:48:29 UTC

Size: 2.90 KB

patch

obsolete

># HG changeset patch
># Parent a4d14215a66c36dcb2915c8887a015bf107a5a65
>diff --git a/src/cairo-qt-surface.cpp b/src/cairo-qt-surface.cpp
>--- a/src/cairo-qt-surface.cpp
>+++ b/src/cairo-qt-surface.cpp
>@@ -56,20 +56,16 @@
> #include <QtGui/QImage>
> #include <QtGui/QPixmap>
> #include <QtGui/QBrush>
> #include <QtGui/QPen>
> #include <QtGui/QWidget>
> #include <QtGui/QX11Info>
> #include <QtCore/QVarLengthArray>
> 
>-#if (QT_VERSION >= QT_VERSION_CHECK(4, 7, 0)) || defined(QT_GLYPHS_API_BACKPORT)
>-extern void qt_draw_glyphs(QPainter *, const quint32 *glyphs, const QPointF *positions, int count);
>-#endif
>-
> #include <sys/time.h>
> 
> /* Enable workaround slow regional Qt paths */
> #define ENABLE_FAST_FILL 0
> #define ENABLE_FAST_CLIP 0
> 
> #if 0
> #define D(x)  x
>@@ -1365,50 +1361,17 @@ _cairo_qt_surface_show_glyphs (void *abs
> 			       cairo_operator_t op,
> 			       const cairo_pattern_t *source,
> 			       cairo_glyph_t *glyphs,
> 			       int num_glyphs,
> 			       cairo_scaled_font_t *scaled_font,
> 			       cairo_clip_t *clip,
> 			       int *remaining_glyphs)
> {
>-#if (QT_VERSION >= QT_VERSION_CHECK(4, 7, 0)) || defined(QT_GLYPHS_API_BACKPORT)
>-    cairo_qt_surface_t *qs = (cairo_qt_surface_t *) abstract_surface;
>-
>-    // pick out the colour to use from the cairo source
>-    cairo_solid_pattern_t *solid = (cairo_solid_pattern_t*) source;
>-    cairo_scaled_glyph_t* glyph;
>-    // documentation says you have to freeze the cache, but I don't believe it
>-    _cairo_scaled_font_freeze_cache(scaled_font);
>-
>-    QColor tempColour(solid->color.red * 255, solid->color.green * 255, solid->color.blue * 255);
>-    QVarLengthArray<QPointF> positions(num_glyphs);
>-    QVarLengthArray<unsigned int> glyphss(num_glyphs);
>-    FT_Face face = cairo_ft_scaled_font_lock_face (scaled_font);
>-    const FT_Size_Metrics& ftMetrics = face->size->metrics;
>-    QFont font(face->family_name);
>-    font.setStyleStrategy(QFont::NoFontMerging);
>-    font.setBold(face->style_flags & FT_STYLE_FLAG_BOLD);
>-    font.setItalic(face->style_flags & FT_STYLE_FLAG_ITALIC);
>-    font.setKerning(face->face_flags & FT_FACE_FLAG_KERNING);
>-    font.setPixelSize(ftMetrics.y_ppem);
>-    cairo_ft_scaled_font_unlock_face(scaled_font);
>-    qs->p->setFont(font);
>-    qs->p->setPen(tempColour);
>-    for (int currentGlyph = 0; currentGlyph < num_glyphs; currentGlyph++) {
>-        positions[currentGlyph].setX(glyphs[currentGlyph].x);
>-        positions[currentGlyph].setY(glyphs[currentGlyph].y);
>-        glyphss[currentGlyph] = glyphs[currentGlyph].index;
>-    }
>-    qt_draw_glyphs(qs->p, glyphss.data(), positions.data(), num_glyphs);
>-    _cairo_scaled_font_thaw_cache(scaled_font);
>-    return CAIRO_INT_STATUS_SUCCESS;
>-#else
>     return CAIRO_INT_STATUS_UNSUPPORTED;
>-#endif
> }
> 
> static cairo_int_status_t
> _cairo_qt_surface_mask (void *abstract_surface,
> 			cairo_operator_t op,
> 			const cairo_pattern_t *source,
> 			const cairo_pattern_t *mask,
> 			cairo_clip_t	    *clip)

Actions: View | Splinter Review

Attachments on bug 29092: 37099 | 38454 | 50411 | 54751

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.