QQuickUniversalFocusRectangle: optimize string usage

Replace QString::arg() chain with multi-arg overloaded function.
It's faster because of args are replaced in one pass.

While touching the code, also cache width and height of bound.

Change-Id: Id8d19143f426ed1998713204dc9c8d4b51e58db5
Reviewed-by: Mitch Curtis <mitch.curtis@theqtcompany.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/src/imports/controls/universal/qquickuniversalfocusrectangle.cpp b/src/imports/controls/universal/qquickuniversalfocusrectangle.cpp
index a9b77ee0..3850e4be 100644
--- a/src/imports/controls/universal/qquickuniversalfocusrectangle.cpp
+++ b/src/imports/controls/universal/qquickuniversalfocusrectangle.cpp
@@ -55,9 +55,11 @@ void QQuickUniversalFocusRectangle::paint(QPainter *painter)
         return;
 
     QRect bounds = boundingRect().toAlignedRect();
-    const QString key = QStringLiteral("qquickuniversalfocusrectangle_%1_%2").arg(bounds.width()).arg(bounds.height());
+    const int boundsWidth = bounds.width();
+    const int boundsHeight = bounds.width();
+    const QString key = QStringLiteral("qquickuniversalfocusrectangle_%1_%2").arg(QString::number(boundsWidth), QString::number(boundsHeight));
 
-    QPixmap pixmap(bounds.width(), bounds.height());
+    QPixmap pixmap(boundsWidth, boundsHeight);
     if (!QPixmapCache::find(key, &pixmap)) {
         bounds.adjust(0, 0, -1, -1);
         pixmap.fill(Qt::transparent);