D9094: Use a faster way to determine kio-stash isn't installed

Eike Hein noreply at phabricator.kde.org
Fri Dec 1 21:52:16 GMT 2017


hein created this revision.
hein added reviewers: davidedmundson, elvisangelaccio, shortstheory.
Restricted Application added a subscriber: Dolphin.

REVISION SUMMARY
  Dolphin needs to figure out whether or not kio-stash is installed
  to set the enabled state of a UI action.
  
  When KProtocolInfo::isKnownProtocol can't find a protocol it gets
  worried its protocol cache might be out of date, so it rebuilds it
  (doing plenty of disk I/O) and looks again. kio-stash is currently
  not yet installed on many systems, so this means most Dolphin
  startups out there.
  
  This patch switches to using QDBusConnectionInterface::isServiceRegistered
  instead to determine whether the stash notifier daemon is running,
  which should be faster than doing disk I/O.

REPOSITORY
  R318 Dolphin

BRANCH
  master

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

AFFECTED FILES
  src/dolphinmainwindow.cpp

To: hein, davidedmundson, elvisangelaccio, shortstheory
Cc: #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171201/d420ace7/attachment.htm>


More information about the kfm-devel mailing list