[PATCH] D21420: Properly handle short file names on the command line in Windows

[David Majnemer via llvm-commits]

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

LGTM w/ `GetLongPathNameW > MAX_PATH` addressed.


================
Comment at: lib/Support/Windows/Process.inc:236-237
@@ +235,4 @@
+  DWORD Length = GetLongPathNameW(Arg, LongPath.data(), LongPath.capacity());
+  if (Length == 0 || Length > MAX_PATH)
+    return mapWindowsError(GetLastError());
+  LongPath.set_size(Length);
----------------
Is an error code set for the case where `Length > MAX_PATH`?  The MSDN docs seemed a little ambiguous.


http://reviews.llvm.org/D21420