D18563: Don't create directory tree when a new folder has a '/' in the name

Andrius da Costa Ribas noreply at phabricator.kde.org
Wed Jan 30 20:51:03 GMT 2019


andriusr added inline comments.

INLINE COMMENTS

> ngraham wrote in knewfilemenu.cpp:907
> If Windows allows slashes in the filename, shouldn't they just be a part of the filename and not create a directory tree? If so, then we don't even need this `#ifdef` condition at all; `KIO::mkdir(url);` will take care of the behavior on each individual platform

if this is a QUrl, then `/` is the separator on every platform (e.g.: `file://C:/WINDOWS/System32`)

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D18563

To: shubham, ngraham, #frameworks, #dolphin, dfaure, elvisangelaccio, pino
Cc: andriusr, emateli, cfeck, acrouthamel, markg, ndavis, dfaure, elvisangelaccio, pino, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190130/cba0cf6b/attachment.htm>


More information about the kfm-devel mailing list