[Nepomuk] Review Request 112011: Use KDBusConnectionPool instead of QDBusConnection

[David Faure]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112011/#review37931
-----------------------------------------------------------


"Testing done" is empty. Please make sure you test this, because it could change the thread in which registered objects get called (I'm not actually sure how that works with QDBusConnection).

For outgoing calls using KDBusConnectionPool is a very simple change, but for registering objects for incoming calls, better double-check.

"Ship it" if you tested that the incoming calls happen in the right thread.

- David Faure


On Aug. 11, 2013, 7:38 a.m., Vishesh Handa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112011/
> -----------------------------------------------------------
> 
> (Updated Aug. 11, 2013, 7:38 a.m.)
> 
> 
> Review request for Nepomuk and David Faure.
> 
> 
> Description
> -------
> 
>     Use KDBusConnectionPool instead of QDBusConnection
>     
>     QDBusConnection cannot be used across multiple threads
> 
> 
> This addresses bugs 315078 and 319165.
>     http://bugs.kde.org/show_bug.cgi?id=315078
>     http://bugs.kde.org/show_bug.cgi?id=319165
> 
> 
> Diffs
> -----
> 
>   services/storage/backup/backupmanager.cpp 60ce815 
>   services/storage/datamanagementcommand.cpp 5d8036c 
>   services/storage/ontologyloader.cpp 9acac5f 
>   services/storage/query/folderconnection.cpp 1731180 
>   services/storage/query/queryservice.cpp 593abd9 
>   services/storage/repository.cpp 6a2dd18 
>   services/storage/resourcewatcherconnection.cpp 0b245d7 
>   services/storage/resourcewatchermanager.cpp 0b20cb8 
>   services/storage/storage.cpp 1b63dff 
> 
> Diff: http://git.reviewboard.kde.org/r/112011/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Vishesh Handa
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20130816/30b5e942/attachment.html>