Skip to content

Commit f63c9fa

Browse files
authored
Merge pull request #15953 from tamasvajk/buildless/impr2
C#: Iterate text files only once
2 parents d2bb951 + 881c426 commit f63c9fa

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/DependencyManager.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ public DependencyManager(string srcDir, ILogger logger)
6565
var allFiles = GetAllFiles().ToList();
6666
var binaryFileExtensions = new HashSet<string>(new[] { ".dll", ".exe" }); // TODO: add more binary file extensions.
6767
var allNonBinaryFiles = allFiles.Where(f => !binaryFileExtensions.Contains(f.Extension.ToLowerInvariant())).ToList();
68-
var smallNonBinaryFiles = allNonBinaryFiles.SelectSmallFiles(logger).SelectFileNames();
68+
var smallNonBinaryFiles = allNonBinaryFiles.SelectSmallFiles(logger).SelectFileNames().ToList();
6969
this.fileContent = new FileContent(logger, smallNonBinaryFiles);
7070
this.nonGeneratedSources = allNonBinaryFiles.SelectFileNamesByExtension(".cs").ToList();
7171
this.generatedSources = new();

0 commit comments

Comments
 (0)