D11204: Support NTFS hidden files

Mark Gaiser noreply at phabricator.kde.org
Sun Mar 25 16:53:06 UTC 2018


markg added inline comments.

INLINE COMMENTS

> rominf wrote in file_unix.cpp:546-550
> The logic behind this is that `getxattr` exists only on Linux (https://www.gnu.org/software/gnulib/manual/html_node/getxattr.html#getxattr) and `file_unix.cpp` is more Linux-y than `file.cpp`. If you think that I'm wrong, I can move it.

I "think" it belongs in createUDSEntry, but i'm not sure either. It would keep this else nice and clean. But i do see why you'd want to put it here... What's your take on this, @dfaure?

REPOSITORY
  R241 KIO

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

To: rominf, #dolphin, #frameworks, markg, dfaure
Cc: dfaure, markg, elvisangelaccio, ltoscano, anthonyfieroni, broulik, #frameworks, #dolphin, michaelh, ngraham
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180325/2e007f8b/attachment.html>


More information about the Kde-frameworks-devel mailing list