remove value tuple (#4950)

* remove value tuple * typo * fix build
Particular · Sep 15, 2017 · 00fe48d · 00fe48d
1 parent 6a0c43c
commit 00fe48d
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 12 deletions.
diff --git a/src/NServiceBus.Core.Tests/AssemblyScanner/When_validating_assemblies.cs b/src/NServiceBus.Core.Tests/AssemblyScanner/When_validating_assemblies.cs
@@ -15,7 +15,7 @@ public void Should_not_validate_system_assemblies()
 
             foreach (var assembly in systemAssemblies)
             {
-                var (shouldLoad, reason) = validator.ValidateAssemblyFile(assembly.Location);
+                validator.ValidateAssemblyFile(assembly.Location, out var shouldLoad, out var reason);
 
                 Assert.IsFalse(shouldLoad, $"Should not validate {assembly.FullName}");
                 Assert.That(reason == "File is a .NET runtime assembly.");
@@ -27,7 +27,7 @@ public void Should_validate_NServiceBus_Core_assembly()
         {
             var validator = new AssemblyValidator();
 
-            var (shouldLoad, reason) = validator.ValidateAssemblyFile(typeof(EndpointConfiguration).Assembly.Location);
+            validator.ValidateAssemblyFile(typeof(EndpointConfiguration).Assembly.Location, out var shouldLoad, out var reason);
 
             Assert.IsTrue(shouldLoad);
             Assert.That(reason == "File is a .NET assembly.");
@@ -38,7 +38,7 @@ public void Should_validate_non_system_assemblies()
         {
             var validator = new AssemblyValidator();
 
-            var (shouldLoad, reason) = validator.ValidateAssemblyFile(GetType().Assembly.Location);
+            validator.ValidateAssemblyFile(GetType().Assembly.Location, out var shouldLoad, out var reason);
 
             Assert.IsTrue(shouldLoad);
             Assert.That(reason == "File is a .NET assembly.");

diff --git a/src/NServiceBus.Core/Hosting/Helpers/AssemblyScanner.cs b/src/NServiceBus.Core/Hosting/Helpers/AssemblyScanner.cs
@@ -140,7 +140,7 @@ bool TryLoadScannableAssembly(string assemblyPath, AssemblyScannerResults result
                 return false;
             }
 
-            var (shouldLoad, reason) = assemblyValidator.ValidateAssemblyFile(assemblyPath);
+            assemblyValidator.ValidateAssemblyFile(assemblyPath, out var shouldLoad, out var reason);
 
             if (!shouldLoad)
             {

diff --git a/src/NServiceBus.Core/Hosting/Helpers/AssemblyValidator.cs b/src/NServiceBus.Core/Hosting/Helpers/AssemblyValidator.cs
@@ -5,23 +5,28 @@
 
     class AssemblyValidator
     {
-        public (bool shouldLoad, string reason) ValidateAssemblyFile(string assemblyPath)
+        public void ValidateAssemblyFile(string assemblyPath, out bool shouldLoad, out string reason)
         {
             try
             {
                 var token = AssemblyName.GetAssemblyName(assemblyPath).GetPublicKeyToken();
 
                 if (IsRuntimeAssembly(token))
                 {
-                    return (false, "File is a .NET runtime assembly.");
+                    shouldLoad = false;
+                    reason = "File is a .NET runtime assembly.";
+                    return;
                 }
             }
             catch (BadImageFormatException)
             {
-                return (false, "File is not a .NET assembly.");
+                shouldLoad = false;
+                reason = "File is not a .NET assembly.";
+                return;
             }
 
-            return (true, "File is a .NET assembly.");
+            shouldLoad = true;
+            reason = "File is a .NET assembly.";
         }
 
         public static bool IsRuntimeAssembly(byte[] publicKeyToken)

diff --git a/src/NServiceBus.Core/NServiceBus.Core.csproj b/src/NServiceBus.Core/NServiceBus.Core.csproj
@@ -40,10 +40,6 @@
     <DotNetCliToolReference Include="dotnet-sourcelink-git" Version="2.1.2" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' == 'net452'">
-    <PackageReference Include="System.ValueTuple" Version="4.4.0" />
-  </ItemGroup>
-
   <ItemGroup Condition="'$(TargetFramework)' == 'netstandard2.0'">
     <PackageReference Include="System.Reflection.Emit" Version="4.3.0" />
     <PackageReference Include="System.Reflection.Emit.Lightweight" Version="4.3.0" />