Merge pull request #24 from CandyCoded/hotfix/fixed-possible-null

neogeek · web-flow · commit 80675442715f · 2020-01-08T03:06:22.000-05:00
[hotfix] Fixed possible null warning.
diff --git a/Assets/Plugins/CandyCoded.GitStatus/Scripts/CustomEditor/Git.cs b/Assets/Plugins/CandyCoded.GitStatus/Scripts/CustomEditor/Git.cs
@@ -140,7 +140,7 @@ public static void DiscardChanges(string path)
                 CreateNoWindow = true
             });
 
-            if (process.StandardError.ReadLine().StartsWith("error: pathspec"))
+            if (process?.StandardError.ReadLine() is string line && line.StartsWith("error: pathspec"))
             {
 
                 Debug.LogError("File not tracked by git.");

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ public static void DiscardChanges(string path)`
`140`	`140`	`CreateNoWindow = true`
`141`	`141`	`});`
`142`	`142`
`143`		`- if (process.StandardError.ReadLine().StartsWith("error: pathspec"))`
	`143`	`+ if (process?.StandardError.ReadLine() is string line && line.StartsWith("error: pathspec"))`
`144`	`144`	`{`
`145`	`145`
`146`	`146`	`Debug.LogError("File not tracked by git.");`