[Nepomuk] [PATCHES] QDBusServiceWatcher and Strigi

Sebastian Trüg trueg at kde.org
Sat Apr 10 10:29:11 CEST 2010


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
> Nepomuk at kde.org
> https://mail.kde.org/mailman/listinfo/nepomuk


More information about the Nepomuk mailing list