From 0e91a428e8dde846ad23b69c065e9bfb49d4dc32 Mon Sep 17 00:00:00 2001
From: xtqqczze <xtqqczze@gmail.com>
Date: Mon, 30 Nov 2020 13:02:45 +0000
Subject: [PATCH] Replace GetFiles in TestModuleManifestCommand

* Replace GetFiles with EnumerateFiles to avoid large array allocation.
* Simplify if-return
---
 .../engine/Modules/TestModuleManifestCommand.cs    | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

diff --git a/src/System.Management.Automation/engine/Modules/TestModuleManifestCommand.cs b/src/System.Management.Automation/engine/Modules/TestModuleManifestCommand.cs
index b31ec151af5..265287a010f 100644
--- a/src/System.Management.Automation/engine/Modules/TestModuleManifestCommand.cs
+++ b/src/System.Management.Automation/engine/Modules/TestModuleManifestCommand.cs
@@ -420,23 +420,13 @@ private static bool IsValidGacAssembly(string assemblyName)
 
             try
             {
-                var allFiles = Directory.GetFiles(gacPath, assemblyFile, SearchOption.AllDirectories);
-
-                if (allFiles.Length == 0)
-                {
-                    var allNgenFiles = Directory.GetFiles(gacPath, ngenAssemblyFile, SearchOption.AllDirectories);
-                    if (allNgenFiles.Length == 0)
-                    {
-                        return false;
-                    }
-                }
+                return Directory.EnumerateFiles(gacPath, assemblyFile, SearchOption.AllDirectories).Any()
+                    || Directory.EnumerateFiles(gacPath, ngenAssemblyFile, SearchOption.AllDirectories).Any();
             }
             catch
             {
                 return false;
             }
-
-            return true;
 #endif
         }
     }