Skip to content

Commit 1a5d462

Browse files
authored
Merge pull request #435 from calumgrant/cs/fix-extractor
C#: Fix an ArgumentNullException.
2 parents 6fdb0aa + 789e2fc commit 1a5d462

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

csharp/extractor/Semmle.Extraction.CSharp/Analyser.cs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -413,13 +413,15 @@ public void LogDiagnostics(string[] roslynArgs)
413413
Logger.Log(Severity.Info, " Current working directory: {0}", Directory.GetCurrentDirectory());
414414

415415
if (roslynArgs != null)
416+
{
416417
Logger.Log(Severity.Info, $" Arguments to Roslyn: {string.Join(' ', roslynArgs)}");
417418

418-
// Create a new file in the log folder.
419-
var argsFile = Path.Combine(Extractor.GetCSharpLogDirectory(), $"csharp.{Path.GetRandomFileName()}.txt");
419+
// Create a new file in the log folder.
420+
var argsFile = Path.Combine(Extractor.GetCSharpLogDirectory(), $"csharp.{Path.GetRandomFileName()}.txt");
420421

421-
if (roslynArgs.ArchiveCommandLine(argsFile))
422-
Logger.Log(Severity.Info, $" Arguments have been written to {argsFile}");
422+
if (roslynArgs.ArchiveCommandLine(argsFile))
423+
Logger.Log(Severity.Info, $" Arguments have been written to {argsFile}");
424+
}
423425

424426
foreach (var error in FilteredDiagnostics)
425427
{

0 commit comments

Comments
 (0)