From kde-core-devel  Thu Dec 04 02:00:19 2008
From: Sebastian Sauer <mail () dipe ! org>
Date: Thu, 04 Dec 2008 02:00:19 +0000
To: kde-core-devel
Subject: kcmodule and kcmoduleproxy
Message-Id: <200812040300.19988.mail () dipe ! org>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=122835969412478

Hi *

The KCMultiDialog class used for e.g. "Configure Konqueror" uses kcmodule for 
its configurable items and kcmoduleproxy which allows to delay loading the 
kcmodule plugins till they are requested.

Current situation is, that the kcmodule's are just always loaded and not only 
if they are needed/displayed. That results here in a delay of around 10 
seconds if I call "Configure Konqueror" till the dialog is displayed and does 
render the kcmoduleproxy's unneeded. The reason for that is, that on 
addPage() also KCModuleProxy::minimumSizeHint() got called and there the 
kcmodule is loaded.

Attached patch does fix that. Ok to commit?

Index: kcmoduleproxy.cpp
===================================================================
--- kcmoduleproxy.cpp   (revision 892245)
+++ kcmoduleproxy.cpp   (working copy)
@@ -343,7 +343,6 @@

 QSize KCModuleProxy::minimumSizeHint() const
 {
-       realModule();
        return QWidget::minimumSizeHint();
 }