dfaure added inline comments.
INLINE COMMENTS
> mkpathjob.cpp:46
> for (; i < basePathComponents.count() && i <
> m_pathComponents.count(); ++i) {
> if (m_pathComponents.at(i) == basePathComponents.at(i)) {
> + if (m_url.path() == QLatin1Char('/')) {
I think it's time to put m_pathComponents.at(i) into a local variable ;-)
> mkpathjob.cpp:65
> + QString testDir;
> + if (m_url.toLocalFile() == QLatin1Char('/')) {
> + testDir = m_url.toLocalFile() + m_pathComponents.at(i);
same here with m_url.toLocalFile()
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D8836
To: elvisangelaccio, #frameworks, dfaure, thiago
