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

(Updated Sept. 17, 2013, 11:14 a.m.)


Status
------

This change has been discarded.


Review request for KDE Frameworks.


Description
-------

It happens that sometimes urls get a double shash instead of one, to trash:/foo 
becoming trash://foo (an hostname)
this happens because if url.path() == "/"
the line
itemURL.setPath(itemURL.path() + '/' + fileName);
just doubles the /

I'm not sure the patch is correct, if path() is guaranteed to always have a 
trailing slash it may be simpler (altough qurl documentation doesn't suggest so)


Diffs
-----

  staging/kio/src/core/job.cpp e066ef4 

Diff: http://git.reviewboard.kde.org/r/112547/diff/


Testing
-------


Thanks,

Marco Martin

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to