D15237: [ViewProperties] Check part of home first before doing file system stuff

Kai Uwe Broulik noreply at phabricator.kde.org
Mon Sep 3 11:17:00 BST 2018


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

REVISION SUMMARY
  There's no point in creating a `QFileInfo` instance and checking for file properties if we're not going to do anything with it when not inside home

TEST PLAN
  - Accessing a slow or stale mount in `/run/user/foo/bar` no longer blocks here and is a nice simple optimization anyway imho

REPOSITORY
  R318 Dolphin

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

AFFECTED FILES
  src/views/viewproperties.cpp

To: broulik, #dolphin, elvisangelaccio, lbeltrame
Cc: kfm-devel, spoorun, navarromorales, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180903/d3fd9737/attachment.htm>


More information about the kfm-devel mailing list