Review Request 116681: Fix kio-help by adding a separator

Luigi Toscano luigi.toscano at tiscali.it
Sun Mar 9 21:18:23 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116681/
-----------------------------------------------------------

Review request for KDE Frameworks.


Repository: kio


Description
-------

Looking at the code it seems that QStandardPaths::locateAll() does not add the final '/' as the previous kdelibs code.

Without this fix kio-help does find the documentation.

General remark: maybe other parts of code which use QStandardPaths should be checked.


Diffs
-----

  src/ioslaves/help/kio_help.cpp 85b7975 

Diff: https://git.reviewboard.kde.org/r/116681/diff/


Testing
-------

kioclient cat help:/kioslave/ftp (from frameworks branch of kde-runtime) now works.


Thanks,

Luigi Toscano

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140309/deaffaa4/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list