Warning cleanup

src/GitHub.Api/Application/ApiClient.cs

@@ -1,7 +1,6 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using System.Threading;
 using System.Threading.Tasks;
 using Octokit;
 
@@ -27,14 +26,10 @@ public static IApiClient Create(UriString repositoryUrl, IKeychain keychain)
         private readonly IKeychain keychain;
         private readonly IGitHubClient githubClient;
         private readonly ILoginManager loginManager;
-        private static readonly SemaphoreSlim sem = new SemaphoreSlim(1);
 
         IList<Organization> organizationsCache;
         Octokit.User userCache;
 
-        string owner;
-        bool? isEnterprise;
-
         public ApiClient(UriString hostUrl, IKeychain keychain, IGitHubClient githubClient)
         {
             Guard.ArgumentNotNull(hostUrl, nameof(hostUrl));

src/GitHub.Api/Application/ApplicationManagerBase.cs

@@ -39,7 +39,7 @@ protected void Initialize()
             Logging.TracingEnabled = UserSettings.Get(Constants.TraceLoggingKey, false);
             ProcessManager = new ProcessManager(Environment, Platform.GitEnvironment, CancellationToken);
             Platform.Initialize(ProcessManager, TaskManager);
-            GitClient = new GitClient(Environment, ProcessManager, Platform.CredentialManager, TaskManager);
+            GitClient = new GitClient(Environment, ProcessManager, TaskManager);
             SetupMetrics();
         }
 
@@ -127,7 +127,7 @@ public void RestartRepository()
         {
             if (Environment.RepositoryPath != null)
             {
-                repositoryManager = Unity.RepositoryManager.CreateInstance(Platform, TaskManager, UsageTracker, GitClient, Environment.RepositoryPath);
+                repositoryManager = Unity.RepositoryManager.CreateInstance(Platform, TaskManager, GitClient, Environment.RepositoryPath);
                 repositoryManager.Initialize();
                 Environment.Repository.Initialize(repositoryManager);
                 repositoryManager.Start();

src/GitHub.Api/Events/RepositoryWatcher.cs

@@ -30,10 +30,8 @@ class RepositoryWatcher : IRepositoryWatcher
         private readonly CancellationToken cancellationToken;
         private readonly NPath[] ignoredPaths;
         private readonly ManualResetEventSlim pauseEvent;
-        private readonly bool disableNative;
         private NativeInterface nativeInterface;
         private bool running;
-        private Task task;
         private int lastCountOfProcessedEvents = 0;
         private bool processingEvents;
         private readonly ManualResetEventSlim signalProcessingEventsDone = new ManualResetEventSlim(false);
@@ -68,8 +66,7 @@ public void Initialize()
 
             try
             {
-                if (!disableNative)
-                    nativeInterface = new NativeInterface(pathsRepositoryPath);
+                nativeInterface = new NativeInterface(pathsRepositoryPath);
             }
             catch (Exception ex)
             {
@@ -79,12 +76,6 @@ public void Initialize()
 
         public void Start()
         {
-            if (disableNative)
-            {
-                Logger.Trace("Native interface is disabled");
-                return;
-            }
-
             if (nativeInterface == null)
             {
                 Logger.Warning("NativeInterface is null");
@@ -95,7 +86,7 @@ public void Start()
 
             running = true;
             pauseEvent.Reset();
-            task = Task.Factory.StartNew(WatcherLoop, cancellationToken, TaskCreationOptions.None, TaskScheduler.Default);
+            Task.Factory.StartNew(WatcherLoop, cancellationToken, TaskCreationOptions.None, TaskScheduler.Default);
         }
 
         public void Stop()

src/GitHub.Api/Git/GitClient.cs

@@ -87,16 +87,13 @@ class GitClient : IGitClient
     {
         private readonly IEnvironment environment;
         private readonly IProcessManager processManager;
-        private readonly ICredentialManager credentialManager;
         private readonly ITaskManager taskManager;
         private readonly CancellationToken cancellationToken;
 
-        public GitClient(IEnvironment environment, IProcessManager processManager,
-            ICredentialManager credentialManager, ITaskManager taskManager)
+        public GitClient(IEnvironment environment, IProcessManager processManager, ITaskManager taskManager)
         {
             this.environment = environment;
             this.processManager = processManager;
-            this.credentialManager = credentialManager;
             this.taskManager = taskManager;
             this.cancellationToken = taskManager.Token;
         }

src/GitHub.Api/Git/GitConfig.cs

@@ -216,16 +216,16 @@ public string GetString(string key)
             public int GetInt(string key)
             {
                 var value = this[key];
-                var result = 0;
-                var success = int.TryParse(value, out result);
+                int result = 0;
+                int.TryParse(value, out result);
                 return result;
             }
 
             public float GetFloat(string key)
             {
                 var value = this[key];
-                var result = 0F;
-                var success = float.TryParse(value, out result);
+                float result = 0F;
+                float.TryParse(value, out result);
                 return result;
             }
 

src/GitHub.Api/Git/GitCredentialManager.cs

@@ -11,14 +11,12 @@ class GitCredentialManager : ICredentialManager
         private ICredential credential;
         private string credHelper = null;
 
-        private readonly IEnvironment environment;
         private readonly IProcessManager processManager;
         private readonly ITaskManager taskManager;
 
-        public GitCredentialManager(IEnvironment environment, IProcessManager processManager,
+        public GitCredentialManager(IProcessManager processManager,
             ITaskManager taskManager)
         {
-            this.environment = environment;
             this.processManager = processManager;
             this.taskManager = taskManager;
         }

src/GitHub.Api/Git/RepositoryManager.cs

@@ -1,7 +1,6 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using System.Threading;
 using System.Threading.Tasks;
 
 namespace GitHub.Unity
@@ -94,13 +93,11 @@ public RepositoryPathConfiguration(NPath repositoryPath)
 
     class RepositoryManager : IRepositoryManager
     {
-        private readonly CancellationToken cancellationToken;
         private readonly IGitConfig config;
         private readonly IGitClient gitClient;
         private readonly IPlatform platform;
         private readonly IRepositoryPathConfiguration repositoryPaths;
         private readonly ITaskManager taskManager;
-        private readonly IUsageTracker usageTracker;
         private readonly IRepositoryWatcher watcher;
 
         private bool isBusy;
@@ -119,23 +116,21 @@ class RepositoryManager : IRepositoryManager
         public event Action<string, string> OnRemoteBranchRemoved;
         public event Action<GitStatus> OnStatusUpdated;
 
-        public RepositoryManager(IPlatform platform, ITaskManager taskManager, IUsageTracker usageTracker, IGitConfig gitConfig,
+        public RepositoryManager(IPlatform platform, ITaskManager taskManager, IGitConfig gitConfig,
             IRepositoryWatcher repositoryWatcher, IGitClient gitClient,
-            IRepositoryPathConfiguration repositoryPaths, CancellationToken cancellationToken)
+            IRepositoryPathConfiguration repositoryPaths)
         {
             this.repositoryPaths = repositoryPaths;
             this.platform = platform;
             this.taskManager = taskManager;
-            this.usageTracker = usageTracker;
-            this.cancellationToken = cancellationToken;
             this.gitClient = gitClient;
             this.watcher = repositoryWatcher;
             this.config = gitConfig;
 
             SetupWatcher();
         }
 
-        public static RepositoryManager CreateInstance(IPlatform platform, ITaskManager taskManager, IUsageTracker usageTracker,
+        public static RepositoryManager CreateInstance(IPlatform platform, ITaskManager taskManager,
             IGitClient gitClient, NPath repositoryRoot)
         {
             var repositoryPathConfiguration = new RepositoryPathConfiguration(repositoryRoot);
@@ -144,8 +139,8 @@ public static RepositoryManager CreateInstance(IPlatform platform, ITaskManager
 
             var repositoryWatcher = new RepositoryWatcher(platform, repositoryPathConfiguration, taskManager.Token);
 
-            return new RepositoryManager(platform, taskManager, usageTracker, gitConfig, repositoryWatcher,
-                gitClient, repositoryPathConfiguration, taskManager.Token);
+            return new RepositoryManager(platform, taskManager, gitConfig, repositoryWatcher,
+                gitClient, repositoryPathConfiguration);
         }
 
         public void Initialize()

src/GitHub.Api/Helpers/SimpleJson.cs

@@ -517,7 +517,6 @@ static class SimpleJson
 
         private static readonly char[] EscapeTable;
         private static readonly char[] EscapeCharacters = new char[] { '"', '\\', '\b', '\f', '\n', '\r', '\t' };
-        private static readonly string EscapeCharactersString = new string(EscapeCharacters);
 
         static SimpleJson()
         {

src/GitHub.Api/NewTaskSystem/BaseOutputProcessor.cs

@@ -84,7 +84,6 @@ public override void LineReceived(string line)
 
     abstract class FirstResultOutputProcessor<T> : BaseOutputProcessor<T>
     {
-        private readonly StringBuilder sb = new StringBuilder();
         private bool isSet = false;
         public override void LineReceived(string line)
         {

src/GitHub.Api/Platform/Platform.cs

@@ -26,7 +26,7 @@ public IPlatform Initialize(IProcessManager processManager, ITaskManager taskMan
 
             if (CredentialManager == null)
             {
-                CredentialManager = new GitCredentialManager(Environment, processManager, taskManager);
+                CredentialManager = new GitCredentialManager(processManager, taskManager);
                 Keychain = new Keychain(Environment, CredentialManager);
                 Keychain.Initialize();
             }

src/UnityExtension/Assets/Editor/GitHub.Unity/ApplicationCache.cs

@@ -38,7 +38,6 @@ sealed class EnvironmentCache : ScriptObjectSingleton<EnvironmentCache>
         [SerializeField] private string unityApplication;
         [SerializeField] private string unityAssetsPath;
         [SerializeField] private string extensionInstallPath;
-        [SerializeField] private string gitExecutablePath;
         [SerializeField] private string unityVersion;
 
         [NonSerialized] private IEnvironment environment;
@@ -80,7 +79,6 @@ public void Flush()
             unityApplication = Environment.UnityApplication;
             unityAssetsPath = Environment.UnityAssetsPath;
             extensionInstallPath = Environment.ExtensionInstallPath;
-            gitExecutablePath = Environment.GitExecutablePath;
             Save(true);
         }
     }

src/UnityExtension/Assets/Editor/GitHub.Unity/Misc/Styles.cs

@@ -49,8 +49,6 @@ class Styles
 
         private const string WarningLabel = "<b>Warning:</b> {0}";
 
-        private static Color headerGreyColor = new Color(0.878f, 0.878f, 0.878f, 1.0f);
-
         private static GUIStyle label,
                                 boldLabel,
                                 errorLabel,
@@ -97,8 +95,6 @@ class Styles
                                  lockIcon,
                                  dropdownListIcon;
 
-        private static Color timelineBarColor;
-
         public static Texture2D GetFileStatusIcon(GitFileStatus status, bool isLocked)
         {
             if (isLocked)
@@ -597,18 +593,6 @@ public static GUIStyle GenericBoxStyle
             }
         }
 
-        public static Color TimelineBarColor
-        {
-            get
-            {
-                if (timelineBarColor == null)
-                {
-                    timelineBarColor = new Color(0.51F, 0.51F, 0.51F, 0.2F);
-                }
-                return timelineBarColor;
-            }
-        }
-
         public static Texture2D ActiveBranchIcon
         {
             get

src/UnityExtension/Assets/Editor/GitHub.Unity/Threading/SingleThreadSynchronizationContext.cs

@@ -1,4 +1,3 @@
-using GitHub.Unity;
 using System;
 using System.Threading;
 using UnityEditor;
@@ -7,8 +6,6 @@ namespace GitHub.Unity
 {
     class MainThreadSynchronizationContext : SynchronizationContext, IMainThreadSynchronizationContext
     {
-        private static readonly ILogging logger = Logging.GetLogger<MainThreadSynchronizationContext>();
-
         public void Schedule(Action action)
         {
             Guard.ArgumentNotNull(action, "action");

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/BaseWindow.cs

@@ -7,7 +7,6 @@ namespace GitHub.Unity
     abstract class BaseWindow :  EditorWindow, IView
     {
         [NonSerialized] private bool initialized = false;
-        [NonSerialized] private IApplicationManager cachedManager;
         [NonSerialized] private IRepository cachedRepository;
         [NonSerialized] private bool initializeWasCalled;
         [NonSerialized] private bool inLayout;

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/HistoryView.cs

@@ -35,7 +35,6 @@ class HistoryView : Subview
 
         [NonSerialized] private int historyStartIndex;
         [NonSerialized] private int historyStopIndex;
-        [NonSerialized] private float lastWidth;
         [NonSerialized] private int listID;
         [NonSerialized] private int newSelectionIndex;
         [NonSerialized] private float scrollOffset;
@@ -60,7 +59,6 @@ public override void InitializeView(IView parent)
         {
             base.InitializeView(parent);
 
-            lastWidth = Position.width;
             selectionIndex = newSelectionIndex = -1;
 
             changesetTree.InitializeView(this);
@@ -453,15 +451,13 @@ private void RevertCommit()
         private bool HistoryEntry(GitLogEntry entry, LogEntryState state, bool selected)
         {
             var entryRect = GUILayoutUtility.GetRect(Styles.HistoryEntryHeight, Styles.HistoryEntryHeight);
-            var timelineBarRect = new Rect(entryRect.x + Styles.BaseSpacing, 0, 2, Styles.HistoryDetailsHeight);
 
             if (Event.current.type == EventType.Repaint)
             {
                 var keyboardFocus = GUIUtility.keyboardControl == listID;
 
                 var summaryRect = new Rect(entryRect.x, entryRect.y + (Styles.BaseSpacing / 2), entryRect.width, Styles.HistorySummaryHeight + Styles.BaseSpacing);
                 var timestampRect = new Rect(entryRect.x, entryRect.yMax - Styles.HistoryDetailsHeight - (Styles.BaseSpacing / 2), entryRect.width, Styles.HistoryDetailsHeight);
-                var authorRect = new Rect(timestampRect.xMax, timestampRect.y, timestampRect.width, timestampRect.height);
 
                 var contentOffset = new Vector2(Styles.BaseSpacing * 2, 0);
 
@@ -610,7 +606,6 @@ private void Push()
 
         private void Fetch()
         {
-            var remote = Repository.CurrentRemote.HasValue ? Repository.CurrentRemote.Value.Name : String.Empty;
             Repository
                 .Fetch()
                 .FinallyInUI((success, e) => {

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/SettingsView.cs

@@ -11,7 +11,6 @@ namespace GitHub.Unity
     [Serializable]
     class SettingsView : Subview
     {
-        private const string GitInstallTitle = "Git installation";
         private const string GitRepositoryTitle = "Repository Configuration";
         private const string GitRepositoryRemoteLabel = "Remote";
         private const string GitRepositorySave = "Save Repository";
@@ -20,15 +19,9 @@ class SettingsView : Subview
         private const string EnableTraceLoggingLabel = "Enable Trace Logging";
         private const string MetricsOptInLabel = "Help us improve by sending anonymous usage data";
         private const string DefaultRepositoryRemoteName = "origin";
-        private const string BrowseButton = "...";
-        private const string PathToGit = "Path to Git";
-        private const string GitPathSaveButton = "Save Path";
 
-        [NonSerialized] private int newGitIgnoreRulesSelection = -1;
         [NonSerialized] private bool isBusy;
 
-        [SerializeField] private int gitIgnoreRulesSelection = 0;
-        [SerializeField] private string initDirectory;
         [SerializeField] private List<GitLock> lockedFiles = new List<GitLock>();
         [SerializeField] private Vector2 lockScrollPos;
         [SerializeField] private string repositoryRemoteName;
@@ -39,16 +32,12 @@ class SettingsView : Subview
         [NonSerialized] private bool remoteHasChanged;
         [NonSerialized] private bool locksHaveChanged;
 
-        [SerializeField] private string newGitName;
-        [SerializeField] private string newGitEmail;
         [SerializeField] private string newRepositoryRemoteUrl;
-        [SerializeField] private User cachedUser;
 
         [SerializeField] private bool metricsEnabled;
         [NonSerialized] private bool metricsHasChanged;
 
         [SerializeField] private GitPathView gitPathView = new GitPathView();
-
         [SerializeField] private UserSettingsView userSettingsView = new UserSettingsView();
 
         public override void InitializeView(IView parent)

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/Window.cs

@@ -283,7 +283,7 @@ private void DoHeaderGUI()
         private void DoToolbarGUI()
         {
             // Subtabs & toolbar
-            Rect mainNavRect = EditorGUILayout.BeginHorizontal(EditorStyles.toolbar);
+            GUILayout.BeginHorizontal(EditorStyles.toolbar);
             {
                 changeTab = activeTab;
                 EditorGUI.BeginChangeCheck();