diff options
author | Jeffrey Stedfast <jestedfa@microsoft.com> | 2020-01-17 23:06:55 +0300 |
---|---|---|
committer | Matt Ward <ward.matt@gmail.com> | 2020-01-22 12:55:40 +0300 |
commit | dd8f768cacda347c9fcb2a3086914f9581c4ccc3 (patch) | |
tree | 30aa677ee67da618d39c711b1cd653dc1502dfc8 | |
parent | ef3782e3802c3982838bbf840ee63ec360d4d852 (diff) |
[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
-rw-r--r-- | main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger.VSTextView/QuickInfo/DebuggerQuickInfoSource.cs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger.VSTextView/QuickInfo/DebuggerQuickInfoSource.cs b/main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger.VSTextView/QuickInfo/DebuggerQuickInfoSource.cs index 05ba944d75..56a431e457 100644 --- a/main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger.VSTextView/QuickInfo/DebuggerQuickInfoSource.cs +++ b/main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger.VSTextView/QuickInfo/DebuggerQuickInfoSource.cs @@ -135,7 +135,7 @@ namespace MonoDevelop.Debugger.VSTextView.QuickInfo if (cancellationToken.IsCancellationRequested) return; - if (val == null || val.IsUnknown || val.IsNotSupported) + if (val == null || val.IsUnknown || val.IsNotSupported || val.IsError) return; timer.Success = true; |