Ignore Visual C++ internal exception for reporting thread-names

Author: fischman-bcny

Headers/DebugServer2/Host/Windows/ExtraWrappers.h

@@ -19,6 +19,7 @@
 #define DS2_EXCEPTION_UNCAUGHT_COM 0x800706BA
 #define DS2_EXCEPTION_UNCAUGHT_USER 0xE06D7363
 #define DS2_EXCEPTION_UNCAUGHT_WINRT 0x40080201
+#define DS2_EXCEPTION_VC_THREAD_NAME_SET 0x406D1388
 
 #if !defined(HAVE_WaitForDebugEventEx)
 #define WaitForDebugEventEx WaitForDebugEvent

Sources/Target/Windows/Thread.cpp

@@ -125,6 +125,14 @@ void Thread::updateState(DEBUG_EVENT const &de) {
         PRI_PTR_CAST(de.u.Exception.ExceptionRecord.ExceptionAddress));
 
     switch (de.u.Exception.ExceptionRecord.ExceptionCode) {
+    case DS2_EXCEPTION_VC_THREAD_NAME_SET:
+      // This is a special exception code that is used by the Microsoft C++
+      // runtime to notify the debugger that a thread has started or exited.
+      // We don't care about this, so we just ignore it. Some context at
+      // https://github.com/go-delve/delve/issues/1383.
+      _stopInfo.clear();
+      _process->resume();
+      break;
     case STATUS_BREAKPOINT:
     case STATUS_SINGLE_STEP: {
       _stopInfo.event = StopInfo::kEventStop;

Sources/Utils/Windows/Stringify.cpp

@@ -120,6 +120,8 @@ char const *Stringify::ExceptionCode(DWORD code) {
     return "0xE06D7363 (uncaught user exception)";
   case DS2_EXCEPTION_UNCAUGHT_WINRT:
     return "0x40080201 (uncaught WinRT exception)";
+  case DS2_EXCEPTION_VC_THREAD_NAME_SET:
+    return "0x406D1388 (VC thread name set)";
     DO_DEFAULT("unknown exception code", code)
   }
 }