#515 Optimise dependencies to remove polyfil (#516)

thompson-tomo · ardalis · web-flow · commit c3f8e569bf3a · 2024-04-28T11:22:55.000-04:00
* #515 make STJ only included for net standard * Moved to global package refs & add suppressions * Make System.ComponentModel.Annotations oonly for net std * Styling fixes. --------- Co-authored-by: Steve Smith <steve@kentsmiths.com>
diff --git a/Directory.Packages.props b/Directory.Packages.props
@@ -18,15 +18,18 @@
     <PackageVersion Include="Microsoft.EntityFrameworkCore.Design" Version="7.0.13" />
     <PackageVersion Include="Microsoft.EntityFrameworkCore.Sqlite" Version="7.0.13" />
     <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-    <PackageVersion Include="Microsoft.SourceLink.GitHub" Version="8.0.0" />
     <PackageVersion Include="Moq" Version="4.18.4" />
     <PackageVersion Include="Newtonsoft.Json" Version="13.0.2" />
     <PackageVersion Include="protobuf-net" Version="3.2.26" />
-    <PackageVersion Include="SonarAnalyzer.CSharp" Version="9.17.0.82934" />
     <PackageVersion Include="System.ComponentModel.Annotations" Version="5.0.0" />
     <PackageVersion Include="System.Text.Json" Version="8.0.2" />
     <PackageVersion Include="Utf8Json" Version="1.3.7" />
     <PackageVersion Include="xunit" Version="2.7.0" />
     <PackageVersion Include="xunit.runner.visualstudio" Version="2.4.5" />
   </ItemGroup>
+
+  <ItemGroup>
+    <GlobalPackageReference Include="Microsoft.SourceLink.GitHub" Version="8.0.0" PrivateAssets="All" />
+    <GlobalPackageReference  Include="SonarAnalyzer.CSharp" Version="9.17.0.82934" PrivateAssets="All" />
+  </ItemGroup>
 </Project>
diff --git a/benchmarks/SmartEnum.Benchmarks/EqualsBenchmarks.cs b/benchmarks/SmartEnum.Benchmarks/EqualsBenchmarks.cs
@@ -1,8 +1,10 @@
 namespace Ardalis.SmartEnum.Benchmarks
 {
     using BenchmarkDotNet.Attributes;
+    using System.Diagnostics.CodeAnalysis;
 
     [Config(typeof(Config))]
+    [SuppressMessage("Major Bug", "S1764:Identical expressions should not be used on both sides of operators", Justification = "<Pending>")]
     public class EqualsBenchmarks
     {
         ////////////////////////////////////////////////////////////////////////////////
diff --git a/benchmarks/SmartEnum.Benchmarks/SerializationBenchmarks.cs b/benchmarks/SmartEnum.Benchmarks/SerializationBenchmarks.cs
@@ -1,5 +1,6 @@
 namespace Ardalis.SmartEnum.Benchmarks
 {
+    using System.Diagnostics.CodeAnalysis;
     using System.IO;
     using BenchmarkDotNet.Attributes;
 
@@ -68,6 +69,7 @@ public sealed class TestSmartEnumValueClass
         static Stream valueProtoBufDeserializeStream;
 
         [GlobalSetup]
+        [SuppressMessage("Critical Code Smell", "S2696:Instance members should not write to \"static\" fields", Justification = "<Pending>")]
         public void GlobalSetup()
         {
             global::Utf8Json.Resolvers.CompositeResolver.Register(
diff --git a/benchmarks/SmartEnum.Benchmarks/SwitchBenchmarks.cs b/benchmarks/SmartEnum.Benchmarks/SwitchBenchmarks.cs
@@ -1,9 +1,12 @@
 namespace Ardalis.SmartEnum.Benchmarks
 {
     using System;
+    using System.Diagnostics.CodeAnalysis;
     using BenchmarkDotNet.Attributes;
 
     [Config(typeof(Config))]
+    [SuppressMessage("Major Bug", "S2583:Conditionally executed code should be reachable", Justification = "<Pending>")]
+    [SuppressMessage("Major Code Smell", "S112:General or reserved exceptions should never be thrown", Justification = "<Pending>")]
     public class SwitchBenchmarks
     {
         ////////////////////////////////////////////////////////////////////////////////
diff --git a/benchmarks/SmartEnum.Benchmarks/TestEnums.cs b/benchmarks/SmartEnum.Benchmarks/TestEnums.cs
@@ -1,9 +1,12 @@
+using System.Diagnostics.CodeAnalysis;
+
 namespace Ardalis.SmartEnum.Benchmarks
 {
     ////////////////////////////////////////////////////////////////////////////////
     // Enum
 
     [global::ProtoBuf.ProtoContract]
+    [SuppressMessage("Minor Code Smell", "S2344:Enumeration type names should not have \"Flags\" or \"Enum\" suffixes", Justification = "<Pending>")]
     public enum TestEnum
     {
         [global::ProtoBuf.ProtoEnum]
diff --git a/src/SmartEnum.AutoFixture/SmartEnum.AutoFixture.csproj b/src/SmartEnum.AutoFixture/SmartEnum.AutoFixture.csproj
@@ -16,8 +16,6 @@
 	</PropertyGroup>
 	<ItemGroup>
 		<PackageReference Include="AutoFixture" />
-		<PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All" />
-		<PackageReference Include="SonarAnalyzer.CSharp" PrivateAssets="All" />
 	</ItemGroup>
 	<ItemGroup>
 		<None Include="$(SolutionDir)img\icon.png" Pack="true" Visible="false" PackagePath="" />
diff --git a/src/SmartEnum.Dapper/DapperSmartEnum.cs b/src/SmartEnum.Dapper/DapperSmartEnum.cs
@@ -1,6 +1,5 @@
 namespace Ardalis.SmartEnum.Dapper
 {
-    //using global::Dapper;
     using System;
 
     /// <summary>
diff --git a/src/SmartEnum.JsonNet/SmartEnum.JsonNet.csproj b/src/SmartEnum.JsonNet/SmartEnum.JsonNet.csproj
@@ -1,4 +1,4 @@
-﻿<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
 		<PackageId>Ardalis.SmartEnum.JsonNet</PackageId>
 		<Title>Ardalis.SmartEnum.JsonNet</Title>
@@ -15,8 +15,6 @@
 	</PropertyGroup>
 	<ItemGroup>
 		<PackageReference Include="Newtonsoft.Json"  />
-		<PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All" />
-		<PackageReference Include="SonarAnalyzer.CSharp" PrivateAssets="All" />
 	</ItemGroup>
 	<ItemGroup>
 		<None Include="$(SolutionDir)img\icon.png" Pack="true" Visible="false" PackagePath="" />
diff --git a/src/SmartEnum.MessagePack/SmartEnum.MessagePack.csproj b/src/SmartEnum.MessagePack/SmartEnum.MessagePack.csproj
@@ -15,8 +15,6 @@
 	</PropertyGroup>
 	<ItemGroup>
 		<PackageReference Include="MessagePack" />
-		<PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All" />
-		<PackageReference Include="SonarAnalyzer.CSharp" PrivateAssets="All" />
 	</ItemGroup>
 	<ItemGroup>
 		<None Include="$(SolutionDir)img\icon.png" Pack="true" Visible="false" PackagePath="" />
diff --git a/src/SmartEnum.ModelBinding/SmartEnumModelBinder.cs b/src/SmartEnum.ModelBinding/SmartEnumModelBinder.cs
@@ -36,14 +36,13 @@ public Task BindModelAsync(ModelBindingContext bindingContext)
                 if (methodInfo.Name == "FromName")
                 {
                     ParameterInfo[] methodsParams = methodInfo.GetParameters();
-                    if (methodsParams.Length == 2)
+                    if (methodsParams.Length == 2 &&
+                        methodsParams[0].ParameterType == typeof(string) &&
+                        methodsParams[1].ParameterType == typeof(bool))
                     {
-                        if (methodsParams[0].ParameterType == typeof(string) && methodsParams[1].ParameterType == typeof(bool))
-                        {
-                            var enumObj = methodInfo.Invoke(null, new object[] { enumKeyName, true });
-                            bindingContext.Result = ModelBindingResult.Success(enumObj);
-                            return Task.CompletedTask;
-                        }
+                        var enumObj = methodInfo.Invoke(null, new object[] { enumKeyName, true });
+                        bindingContext.Result = ModelBindingResult.Success(enumObj);
+                        return Task.CompletedTask;
                     }
                 }
             }
diff --git a/src/SmartEnum.ProtoBufNet/SmartEnum.ProtoBufNet.csproj b/src/SmartEnum.ProtoBufNet/SmartEnum.ProtoBufNet.csproj
@@ -15,8 +15,6 @@
 	</PropertyGroup>
 	<ItemGroup>
 		<PackageReference Include="protobuf-net" />
-		<PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All" />
-		<PackageReference Include="SonarAnalyzer.CSharp" PrivateAssets="All" />
 	</ItemGroup>
 	<ItemGroup>
 		<None Include="$(SolutionDir)img\icon.png" Pack="true" Visible="false" PackagePath="" />
diff --git a/src/SmartEnum.SystemTextJson/SmartEnum.SystemTextJson.csproj b/src/SmartEnum.SystemTextJson/SmartEnum.SystemTextJson.csproj
@@ -1,28 +1,26 @@
 <Project Sdk="Microsoft.NET.Sdk">
-	<PropertyGroup>
-		<PackageId>Ardalis.SmartEnum.SystemTextJson</PackageId>
-		<Title>Ardalis.SmartEnum.SystemTextJson</Title>
-		<Description>System.Text.Json (de)serialization support for Ardalis.SmartEnum.</Description>
-		<Summary>System.Text.Json (de)serialization support for Ardalis.SmartEnum.</Summary>
-    	<PackageLicenseExpression>MIT</PackageLicenseExpression>
-		<PackageTags>enum;smartenum;netstandard2.0;json;system.text.json;converter</PackageTags>
-		<PackageIcon>icon.png</PackageIcon>
-		<PackageReleaseNotes>Updating to net7.</PackageReleaseNotes>
-		<AssemblyName>Ardalis.SmartEnum.SystemTextJson</AssemblyName>
-		<RootNamespace>Ardalis.SmartEnum.SystemTextJson</RootNamespace>
-	</PropertyGroup>
-	<PropertyGroup Condition=" '$(Configuration)' == 'Release' ">
-		<DocumentationFile>bin\$(Configuration)\$(TargetFramework)\$(AssemblyName).xml</DocumentationFile>
-	</PropertyGroup>
-	<ItemGroup>
-		<PackageReference Include="System.Text.Json" />
-		<PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All" />
-		<PackageReference Include="SonarAnalyzer.CSharp" PrivateAssets="All" />
-	</ItemGroup>
-	<ItemGroup>
-		<None Include="$(SolutionDir)img\icon.png" Pack="true" Visible="false" PackagePath="" />
-	</ItemGroup>
-	<ItemGroup>
-		<ProjectReference Include="..\SmartEnum\SmartEnum.csproj" />
-	</ItemGroup>
+    <PropertyGroup>
+        <PackageId>Ardalis.SmartEnum.SystemTextJson</PackageId>
+        <Title>Ardalis.SmartEnum.SystemTextJson</Title>
+        <Description>System.Text.Json (de)serialization support for Ardalis.SmartEnum.</Description>
+        <Summary>System.Text.Json (de)serialization support for Ardalis.SmartEnum.</Summary>
+        <PackageLicenseExpression>MIT</PackageLicenseExpression>
+        <PackageTags>enum;smartenum;netstandard2.0;json;system.text.json;converter</PackageTags>
+        <PackageIcon>icon.png</PackageIcon>
+        <PackageReleaseNotes>Updating to net7.</PackageReleaseNotes>
+        <AssemblyName>Ardalis.SmartEnum.SystemTextJson</AssemblyName>
+        <RootNamespace>Ardalis.SmartEnum.SystemTextJson</RootNamespace>
+    </PropertyGroup>
+    <PropertyGroup Condition=" '$(Configuration)' == 'Release' ">
+        <DocumentationFile>bin\$(Configuration)\$(TargetFramework)\$(AssemblyName).xml</DocumentationFile>
+    </PropertyGroup>
+    <ItemGroup Condition="'$(TargetFramework)' == 'netStandard2.1'">
+        <PackageReference Include="System.Text.Json" />
+    </ItemGroup>
+    <ItemGroup>
+        <None Include="$(SolutionDir)img\icon.png" Pack="true" Visible="false" PackagePath="" />
+    </ItemGroup>
+    <ItemGroup>
+        <ProjectReference Include="..\SmartEnum\SmartEnum.csproj" />
+    </ItemGroup>
 </Project>
diff --git a/src/SmartEnum/Exceptions/InvalidFlagEnumValueParseException.cs b/src/SmartEnum/Exceptions/InvalidFlagEnumValueParseException.cs
@@ -1,11 +1,13 @@
 using System;
+using System.Diagnostics.CodeAnalysis;
 
 namespace Ardalis.SmartEnum.Exceptions
 {
     /// <summary>
-    /// 
+    ///
     /// </summary>
     [Serializable]
+    [SuppressMessage("Major Code Smell", "S3925:\"ISerializable\" should be implemented correctly", Justification = "<Pending>")]
     public class InvalidFlagEnumValueParseException : Exception
     {
         /// <summary>
@@ -16,7 +18,7 @@ public InvalidFlagEnumValueParseException() : base()
         }
 
         /// <summary>
-        /// Initializes a new instance of the <see cref="InvalidFlagEnumValueParseException"/> class with a user specified error <paramref name="message"/>. 
+        /// Initializes a new instance of the <see cref="InvalidFlagEnumValueParseException"/> class with a user specified error <paramref name="message"/>.
         /// </summary>
         /// <param name="message">The error message that explains the reason for the exception.</param>
         public InvalidFlagEnumValueParseException(string message) : base(message)
@@ -28,7 +30,7 @@ public InvalidFlagEnumValueParseException(string message) : base(message)
         /// and a wrapped <paramref name="innerException"/> that is the cause of this exception.
         /// </summary>
         /// <param name="message">The error message that explains the reason for the exception.</param>
-        /// <param name="innerException"></param> The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference, 
+        /// <param name="innerException"></param> The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference,
         /// the current exception is raised in a <c>catch</c> block that handles the inner exception.
         public InvalidFlagEnumValueParseException(string message, Exception innerException) : base(message, innerException)
         {
diff --git a/src/SmartEnum/Exceptions/NegativeValueArgumentException.cs b/src/SmartEnum/Exceptions/NegativeValueArgumentException.cs
@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Text;
 
 namespace Ardalis.SmartEnum.Exceptions
@@ -8,6 +9,7 @@ namespace Ardalis.SmartEnum.Exceptions
     /// The exception that is thrown when a negative value is provided as an argument value.
     /// </summary>
     [Serializable]
+    [SuppressMessage("Major Code Smell", "S3925:\"ISerializable\" should be implemented correctly", Justification = "<Pending>")]
     public class NegativeValueArgumentException : ArgumentException
     {
         /// <summary>
@@ -26,12 +28,12 @@ public NegativeValueArgumentException(string message) : base(message)
         }
 
         /// <summary>
-        /// Initializes a new instance of the <see cref="NegativeValueArgumentException"/> class with a specified error message and 
+        /// Initializes a new instance of the <see cref="NegativeValueArgumentException"/> class with a specified error message and
         /// a reference to the inner exception that is the cause of this exception.
         /// </summary>
         /// <param name="message">The error message that explains the reason for the exception.</param>
         /// <param name="innerException">
-        /// The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference, 
+        /// The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference,
         /// the current exception is raised in a <c>catch</c> block that handles the inner exception.
         /// </param>
         public NegativeValueArgumentException(string message, Exception innerException) : base(message, innerException)
diff --git a/src/SmartEnum/Exceptions/SmartEnumNotFoundException.cs b/src/SmartEnum/Exceptions/SmartEnumNotFoundException.cs
@@ -1,11 +1,13 @@
 namespace Ardalis.SmartEnum
 {
     using System;
+    using System.Diagnostics.CodeAnalysis;
 
     /// <summary>
     /// The exception that is thrown when a item is not found.
     /// </summary>
     [Serializable]
+    [SuppressMessage("Major Code Smell", "S3925:\"ISerializable\" should be implemented correctly", Justification = "<Pending>")]
     public class SmartEnumNotFoundException : Exception
     {
         /// <summary>
@@ -26,12 +28,12 @@ public SmartEnumNotFoundException(string message)
         }
 
         /// <summary>
-        /// Initializes a new instance of the <see cref="SmartEnumNotFoundException"/> class with a specified error message and 
+        /// Initializes a new instance of the <see cref="SmartEnumNotFoundException"/> class with a specified error message and
         /// a reference to the inner exception that is the cause of this exception.
         /// </summary>
         /// <param name="message">The error message that explains the reason for the exception.</param>
         /// <param name="innerException">
-        /// The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference, 
+        /// The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference,
         /// the current exception is raised in a <c>catch</c> block that handles the inner exception.
         /// </param>
         public SmartEnumNotFoundException(string message, Exception innerException)
diff --git a/src/SmartEnum/Exceptions/SmartFlagEnumContainsNegativeValueException.cs b/src/SmartEnum/Exceptions/SmartFlagEnumContainsNegativeValueException.cs
@@ -1,11 +1,13 @@
 using System;
+using System.Diagnostics.CodeAnalysis;
 
 namespace Ardalis.SmartEnum.Exceptions
 {
     /// <summary>
-    /// 
+    ///
     /// </summary>
     [Serializable]
+    [SuppressMessage("Major Code Smell", "S3925:\"ISerializable\" should be implemented correctly", Justification = "<Pending>")]
     public class SmartFlagEnumContainsNegativeValueException : Exception
     {
         /// <summary>
@@ -16,7 +18,7 @@ public SmartFlagEnumContainsNegativeValueException() : base()
         }
 
         /// <summary>
-        /// Initializes a new instance of the <see cref="SmartFlagEnumContainsNegativeValueException"/> class with a user specified error <paramref name="message"/>. 
+        /// Initializes a new instance of the <see cref="SmartFlagEnumContainsNegativeValueException"/> class with a user specified error <paramref name="message"/>.
         /// </summary>
         /// <param name="message">The error message that explains the reason for the exception.</param>
         public SmartFlagEnumContainsNegativeValueException(string message) : base(message)
@@ -28,7 +30,7 @@ public SmartFlagEnumContainsNegativeValueException(string message) : base(messag
         /// and a wrapped <paramref name="innerException"/> that is the cause of this exception.
         /// </summary>
         /// <param name="message">The error message that explains the reason for the exception.</param>
-        /// <param name="innerException"></param> The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference, 
+        /// <param name="innerException"></param> The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference,
         /// the current exception is raised in a <c>catch</c> block that handles the inner exception.
         public SmartFlagEnumContainsNegativeValueException(string message, Exception innerException) : base(message, innerException)
         {
diff --git a/src/SmartEnum/Exceptions/SmartFlagEnumDoesNotContainPowerOfTwoValuesException.cs b/src/SmartEnum/Exceptions/SmartFlagEnumDoesNotContainPowerOfTwoValuesException.cs
@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Text;
 
 namespace Ardalis.SmartEnum.Exceptions
@@ -8,6 +9,7 @@ namespace Ardalis.SmartEnum.Exceptions
     /// The exception that is thrown when a <see cref="SmartFlagEnum{TEnum}"/> does not contain consecutive power of two values.
     /// </summary>
     [Serializable]
+    [SuppressMessage("Major Code Smell", "S3925:\"ISerializable\" should be implemented correctly", Justification = "<Pending>")]
     public class SmartFlagEnumDoesNotContainPowerOfTwoValuesException : Exception
     {
         /// <summary>
@@ -28,12 +30,12 @@ public SmartFlagEnumDoesNotContainPowerOfTwoValuesException(string message)
         }
 
         /// <summary>
-        /// Initializes a new instance of the <see cref="SmartFlagEnumDoesNotContainPowerOfTwoValuesException"/> class with a specified error message and 
+        /// Initializes a new instance of the <see cref="SmartFlagEnumDoesNotContainPowerOfTwoValuesException"/> class with a specified error message and
         /// a reference to the inner exception that is the cause of this exception.
         /// </summary>
         /// <param name="message">The error message that explains the reason for the exception.</param>
         /// <param name="innerException">
-        /// The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference, 
+        /// The exception that is the cause of the current exception. If the <paramref name="innerException"/> parameter is not a null reference,
         /// the current exception is raised in a <c>catch</c> block that handles the inner exception.
         /// </param>
         public SmartFlagEnumDoesNotContainPowerOfTwoValuesException(string message, Exception innerException)
diff --git a/src/SmartEnum/SmartEnum.cs b/src/SmartEnum/SmartEnum.cs
diff --git a/src/SmartEnum/SmartEnum.csproj b/src/SmartEnum/SmartEnum.csproj
diff --git a/src/SmartEnum/SmartEnumNameAttribute.cs b/src/SmartEnum/SmartEnumNameAttribute.cs
diff --git a/src/SmartEnum/SmartFlagEngine.cs b/src/SmartEnum/SmartFlagEngine.cs
diff --git a/src/SmartEnum/SmartFlagEnum.cs b/src/SmartEnum/SmartFlagEnum.cs

Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,10 @@`
`1`	`1`	`namespace Ardalis.SmartEnum.Benchmarks`
`2`	`2`	`{`
`3`	`3`	`using BenchmarkDotNet.Attributes;`
	`4`	`+ using System.Diagnostics.CodeAnalysis;`
`4`	`5`
`5`	`6`	`[Config(typeof(Config))]`
	`7`	`+ [SuppressMessage("Major Bug", "S1764:Identical expressions should not be used on both sides of operators", Justification = "<Pending>")]`
`6`	`8`	`public class EqualsBenchmarks`
`7`	`9`	`{`
`8`	`10`	`////////////////////////////////////////////////////////////////////////////////`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`namespace Ardalis.SmartEnum.Benchmarks`
`2`	`2`	`{`
	`3`	`+ using System.Diagnostics.CodeAnalysis;`
`3`	`4`	`using System.IO;`
`4`	`5`	`using BenchmarkDotNet.Attributes;`
`5`	`6`
`@@ -68,6 +69,7 @@ public sealed class TestSmartEnumValueClass`
`68`	`69`	`static Stream valueProtoBufDeserializeStream;`
`69`	`70`
`70`	`71`	`[GlobalSetup]`
	`72`	`+ [SuppressMessage("Critical Code Smell", "S2696:Instance members should not write to \"static\" fields", Justification = "<Pending>")]`
`71`	`73`	`public void GlobalSetup()`
`72`	`74`	`{`
`73`	`75`	`global::Utf8Json.Resolvers.CompositeResolver.Register(`
Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,12 @@`
`1`	`1`	`namespace Ardalis.SmartEnum.Benchmarks`
`2`	`2`	`{`
`3`	`3`	`using System;`
	`4`	`+ using System.Diagnostics.CodeAnalysis;`
`4`	`5`	`using BenchmarkDotNet.Attributes;`
`5`	`6`
`6`	`7`	`[Config(typeof(Config))]`
	`8`	`+ [SuppressMessage("Major Bug", "S2583:Conditionally executed code should be reachable", Justification = "<Pending>")]`
	`9`	`+ [SuppressMessage("Major Code Smell", "S112:General or reserved exceptions should never be thrown", Justification = "<Pending>")]`
`7`	`10`	`public class SwitchBenchmarks`
`8`	`11`	`{`
`9`	`12`	`////////////////////////////////////////////////////////////////////////////////`
Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,12 @@`
	`1`	`+using System.Diagnostics.CodeAnalysis;`
	`2`	`+`
`1`	`3`	`namespace Ardalis.SmartEnum.Benchmarks`
`2`	`4`	`{`
`3`	`5`	`////////////////////////////////////////////////////////////////////////////////`
`4`	`6`	`// Enum`
`5`	`7`
`6`	`8`	`[global::ProtoBuf.ProtoContract]`
	`9`	`+ [SuppressMessage("Minor Code Smell", "S2344:Enumeration type names should not have \"Flags\" or \"Enum\" suffixes", Justification = "<Pending>")]`
`7`	`10`	`public enum TestEnum`
`8`	`11`	`{`
`9`	`12`	`[global::ProtoBuf.ProtoEnum]`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,5 @@`
`1`	`1`	`namespace Ardalis.SmartEnum.Dapper`
`2`	`2`	`{`
`3`		`- //using global::Dapper;`
`4`	`3`	`using System;`
`5`	`4`
`6`	`5`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -36,14 +36,13 @@ public Task BindModelAsync(ModelBindingContext bindingContext)`
`36`	`36`	`if (methodInfo.Name == "FromName")`
`37`	`37`	`{`
`38`	`38`	`ParameterInfo[] methodsParams = methodInfo.GetParameters();`
`39`		`- if (methodsParams.Length == 2)`
	`39`	`+ if (methodsParams.Length == 2 &&`
	`40`	`+ methodsParams[0].ParameterType == typeof(string) &&`
	`41`	`+ methodsParams[1].ParameterType == typeof(bool))`
`40`	`42`	`{`
`41`		`- if (methodsParams[0].ParameterType == typeof(string) && methodsParams[1].ParameterType == typeof(bool))`
`42`		`- {`
`43`		`- var enumObj = methodInfo.Invoke(null, new object[] { enumKeyName, true });`
`44`		`- bindingContext.Result = ModelBindingResult.Success(enumObj);`
`45`		`- return Task.CompletedTask;`
`46`		`- }`
	`43`	`+ var enumObj = methodInfo.Invoke(null, new object[] { enumKeyName, true });`
	`44`	`+ bindingContext.Result = ModelBindingResult.Success(enumObj);`
	`45`	`+ return Task.CompletedTask;`
`47`	`46`	`}`
`48`	`47`	`}`
`49`	`48`	`}`