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