elvisangelaccio created this revision.
elvisangelaccio added a reviewer: dfaure.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.

REVISION SUMMARY
  Since 5.10 (commit 0dee566e98 in qtbase) Qt complains if we make a
  system call with an empty filename (with a "Empty filename passed to
  function" runtime warning).
  
  There is no reason to call `exists()` if we already know that a filename
  is empty. While at it, port to `QFileInfo::exists()` which is faster.

TEST PLAN
  Together with https://phabricator.kde.org/D8696, fixes all the runtime 
warnings when running `tests/kfilewidgettest_gui`.

REPOSITORY
  R241 KIO

BRANCH
  fix-empty-exists

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

AFFECTED FILES
  src/core/knfsshare.cpp
  src/core/ksambashare.cpp

To: elvisangelaccio, dfaure
Cc: #frameworks

Reply via email to