From 3b02ff3c17067b0d2e80ff3c7de21999409c856e Mon Sep 17 00:00:00 2001 From: David Edmundson Date: Wed, 4 Apr 2018 14:22:00 +0100 Subject: [PATCH] Avoid pointless config parse in ksplash Summary: Due to the nature of cascading configs we have already loaded and parsed kdeglobals, it's wasteful to do it again. Especially in ksplash where startup time is important. Test Plan: Changed lnf to org.obnosim.tweak.desktop which contains a new splash Logged out and back in. Still got a different splash. Reviewers: #plasma Subscribers: plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D11923 --- ksplash/ksplashqml/splashwindow.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ksplash/ksplashqml/splashwindow.cpp b/ksplash/ksplashqml/splashwindow.cpp index 0ba65dfe2..9e2035099 100644 --- a/ksplash/ksplashqml/splashwindow.cpp +++ b/ksplash/ksplashqml/splashwindow.cpp @@ -122,7 +122,7 @@ void SplashWindow::setGeometry(const QRect& rect) if (oldGeometryEmpty) { KPackage::Package package = KPackage::PackageLoader::self()->loadPackage(QStringLiteral("Plasma/LookAndFeel")); - KConfigGroup cg(KSharedConfig::openConfig(QStringLiteral("kdeglobals")), "KDE"); + KConfigGroup cg(KSharedConfig::openConfig(), "KDE"); const QString packageName = cg.readEntry("LookAndFeelPackage", QString()); if (!packageName.isEmpty()) { package.setPath(packageName);