From 97b77ccd67ba27576664a168b9e7cf8d53b39ecd Mon Sep 17 00:00:00 2001
From: Stanley Goldman <Stanley.Goldman@gmail.com>
Date: Thu, 13 Jul 2017 12:05:03 -0400
Subject: [PATCH] Remove stateChangeCallback because it is never used

---
 .../GitHub.Unity/UI/ChangesetTreeView.cs      | 36 +++----------------
 1 file changed, 4 insertions(+), 32 deletions(-)

diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesetTreeView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesetTreeView.cs
index 859860037..f271039ef 100644
--- a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesetTreeView.cs
+++ b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesetTreeView.cs
@@ -19,7 +19,6 @@ class ChangesetTreeView : Subview
         [SerializeField] private List<GitCommitTarget> entryCommitTargets = new List<GitCommitTarget>();
         [SerializeField] private List<string> foldedTreeEntries = new List<string>();
         [NonSerialized] private FileTreeNode tree;
-        [NonSerialized] private Action<FileTreeNode> stateChangeCallback;
 
         public override void OnGUI()
         {
@@ -153,7 +152,7 @@ public void UpdateEntries(IList<GitStatusEntry> newEntries)
 
             // Build tree structure
 
-            tree = new FileTreeNode(FileSystemHelpers.FindCommonPath(entries.Select(e => e.Path)), stateChangeCallback);
+            tree = new FileTreeNode(FileSystemHelpers.FindCommonPath(entries.Select(e => e.Path)));
             tree.RepositoryPath = tree.Path;
             for (var index = 0; index < entries.Count; index++)
             {
@@ -161,7 +160,7 @@ public void UpdateEntries(IList<GitStatusEntry> newEntries)
                 if (entryPath.IsChildOf(tree.Path))
                     entryPath = entryPath.RelativeTo(tree.Path.ToNPath());
 
-                var node = new FileTreeNode(entryPath, stateChangeCallback) { Target = entryCommitTargets[index] };
+                var node = new FileTreeNode(entryPath) { Target = entryCommitTargets[index] };
                 if (!string.IsNullOrEmpty(entries[index].ProjectPath))
                 {
                     node.Icon = AssetDatabase.GetCachedIcon(entries[index].ProjectPath);
@@ -209,7 +208,7 @@ private void BuildTree(FileTreeNode parent, FileTreeNode node)
                 if (!found)
                 {
                     var p = parent.RepositoryPath.ToNPath().Combine(root);
-                    BuildTree(parent.Add(new FileTreeNode(root, stateChangeCallback) { RepositoryPath = p }), node);
+                    BuildTree(parent.Add(new FileTreeNode(root) { RepositoryPath = p }), node);
                 }
             }
             else if (nodePath.ExtensionWithDot == ".meta")
@@ -259,25 +258,7 @@ private void TreeNode(FileTreeNode node)
                     }
                     if (EditorGUI.EndChangeCheck())
                     {
-                        var filesAdded = new List<string>();
-                        var filesRemoved = new List<string>();
-                        stateChangeCallback = new Action<FileTreeNode>(s =>
-                        {
-                            if (s.State == CommitState.None)
-                                filesRemoved.Add(s.Path);
-                            else
-                                filesAdded.Add(s.Path);
-                        });
                         node.State = toggled ? CommitState.All : CommitState.None;
-                        if (filesAdded.Count > 0)
-                            GitClient.Add(filesAdded);
-                        if (filesRemoved.Count > 0)
-                            GitClient.Remove(filesAdded);
-                        if (filesAdded.Count > 0|| filesRemoved.Count > 0)
-                        {
-                            GitClient.Status();
-                        }
-                        // we might need to run git status after these calls
                     }
                 }
 
@@ -421,7 +402,6 @@ private enum CommitState
 
         private class FileTreeNode
         {
-            private readonly Action<FileTreeNode> stateChangeCallback;
             private List<FileTreeNode> children;
             private string path;
             private CommitState state;
@@ -432,15 +412,8 @@ private class FileTreeNode
             public string RepositoryPath;
             public GitCommitTarget Target { get; set; }
 
-            public FileTreeNode(Action<FileTreeNode> stateChangeCallback)
-            {
-                this.stateChangeCallback = stateChangeCallback;
-                children = new List<FileTreeNode>();
-            }
-
-            public FileTreeNode(string path, Action<FileTreeNode> stateChangeCallback)
+            public FileTreeNode(string path)
             {
-                this.stateChangeCallback = stateChangeCallback;
                 this.path = path ?? "";
                 Label = this.path;
                 children = new List<FileTreeNode>();
@@ -514,7 +487,6 @@ public CommitState State
                     }
 
                     state = value;
-                    stateChangeCallback.SafeInvoke(this);
 
                     if (children == null)
                     {