D17816: Support for xattrs on kio copy/move

Ömer Fadıl Usta noreply at phabricator.kde.org
Thu Sep 10 14:43:46 BST 2020


usta added inline comments.

INLINE COMMENTS

> jobtest.cpp:465
> +{
> +    QString writeTest = dir + "/fsXattrTestFile";
> +    createTestFile(writeTest);

const ?

> file_unix.cpp:1517
>          }
>          acl = acl_from_text(ACLString.toLatin1().constData());
>          if (acl_valid(acl) == 0) {     // let's be safe

isnt this ignoring acl_from_mode part ? I mean not sure but i think we need to check if it is nullptr or not before assigning it otherwise we will ignore the acl_from_mode part.

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

To: arrowd, dfaure, chinmoyr, bruns, #frameworks, tmarshall, usta, cochise
Cc: usta, scheirle, tmarshall, arrowd, cfeck, bruns, phidrho, dhaumann, funkybomber, abika, pino, davidedmundson, ngraham, atha.kane, spoorun, nicolasfella, kde-frameworks-devel, LeGast00n, cblack, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200910/0a3415c0/attachment.htm>


More information about the Kde-frameworks-devel mailing list