From 8fb62cc0a4d924b80074cca41c84a2695873652a Mon Sep 17 00:00:00 2001 From: Aleix Pol Date: Wed, 9 Jul 2014 12:43:08 +0200 Subject: [PATCH] Cleanup Re-use containment variable instead of keep fetching it every time. It's not a trivial function, so better reduce the calls. --- src/plasma/private/applet_p.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/plasma/private/applet_p.cpp b/src/plasma/private/applet_p.cpp index 218b26fb1..2f680bf19 100644 --- a/src/plasma/private/applet_p.cpp +++ b/src/plasma/private/applet_p.cpp @@ -314,12 +314,13 @@ void AppletPrivate::setUiReady() } else { c = q->containment(); if (c) { - q->containment()->d->loadingApplets.remove(q); - Applet *a = static_cast(q->containment()); - if (q->containment()->d->loadingApplets.isEmpty() && !a->d->uiReady) { + c->d->loadingApplets.remove(q); + Applet *a = qobject_cast(c); + Q_ASSERT(a); + if (c->d->loadingApplets.isEmpty() && a && !a->d->uiReady) { a->d->uiReady = true; if (a->d->started) { - emit q->containment()->uiReadyChanged(true); + emit c->uiReadyChanged(true); } } }