<div dir="ltr">Ha yes I remember this part.<div>I delayed it because I wasn't sure if it might not be better to handle this translation directly in Error class (i.e. <span style="color:rgb(0,0,0);white-space:pre-wrap">error.GetSocketError() with specific Windows implementation). That would avoid lot of #ifdef in all socket code.</span></div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 8, 2013 at 3:54 PM, Carlo Kok <span dir="ltr"><<a href="mailto:ck@remobjects.com" target="_blank">ck@remobjects.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Op 7-7-2013 11:44, Virgile Bello schreef:<div><div class="h5"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Here is a patch allowing compilation of LLDB with MinGW.<br>
(it still compiles on Linux after the patch)<br>
Can someone please review (and merge in trunk if everything seems good)?<br>
<br>
Soon some other patches should follow:<br>
- 1 for MSVC compilation<br>
- 1 to add a lldbProcessWindows using Win32 debugging API, so that<br>
gdb/clang compiled programs can be debugged in Windows.<br>
<br>
Hopefully I can get all this into trunk quickly so that I don't need to<br>
rebase/merge too much.<br>
Thanks!<br>
</blockquote>
<br>
<br></div></div>
Patch looks good. the only thing I'm missing is the error handing in ConnectionFileDescriptor.cpp, though that can be done as an incremental patch (Windows sockets return WSAWOULDBLOCK instead of EAGAIN etc), see:<br>


<a href="http://llvm.org/svn/llvm-project/lldb/branches/windows/source/Core/ConnectionFileDescriptor.cpp" target="_blank">http://llvm.org/svn/llvm-<u></u>project/lldb/branches/windows/<u></u>source/Core/<u></u>ConnectionFileDescriptor.cpp</a><br>


<br>
in the Windows part.<br>
</blockquote></div><br></div>