Merge pull request #9088 from mono/jstedfast-debugger-updatepinicon-crash-fix

jstedfast · web-flow · commit 7965de466913 · 2019-10-25T15:21:02.000-04:00
[Debugger] ViewAtColumn can return null so protect against that
diff --git a/main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger/ObjectValue/Mac/MacObjectValueTreeView.cs b/main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger/ObjectValue/Mac/MacObjectValueTreeView.cs
@@ -653,7 +653,7 @@ void UpdatePinIcon (nint row, bool hover)
 			if (rowView != null) {
 				var pinView = (MacDebuggerObjectPinView) rowView.ViewAtColumn (ColumnCount - 1);
 
-				pinView.SetMouseHover (hover);
+				pinView?.SetMouseHover (hover);
 			}
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -653,7 +653,7 @@ void UpdatePinIcon (nint row, bool hover)`
`653`	`653`	`if (rowView != null) {`
`654`	`654`	`var pinView = (MacDebuggerObjectPinView) rowView.ViewAtColumn (ColumnCount - 1);`
`655`	`655`
`656`		`- pinView.SetMouseHover (hover);`
	`656`	`+ pinView?.SetMouseHover (hover);`
`657`	`657`	`}`
`658`	`658`	`}`
`659`	`659`