One comment though: for empty strings you should rather use QString()
than QLatin1String("").Cheers, Sebastian On 04/10/2010 12:07 AM, Vishesh Handa wrote: > The first patch (strigi_icon.diff) simply adds the /Strigi/ icon as the > window icon in the Strigi Status Widget. Prior to this patch it showed a > Xorg icon. > > The second and third patches* *implement the QDBusServiceWatcher class* > *in /nepomuk/services/queryservice//* *& > nepomuk/kioslaves/search/kdedmodule/ and* have not been tested. *That's > cause I have no idea how to test them. So, please, *DO NOT* commit them. > (Feel free to commit the Strigi patch :-) > > If somebody knows, I would like to know how I can test them. Both the > patches have a fairly similar design, and if one works so should the other. > > Thanks > > - Vishesh Handa > > > > _______________________________________________ > Nepomuk mailing list > [email protected] > https://mail.kde.org/mailman/listinfo/nepomuk _______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
