<div dir="ltr">I should be able to move it into `HandleExceptionEvent`</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 18, 2015 at 1:39 PM Adrian McCarthy <<a href="mailto:amccarth@google.com">amccarth@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">amccarth accepted this revision.<br>
This revision is now accepted and ready to land.<br>
<br>
================<br>
Comment at: source/Plugins/Process/Windows/Live/DebuggerThread.cpp:309<br>
@@ -308,1 +308,3 @@<br>
                     {<br>
+                        if (m_pid_to_detach != 0 && dbe.u.Exception.ExceptionRecord.ExceptionCode == EXCEPTION_BREAKPOINT) {<br>
+                            WINLOG_IFANY(WINDOWS_LOG_EVENT | WINDOWS_LOG_EXCEPTION | WINDOWS_LOG_PROCESS,<br>
----------------<br>
Open brace goes on the next line.  (Which appears to have originally been my mistake.  Sorry.)<br>
<br>
================<br>
Comment at: source/Plugins/Process/Windows/Live/DebuggerThread.cpp:316<br>
@@ -309,1 +315,3 @@<br>
+                        }<br>
+<br>
                         // Don't perform any blocking operations while we're shutting down.  That will<br>
----------------<br>
This is fine, but I'm growing concerned about the growing spaghetti-ness.  It's too bad we can't push this logic down into HandleExceptionEvent, in order to keep the cases in this deeply-nested switch statement easy to understand.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12980" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12980</a><br>
<br>
<br>
<br>
</blockquote></div>