D7043: Fix assert in UrlUtil::firstChildUrl()

2017-08-02 Thread Daniel Vrátil
This revision was automatically updated to reflect the committed changes. Closed by commit R241:7e03e503b00c: Fix assert in UrlUtil::firstChildUrl() (authored by dvratil). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7043?vs=17511&id=17540 REVISION DETAIL htt

D7043: Fix assert in UrlUtil::firstChildUrl()

2017-08-02 Thread David Faure
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Patch looks good. The word "normalize" in the commit log is somewhat confusing, this is only about stripping trailing slashes, not about QUrl::NormalizePathSegments (which is another matter

D7043: Fix assert in UrlUtil::firstChildUrl()

2017-08-01 Thread Daniel Vrátil
dvratil created this revision. dvratil added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY When lastUrl = "file:///folder/" and currentUrl = "file:///folder" then currentUrl.isParentOf(lastUrl) returns true, which causes the code to hit an asse