Skip to content
This repository was archived by the owner on Oct 4, 2021. It is now read-only.

Commit 596c8e1

Browse files
authored
Merge pull request #9042 from mono/jstedfast-debugger-tooltip-show-crash-fix
[Debugger] Make sure tooltip request wasn't cancelled before showing
2 parents 1b2255c + 121230b commit 596c8e1

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger.VSTextView/QuickInfo/DebuggerQuickInfoSource.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,11 @@ private async Task EvaluateAndShowTooltipAsync (IAsyncQuickInfoSession session,
177177
var bounds = view.TextViewLines.GetCharacterBounds (point);
178178
var rect = new CoreGraphics.CGRect (bounds.Left - view.ViewportLeft, bounds.Top - view.ViewportTop, bounds.Width, bounds.Height);
179179

180+
if (cocoaView.IsClosed) {
181+
LoggingService.LogWarning ("Editor window has been closed before debugger tooltip was shown. How did this happen?");
182+
return;
183+
}
184+
180185
window.Show (rect, cocoaView.VisualElement, AppKit.NSRectEdge.MaxXEdge);
181186
#else
182187
throw new NotImplementedException ();

0 commit comments

Comments
 (0)