ahmadsamir added inline comments. INLINE COMMENTS
> apol wrote in kcoredirlister.cpp:1045 > Is this to force dir at the end? You mean at the front? I am not sure, but I matched the behaviour of the original code, it first appended "dir" then the QUrl list from canonicalUrls. It sort of makes sense, since we want to update the listing (kcoredirlister) of "dir" then the symlinks to it. From the comment right before canonicalUrls declaration[1], see https://bugs.kde.org/show_bug.cgi?id=213799 [1] https://cgit.kde.org/kio.git/tree/src/core/kcoredirlister_p.h#n474 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D27947 To: ahmadsamir, #frameworks, dfaure, meven Cc: apol, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns