WM_CLOSE needs it's own case.

comintern · comintern · commit d524bb12e5aa · 2016-06-28T12:24:28.000-05:00
diff --git a/RetailCoder.VBE/Common/RubberduckHooks.cs b/RetailCoder.VBE/Common/RubberduckHooks.cs
@@ -224,13 +224,15 @@ private IntPtr WindowProc(IntPtr hWnd, uint uMsg, IntPtr wParam, IntPtr lParam)
                     case WM.SETFOCUS:
                         Attach();
                         break;
-                    case WM.NCACTIVATE:
-                    case WM.CLOSE:
+                    case WM.NCACTIVATE:                   
                         if (wParam == IntPtr.Zero)
                         {
                             Detach();
                         }
                         break;
+                    case WM.CLOSE:
+                        Detach();
+                        break;
                 }
 
                 return suppress 

Original file line number	Diff line number	Diff line change
`@@ -224,13 +224,15 @@ private IntPtr WindowProc(IntPtr hWnd, uint uMsg, IntPtr wParam, IntPtr lParam)`
`224`	`224`	`case WM.SETFOCUS:`
`225`	`225`	`Attach();`
`226`	`226`	`break;`
`227`		`- case WM.NCACTIVATE:`
`228`		`- case WM.CLOSE:`
	`227`	`+ case WM.NCACTIVATE:`
`229`	`228`	`if (wParam == IntPtr.Zero)`
`230`	`229`	`{`
`231`	`230`	`Detach();`
`232`	`231`	`}`
`233`	`232`	`break;`
	`233`	`+ case WM.CLOSE:`
	`234`	`+ Detach();`
	`235`	`+ break;`
`234`	`236`	`}`
`235`	`237`
`236`	`238`	`return suppress`