[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