C#: Address review comments

hvitved · hvitved · commit a9f08671cabd · 2019-11-14T11:00:19.000+01:00
diff --git a/csharp/autobuilder/Semmle.Autobuild/Autobuilder.cs b/csharp/autobuilder/Semmle.Autobuild/Autobuilder.cs
@@ -216,13 +216,12 @@ public int AttemptBuild()
 
             void startCallback(string s, bool silent)
             {
-                if (!silent) Log(Severity.Info, $"\nRunning {s}");
+                Log(silent ? Severity.Debug : Severity.Info, $"\nRunning {s}");
             }
 
             void exitCallback(int ret, string msg, bool silent)
             {
-                if (!silent)
-                    Log(Severity.Info, $"Exit code {ret}{(string.IsNullOrEmpty(msg) ? "" : $": {msg}")}");
+                Log(silent ? Severity.Debug : Severity.Info, $"Exit code {ret}{(string.IsNullOrEmpty(msg) ? "" : $": {msg}")}");
             }
 
             return script.Run(Actions, startCallback, exitCallback);

Original file line number	Diff line number	Diff line change
`@@ -216,13 +216,12 @@ public int AttemptBuild()`
`216`	`216`
`217`	`217`	`void startCallback(string s, bool silent)`
`218`	`218`	`{`
`219`		`- if (!silent) Log(Severity.Info, $"\nRunning {s}");`
	`219`	`+ Log(silent ? Severity.Debug : Severity.Info, $"\nRunning {s}");`
`220`	`220`	`}`
`221`	`221`
`222`	`222`	`void exitCallback(int ret, string msg, bool silent)`
`223`	`223`	`{`
`224`		`- if (!silent)`
`225`		`- Log(Severity.Info, $"Exit code {ret}{(string.IsNullOrEmpty(msg) ? "" : $": {msg}")}");`
	`224`	`+ Log(silent ? Severity.Debug : Severity.Info, $"Exit code {ret}{(string.IsNullOrEmpty(msg) ? "" : $": {msg}")}");`
`226`	`225`	`}`
`227`	`226`
`228`	`227`	`return script.Run(Actions, startCallback, exitCallback);`