D22727: allow kio-extras to build with mingw on win32 and remove unnecessary includes
David Faure
noreply at phabricator.kde.org
Thu Aug 1 23:49:20 BST 2019
dfaure requested changes to this revision.
dfaure added a comment.
The Qt documentation says "Warning: Because of differences in the platforms supported by Qt, the semantics of ReadUser, WriteUser and ExeUser are platform-dependent: On Unix, the rights of the owner of the file are returned and on Windows the rights of the current user are returned. "
But, wait, this code is mixing "int permissions" (*) with the QFileDevice enum, that doesn't make any sense to me.
(*) this comes from KIO::put, which takes unix permissions on unix, not sure what it takes on Windows...
This doesn't match: unix permissions are octal (e.g. group read is 040 in octal), QFileDevice enum is hex (0x040).
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D22727
To: brute4s99, vonreth, dfaure, pino
Cc: pino, kde-frameworks-devel, kfm-devel, aprcela, vmarinescu, fprice, LeGast00n, sbergeron, fbampaloukas, alexde, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190801/e24804c0/attachment.html>
More information about the Kde-frameworks-devel
mailing list