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

dfaure (David Faure) noreply at phabricator.kde.org
Tue Apr 26 19:32:10 UTC 2016


dfaure added a comment.


  I'm pretty sure it should work. What changed in QUrl is that setPath used to normalize the path and doesn't do that anymore.
  (qtbase commit 2e1de7f3c4ca). So in 5.5 the double slash got simplified to a single slash, while in 5.6 it stays and breaks the test.
  Not putting a double slash in the first place will work with both Qt versions.

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/e782f4af/attachment.html>


More information about the Plasma-devel mailing list