Don't use anonymous delegate to unsubscribe event

haacked · haacked · commit d637fb721f0d · 2015-10-08T15:59:05.000-07:00
diff --git a/SeeGitApp/Extensions/CommitAdornerBehavior.cs b/SeeGitApp/Extensions/CommitAdornerBehavior.cs
@@ -167,8 +167,8 @@ private void HideAdorner(object parameter)
             {
                 if (_adornerControl.IsMouseOver)
                 {
-                    _adornerControl.MouseLeave -= (s, e) => _adornerControl.Visibility = AdornerVisible;
-                    _adornerControl.MouseLeave += (s, e) => _adornerControl.Visibility = AdornerVisible;
+                    _adornerControl.MouseLeave -= SetVisibility;
+                    _adornerControl.MouseLeave += SetVisibility;
                 }
                 else
                 {
@@ -177,6 +177,11 @@ private void HideAdorner(object parameter)
             }
         }
 
+        private void SetVisibility(object source, MouseEventArgs e)
+        {
+            _adornerControl.Visibility = AdornerVisible;
+        }
+
         /// <summary>
         /// ShowAdornerCommand
         /// </summary>

Original file line number	Diff line number	Diff line change
`@@ -167,8 +167,8 @@ private void HideAdorner(object parameter)`
`167`	`167`	`{`
`168`	`168`	`if (_adornerControl.IsMouseOver)`
`169`	`169`	`{`
`170`		`- _adornerControl.MouseLeave -= (s, e) => _adornerControl.Visibility = AdornerVisible;`
`171`		`- _adornerControl.MouseLeave += (s, e) => _adornerControl.Visibility = AdornerVisible;`
	`170`	`+ _adornerControl.MouseLeave -= SetVisibility;`
	`171`	`+ _adornerControl.MouseLeave += SetVisibility;`
`172`	`172`	`}`
`173`	`173`	`else`
`174`	`174`	`{`
`@@ -177,6 +177,11 @@ private void HideAdorner(object parameter)`
`177`	`177`	`}`
`178`	`178`	`}`
`179`	`179`
	`180`	`+ private void SetVisibility(object source, MouseEventArgs e)`
	`181`	`+ {`
	`182`	`+ _adornerControl.Visibility = AdornerVisible;`
	`183`	`+ }`
	`184`	`+`
`180`	`185`	`/// <summary>`
`181`	`186`	`/// ShowAdornerCommand`
`182`	`187`	`/// </summary>`