refactor: remove redundant field initialization

Author: NathanBaulch

src/App.Commands.cs

@@ -22,7 +22,7 @@ public Command(Action<object> action)
             public bool CanExecute(object parameter) => _action != null;
             public void Execute(object parameter) => _action?.Invoke(parameter);
 
-            private Action<object> _action = null;
+            private readonly Action<object> _action;
         }
 
         public static bool IsCheckForUpdateCommandVisible

src/App.axaml.cs

@@ -703,10 +703,10 @@ private string FixFontFamilyName(string input)
         [GeneratedRegex(@"^[a-z]+\s+([a-fA-F0-9]{4,40})(\s+.*)?$")]
         private static partial Regex REG_REBASE_TODO();
 
-        private Models.IpcChannel _ipcChannel = null;
-        private ViewModels.Launcher _launcher = null;
-        private ResourceDictionary _activeLocale = null;
-        private ResourceDictionary _themeOverrides = null;
-        private ResourceDictionary _fontsOverrides = null;
+        private Models.IpcChannel _ipcChannel;
+        private ViewModels.Launcher _launcher;
+        private ResourceDictionary _activeLocale;
+        private ResourceDictionary _themeOverrides;
+        private ResourceDictionary _fontsOverrides;
     }
 }

src/Commands/Blame.cs

@@ -90,7 +90,7 @@ private void ParseLine(string line)
         private readonly StringBuilder _content = new StringBuilder();
         private readonly string _dateFormat = Models.DateTimeFormat.Active.DateOnly;
         private string _lastSHA = string.Empty;
-        private bool _needUnifyCommitSHA = false;
+        private bool _needUnifyCommitSHA;
         private int _minSHALen = 64;
     }
 }

src/Commands/Command.cs

@@ -12,7 +12,7 @@ public partial class Command
     {
         public class Result
         {
-            public bool IsSuccess { get; set; } = false;
+            public bool IsSuccess { get; set; }
             public string StdOut { get; set; } = string.Empty;
             public string StdErr { get; set; } = string.Empty;
 
@@ -27,15 +27,15 @@ public enum EditorType
         }
 
         public string Context { get; set; } = string.Empty;
-        public string WorkingDirectory { get; set; } = null;
+        public string WorkingDirectory { get; set; }
         public EditorType Editor { get; set; } = EditorType.CoreEditor;
         public string SSHKey { get; set; } = string.Empty;
         public string Args { get; set; } = string.Empty;
 
         // Only used in `ExecAsync` mode.
         public CancellationToken CancellationToken { get; set; } = CancellationToken.None;
         public bool RaiseError { get; set; } = true;
-        public Models.ICommandLog Log { get; set; } = null;
+        public Models.ICommandLog Log { get; set; }
 
         public void Exec()
         {

src/Commands/Config.cs

@@ -60,6 +60,6 @@ public async Task<bool> SetAsync(string key, string value, bool allowEmpty = fal
             return await ExecAsync().ConfigureAwait(false);
         }
 
-        private bool _isLocal = false;
+        private readonly bool _isLocal;
     }
 }

src/Commands/Diff.cs

@@ -262,8 +262,8 @@ private void ProcessInlineHighlights()
         private readonly Models.DiffResult _result = new Models.DiffResult();
         private readonly List<Models.TextDiffLine> _deleted = new List<Models.TextDiffLine>();
         private readonly List<Models.TextDiffLine> _added = new List<Models.TextDiffLine>();
-        private Models.TextDiffLine _last = null;
-        private int _oldLine = 0;
-        private int _newLine = 0;
+        private Models.TextDiffLine _last;
+        private int _oldLine;
+        private int _newLine;
     }
 }

src/Commands/QueryCommits.cs

@@ -142,9 +142,9 @@ private async Task MarkFirstMergedAsync()
             }
         }
 
-        private List<Models.Commit> _commits = new List<Models.Commit>();
-        private Models.Commit _current = null;
-        private bool _findFirstMerged = false;
-        private bool _isHeadFound = false;
+        private readonly List<Models.Commit> _commits = new List<Models.Commit>();
+        private readonly bool _findFirstMerged;
+        private Models.Commit _current;
+        private bool _isHeadFound;
     }
 }

src/Commands/QueryCommitsForInteractiveRebase.cs

@@ -89,8 +89,8 @@ private void ParseParent(string data)
             _current.Commit.Parents.AddRange(data.Split(' ', StringSplitOptions.RemoveEmptyEntries));
         }
 
-        private List<Models.InteractiveCommit> _commits = [];
-        private Models.InteractiveCommit _current = null;
+        private readonly List<Models.InteractiveCommit> _commits = [];
         private readonly string _boundary;
+        private Models.InteractiveCommit _current;
     }
 }

src/Models/AvatarManager.cs

@@ -30,17 +30,17 @@ public static AvatarManager Instance
             }
         }
 
-        private static AvatarManager _instance = null;
+        private static AvatarManager _instance;
 
         [GeneratedRegex(@"^(?:(\d+)\+)?(.+?)@.+\.github\.com$")]
         private static partial Regex REG_GITHUB_USER_EMAIL();
 
         private readonly Lock _synclock = new();
+        private readonly List<IAvatarHost> _avatars = new List<IAvatarHost>();
+        private readonly Dictionary<string, Bitmap> _resources = new Dictionary<string, Bitmap>();
+        private readonly HashSet<string> _requesting = new HashSet<string>();
+        private readonly HashSet<string> _defaultAvatars = new HashSet<string>();
         private string _storePath;
-        private List<IAvatarHost> _avatars = new List<IAvatarHost>();
-        private Dictionary<string, Bitmap> _resources = new Dictionary<string, Bitmap>();
-        private HashSet<string> _requesting = new HashSet<string>();
-        private HashSet<string> _defaultAvatars = new HashSet<string>();
 
         public void Start()
         {

src/Models/Blame.cs

@@ -4,7 +4,7 @@ namespace SourceGit.Models
 {
     public class BlameLineInfo
     {
-        public bool IsFirstInGroup { get; set; } = false;
+        public bool IsFirstInGroup { get; set; }
         public string CommitSHA { get; set; } = string.Empty;
         public string Author { get; set; } = string.Empty;
         public string Time { get; set; } = string.Empty;
@@ -15,6 +15,6 @@ public class BlameData
         public string File { get; set; } = string.Empty;
         public List<BlameLineInfo> LineInfos { get; set; } = new List<BlameLineInfo>();
         public string Content { get; set; } = string.Empty;
-        public bool IsBinary { get; set; } = false;
+        public bool IsBinary { get; set; }
     }
 }