From kde-core-devel Tue Aug 19 18:20:30 2003 From: Matthias Kretz Date: Tue, 19 Aug 2003 18:20:30 +0000 To: kde-core-devel Subject: Re: [PATCH] KCModuleLoader et al. X-MARC-Message: https://marc.info/?l=kde-core-devel&m=106131798620310 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--Boundary-02=_ypmQ/G3GuIalnIl" --Boundary-02=_ypmQ/G3GuIalnIl Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline On Tuesday August 19 2003 19:51, Nadeem Hasan wrote: > Comments? =46rom what I've seen in the kutils/kcm* code this looks good to me. But for completeness I think you should also add "const QStringList & args = =3D=20 QStringList()" to the new ctor. =2D-=20 C'ya Matthias ________________________________________________________ Matthias Kretz (Germany) <>< http://Vir.homeip.net/ MatthiasKretz@gmx.net, kretz@kde.org, Matthias.Kretz@urz.uni-heidelberg.de --Boundary-02=_ypmQ/G3GuIalnIl Content-Type: application/pgp-signature Content-Description: signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2 (GNU/Linux) iD8DBQA/Qmpyyg4WnCj6OIoRArTLAKDbB1EMhW97+PemBfO8cOCVzYgtMQCg5ySv Z/SiSuCHX10B29fxX/fGXx8= =CJFC -----END PGP SIGNATURE----- --Boundary-02=_ypmQ/G3GuIalnIl--