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

Commit dd8f768

Browse files
jstedfastmrward
authored andcommitted
[Debugger] Tooltips should not be shown for conditional method invocations
When hovering the mouse cursor over most method invocation expressions, the logic returns a null expression to evaluate (thereby bypassing evaluation of said expression), but when hovering over conditional method invocations, the logic failed. Since the logic is identical to VS Windows, do what VS Windows does in this scenario as well, which is to ignore evaluation errors and not show them. Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1054139
1 parent ef3782e commit dd8f768

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ async Task EvaluateAndShowTooltipAsync (IAsyncQuickInfoSession session, ITextVie
135135
if (cancellationToken.IsCancellationRequested)
136136
return;
137137

138-
if (val == null || val.IsUnknown || val.IsNotSupported)
138+
if (val == null || val.IsUnknown || val.IsNotSupported || val.IsError)
139139
return;
140140

141141
timer.Success = true;

0 commit comments

Comments
 (0)