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

[Jim Ingham via Phabricator via lldb-commits]

jingham added inline comments.


================
Comment at: lldb/tools/lldb-vscode/lldb-vscode.cpp:55
 #define PATH_MAX MAX_PATH
+#endif
 typedef int socklen_t;
----------------
labath wrote:
> 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...).
There's no reason we couldn't add an SBString class to vend non-fixed length strings.  There haven't been enough instances where we really needed it to motivate that, but we have SBStringList to wrap vectors of strings, so it wouldn't be out of place.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65691