D25789: Correctly report if baloo_file is unavailable

Stefan BrĂ¼ns noreply at phabricator.kde.org
Sat Dec 7 02:16:01 GMT 2019


bruns added inline comments.

INLINE COMMENTS

> monitor.cpp:67
>          m_balooRunning = false;
>          QDBusServiceWatcher* balooWatcher = new QDBusServiceWatcher(m_scheduler->service(),
>                                                                  m_bus,

The watcher should be installed unconditionally, and first.

Checking state first and the instaling the watcher is racy.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D25789

To: davidedmundson
Cc: bruns, ngraham, kde-frameworks-devel, #baloo, hurikhan77, lots0logs, LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191207/81a81d2a/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list