diff options
| author | Jesus Fernandez <jesus.fernandez@qt.io> | 2018-08-03 14:09:49 +0200 |
|---|---|---|
| committer | Jesus Fernandez <Jesus.Fernandez@qt.io> | 2018-08-06 09:17:59 +0000 |
| commit | d9c3dbc93a466936a1385a5b15656bd14c67d98e (patch) | |
| tree | a9edafd70426e6b795d7dde9588cfa3d214a11fc /src/plugins/platforms/xcb/nativepainting | |
| parent | eacd78950282bb835e9b8b4e955b4f93f3c5167f (diff) | |
Pass parameters by const-reference
Coverity-Id: 178810
Change-Id: I996955543c32907a54bf095bf52f242dbb7d5400
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
Diffstat (limited to 'src/plugins/platforms/xcb/nativepainting')
| -rw-r--r-- | src/plugins/platforms/xcb/nativepainting/qpaintengine_x11.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/plugins/platforms/xcb/nativepainting/qpaintengine_x11.cpp b/src/plugins/platforms/xcb/nativepainting/qpaintengine_x11.cpp index 8b63e5431d..a3e6cedecd 100644 --- a/src/plugins/platforms/xcb/nativepainting/qpaintengine_x11.cpp +++ b/src/plugins/platforms/xcb/nativepainting/qpaintengine_x11.cpp @@ -226,7 +226,9 @@ public: QXRenderGlyphCache(QXcbX11Info x, QFontEngine::GlyphFormat format, const QTransform &matrix); ~QXRenderGlyphCache(); - bool addGlyphs(const QTextItemInt &ti, QVarLengthArray<glyph_t> glyphs, QVarLengthArray<QFixedPoint> positions); + bool addGlyphs(const QTextItemInt &ti, + const QVarLengthArray<glyph_t> &glyphs, + const QVarLengthArray<QFixedPoint> &positions); bool draw(Drawable src, Drawable dst, const QTransform &matrix, const QTextItemInt &ti); inline GlyphSet glyphSet(); @@ -2608,7 +2610,9 @@ QXRenderGlyphCache::~QXRenderGlyphCache() XRenderFreeGlyphSet(xinfo.display(), gset); } -bool QXRenderGlyphCache::addGlyphs(const QTextItemInt &ti, QVarLengthArray<glyph_t> glyphs, QVarLengthArray<QFixedPoint> positions) +bool QXRenderGlyphCache::addGlyphs(const QTextItemInt &ti, + const QVarLengthArray<glyph_t> &glyphs, + const QVarLengthArray<QFixedPoint> &positions) { Q_ASSERT(ti.fontEngine->type() == QFontEngine::Freetype); |