[Lldb-commits] [PATCH] D65691: Various build fixes for lldb on MinGW

[Pavel Labath via Phabricator via lldb-commits]

labath accepted this revision.
labath added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lldb/tools/lldb-vscode/lldb-vscode.cpp:55
 #define PATH_MAX MAX_PATH
+#endif
 typedef int socklen_t;
----------------
amccarth wrote:
> hhb wrote:
> > amccarth wrote:
> > > Nothing in the rest of this .cpp file uses PATH_MAX, so just delete the `#define` instead of executing it conditionally.
> > PATH_MAX is used in SendProcessEvent (line 283)
> Got it.  When I asked Phabricator to show me all the lines, it didn't show me _all_ the lines.
> 
> That said, it seems unfortunate that it's yet another fixed-length filename buffer instead of getting a ConstString from the FileSpec.  But that's outside the scope of this change.
The problem is that you're dealing with the SBFileSpec here, and that makes things slightly complicated (stable api, not being able to return c++ objects through it, etc...).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65691