[PATCH] D86564: [Support][Windows] Fix incorrect GetFinalPathNameByHandleW() return value check in realPathFromHandle()

[Andrew Ng via Phabricator via llvm-commits]

andrewng accepted this revision.
andrewng added a comment.
This revision is now accepted and ready to land.

LGTM with that last one change.



================
Comment at: llvm/lib/Support/Windows/Path.inc:361
     CountChars = ::GetFinalPathNameByHandleW(
-        H, Buffer.data(), Buffer.capacity() - 1, FILE_NAME_NORMALIZED);
+        H, Buffer.data(), Buffer.capacity(), FILE_NAME_NORMALIZED);
   }
----------------
Sorry, just noticed that this invocation uses `Buffer.data()` whereas the call above uses `Buffer.begin()`. Would be good to make it the same. Looks like `begin()` is the preferred style in this file.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86564/new/

https://reviews.llvm.org/D86564