From 86d032e2ea8021ab7af9867d24bd135cba2d057c Mon Sep 17 00:00:00 2001
From: Calum Grant <calumgrant@github.com>
Date: Wed, 29 Apr 2020 14:51:55 +0100
Subject: [PATCH] C#: Remove nullability warning by lifting local function.
 Warning seems to be caused by a compiler bug.

---
 .../Semmle.Autobuild/Autobuilder.cs           | 36 +++++++++----------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/csharp/autobuilder/Semmle.Autobuild/Autobuilder.cs b/csharp/autobuilder/Semmle.Autobuild/Autobuilder.cs
index f6dbdae788c..a882f5bb80c 100644
--- a/csharp/autobuilder/Semmle.Autobuild/Autobuilder.cs
+++ b/csharp/autobuilder/Semmle.Autobuild/Autobuilder.cs
@@ -118,6 +118,24 @@ namespace Semmle.Autobuild
         /// </summary>
         public IBuildActions Actions { get; }
 
+        IEnumerable<IProjectOrSolution>? FindFiles(string extension, Func<string, ProjectOrSolution> create)
+        {
+            var matchingFiles = GetExtensions(extension).
+                Select(p => (ProjectOrSolution: create(p.Item1), DistanceFromRoot: p.Item2)).
+                Where(p => p.ProjectOrSolution.HasLanguage(this.Options.Language)).
+                ToArray();
+
+            if (matchingFiles.Length == 0)
+               return null;
+
+            if (Options.AllSolutions)
+               return matchingFiles.Select(p => p.ProjectOrSolution);
+
+            return matchingFiles.
+                Where(f => f.DistanceFromRoot == matchingFiles[0].DistanceFromRoot).
+                Select(f => f.ProjectOrSolution);
+        }
+
         /// <summary>
         /// Find all the relevant files and picks the best
         /// solution file and tools.
@@ -151,24 +169,6 @@ namespace Semmle.Autobuild
                     return ret;
                 }
 
-                IEnumerable<IProjectOrSolution>? FindFiles(string extension, Func<string, ProjectOrSolution> create)
-                {
-                    var matchingFiles = GetExtensions(extension).
-                        Select(p => (ProjectOrSolution: create(p.Item1), DistanceFromRoot: p.Item2)).
-                        Where(p => p.ProjectOrSolution.HasLanguage(this.Options.Language)).
-                        ToArray();
-
-                    if (matchingFiles.Length == 0)
-                        return null;
-
-                    if (options.AllSolutions)
-                        return matchingFiles.Select(p => p.ProjectOrSolution);
-
-                    return matchingFiles.
-                        Where(f => f.DistanceFromRoot == matchingFiles[0].DistanceFromRoot).
-                        Select(f => f.ProjectOrSolution);
-                }
-
                 // First look for `.proj` files
                 ret = FindFiles(".proj", f => new Project(this, f))?.ToList();
                 if (ret != null)