[Differential] [Commented On] D1497: [kioslave/desktop] Ensure there are no multiple / when creating the file path

dfaure (David Faure) noreply at phabricator.kde.org
Tue Apr 26 15:09:59 UTC 2016


dfaure added inline comments.

INLINE COMMENTS
  kioslave/desktop/kio_desktop.cpp:161 Can the path really *not* start with a '/' in this code ?
  From a QUrl point of view, it can happen with relative URLs ("desktop:foo.txt"), but we very rarely use that in KDE code, if at all (it's unusable without a reference directory).
  
  So it seems to me that just removing the '/' would be enough in practice.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

BRANCH
  fix-kioslave-desktop

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, Plasma, hein, dfaure
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160426/4df3334e/attachment.html>


More information about the Plasma-devel mailing list