[Lldb-commits] [PATCH] D26209: Fix windows build after r285702, missing include to define PATH_MAX

Rudy Pons via lldb-commits lldb-commits at lists.llvm.org
Tue Nov 1 12:38:28 PDT 2016

Ilod created this revision.
Ilod added reviewers: labath, zturner, clayborg.
Ilod added a subscriber: lldb-commits.

Fix the windows build after r285702.
The removal of TimeValue.h inclusion removed the inner inclusion of PosixApi.h, which defined PATH_MAX on windows, used by many other files.
Adding the PosixApi.h inclusion should not cause a problem (it only include llvm/Support/Compiler.h on non-windows platform, and the needed file on windows ones).



Index: include/lldb/Host/FileSpec.h
--- include/lldb/Host/FileSpec.h
+++ include/lldb/Host/FileSpec.h
@@ -19,6 +19,7 @@
 // Project includes
 #include "lldb/Core/ConstString.h"
 #include "lldb/Core/STLUtils.h"
+#include "lldb/Host/PosixApi.h"
 #include "lldb/lldb-private.h"
 namespace lldb_private {

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26209.76618.patch
Type: text/x-patch
Size: 371 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20161101/da020d8e/attachment.bin>

More information about the lldb-commits mailing list