D29442: Don't use local URL for KMoreTool

Kai Uwe Broulik noreply at phabricator.kde.org
Tue May 5 08:45:10 BST 2020


broulik created this revision.
broulik added reviewers: Dolphin, pdabrowski, elvisangelaccio.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
broulik requested review of this revision.

REVISION SUMMARY
  This method is called a billion times and spawns a blocking(!) StatJob.
  KFind can search local URLs, too, so I don't see why this is necessary.
  Either way, I'd rather have a non-KDE search tool broken than the app.

TEST PLAN
  20.04
  
  - action still seems to work, kfind still starts

REPOSITORY
  R318 Dolphin

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

AFFECTED FILES
  src/dolphinmainwindow.cpp

To: broulik, #dolphin, pdabrowski, elvisangelaccio
Cc: kfm-devel, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200505/2b67f911/attachment.htm>


More information about the kfm-devel mailing list