D20133: [UserMetaData] Untangle Windows, Linux/BSD/Mac and stub code.

Stefan BrĂ¼ns noreply at phabricator.kde.org
Sat Mar 30 22:13:28 GMT 2019


bruns created this revision.
bruns added reviewers: Baloo, Frameworks, Windows, FreeBSD, ngraham, astippich.
Herald added projects: Frameworks, Baloo.
Herald added a subscriber: kde-frameworks-devel.
bruns requested review of this revision.

REVISION SUMMARY
  The Windows code only shares the function signature with the other
  variants, while the actual code is vastly different. One example are
  path and attribute names, which are UTF-16 on Windows and UTF-8 otherwise,
  but QFile::encodeName() is also called on Windows.
  
  This fixes also two small bugs on Windows, one missing CloseHandle in
  case the attribute has a size of 0, and a wrong return value in case
  writing the ADS fails.

TEST PLAN
  Linux: ctest
  Windows/Mac/*BSD: untested

REPOSITORY
  R286 KFileMetaData

BRANCH
  master

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

AFFECTED FILES
  src/xattr_p.h

To: bruns, #baloo, #frameworks, #windows, #freebsd, ngraham, astippich
Cc: kde-frameworks-devel, gennad, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190330/470ef37c/attachment.html>


More information about the Kde-frameworks-devel mailing list