<div dir="ltr">I patched it in this morning, but it doesn't compile on Windows.  I was able to make it work with a few tweaks at the top of SelectHelper.cpp:<div><br></div><div><div>// C Includes</div><div>#include <errno.h></div><div>#if defined(_WIN32)</div><div>#define NOMINMAX</div><div>#include <winsock2.h></div><div>#else</div><div>#include <sys/select.h></div><div>#endif</div></div><div><br></div><div>The key bits:  don't include <sys/select.h> on Windows and #define NOMINMAX on Windows to avoid stupid macros that conflict with std::min and std::max.</div><div><br></div><div>I'd probably put the #if under "// Other libraries and framework includes" rather than "// C Includes", but that's a minor style nit.  Either way, this change makes it compile on Windows.</div><div><br></div><div>Adrian.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 8, 2016 at 5:05 PM, Greg Clayton <span dir="ltr"><<a href="mailto:clayborg@gmail.com" target="_blank">clayborg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg updated this revision to Diff 67253.<br>
clayborg added a comment.<br>
<br>
Updated with all comments taken into account. Zach, please try this out and let me know if it compiles.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D22950" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D22950</a><br>
<br>
Files:<br>
  include/lldb/Utility/<wbr>SelectHelper.h<br>
  lldb.xcodeproj/project.pbxproj<br>
  source/Host/common/Editline.<wbr>cpp<br>
  source/Host/posix/<wbr>ConnectionFileDescriptorPosix.<wbr>cpp<br>
  source/Host/posix/PipePosix.<wbr>cpp<br>
  source/Target/Process.cpp<br>
  source/Utility/SelectHelper.<wbr>cpp<br>
<br>
</blockquote></div><br></div>