<div dir="ltr">I'm going to give it a build/test run and can check in after that.  D3929 needs to go in first so I'll be looking at that one next, then this one.</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Wed, May 28, 2014 at 9:55 AM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Looks good.<br>
<div><div class="h5"><br>
> On May 27, 2014, at 5:46 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
><br>
> Hi tfiala,<br>
><br>
> As part of the contract of a dll-exported C++ interface, classes that are not exported should not be accessible via a public interface of an exported class, and doing so generates warnings on MSVC.  These warnings can be safely suprressed as long as all non-exported classes are part of the exported class's private interface, so this change moves all such methods to the private interface, friending where necessary, and finally suppresses the warnings on MSVC.<br>

><br>
> <a href="http://reviews.llvm.org/D3934" target="_blank">http://reviews.llvm.org/D3934</a><br>
><br>
> Files:<br>
>  CMakeLists.txt<br>
>  include/lldb/API/SBDebugger.h<br>
>  include/lldb/API/SBFrame.h<br>
>  include/lldb/API/SBProcess.h<br>
>  include/lldb/API/SBQueueItem.h<br>
>  include/lldb/API/SBTarget.h<br>
>  include/lldb/API/SBThread.h<br>
>  include/lldb/API/SBValue.h<br>
>  include/lldb/API/SBWatchpoint.h<br>
>  source/API/SBDebugger.cpp<br>
</div></div>> <D3934.9856.patch>_______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>