Review Request 124607: KService: detect ksycoca changing on disk without relying on the DBus signal.
Alex Merry
alex.merry at kde.org
Tue Aug 4 09:43:26 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124607/#review83416
-----------------------------------------------------------
src/sycoca/ksycoca.h (line 177)
<https://git.reviewboard.kde.org/r/124607/#comment57662>
Why have a bool return type on an internal method, when no call actually uses that value?
- Alex Merry
On Aug. 3, 2015, 10:39 p.m., David Faure wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124607/
> -----------------------------------------------------------
>
> (Updated Aug. 3, 2015, 10:39 p.m.)
>
>
> Review request for KDE Frameworks and Vishesh Handa.
>
>
> Repository: kservice
>
>
> Description
> -------
>
> This fixes threads without an event loop (e.g. in krunner) never noticing
> changes in ksycoca.
>
> Done-with: Vishesh Handa
>
>
> Diffs
> -----
>
> autotests/ksycocathreadtest.cpp 0d6ed4b89581fc506ea79c32c1de314637decf3e
> src/services/kmimetypetrader.cpp 655e5de9d272176e06af48a8417ea06e2412b9d8
> src/services/kplugininfo.cpp 56dc0b43036a2a49804ff799360c29c2121dda54
> src/services/kservice.cpp 3639a2875e5dbaefc6511d36d7b7f0fb7697ea74
> src/services/kservicegroup.cpp a2905da347877ccf34677776b63bd0eae518101f
> src/services/kservicetype.cpp 5e71351e5b7a0acef96104d8d14fa5b27817595d
> src/services/kservicetypetrader.cpp 290e44e9161c8db47278543714426fdd3b5a87af
> src/sycoca/ksycoca.h 542029c7c21d495456065e51b485a26509152cd5
> src/sycoca/ksycoca.cpp 32d1689ad9cd01d3a8adb5123a1848278714ca95
> src/sycoca/ksycoca_p.h 393cfcf8f04c369ac9c03712d917c49112771838
>
> Diff: https://git.reviewboard.kde.org/r/124607/diff/
>
>
> Testing
> -------
>
> The improved unittest was failing with threadsWhoSawFakeService() always only 2 out of 5 (the 2 threads with an event loop). Passes now.
>
>
> Thanks,
>
> David Faure
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150804/6e5536e9/attachment.html>
More information about the Kde-frameworks-devel
mailing list