D10071: Change an assert to do as the function documentation tells
Jaime Torres Amate
noreply at phabricator.kde.org
Wed Jan 24 14:10:04 UTC 2018
jtamate updated this revision to Diff 25883.
jtamate added a comment.
Do the right thing in the case of urls that end in / after the slash stripping.
This new test fails without the patch.
QCOMPARE(KIO::UrlUtil::firstChildUrl(lUrl("/d"), lUrl("/")), lUrl("/d"));
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10071?vs=25872&id=25883
REVISION DETAIL
https://phabricator.kde.org/D10071
AFFECTED FILES
autotests/urlutiltest.cpp
src/filewidgets/urlutil_p.h
To: jtamate, #frameworks, dfaure
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180124/5c967216/attachment.html>
More information about the Kde-frameworks-devel
mailing list