Fix code analysis

Author: commonsensesoftware

src/Common/ApiVersion.cs

@@ -393,7 +393,7 @@ public override int GetHashCode()
         /// <param name="version2">The <see cref="ApiVersion"/> to compare against.</param>
         /// <returns>True if the objects are not equal; otherwise, false.</returns>
         public static bool operator !=( ApiVersion? version1, ApiVersion? version2 ) =>
-             version1 is null ? version2 is object : !version1.Equals( version2 );
+             version1 is null ? version2 is not null : !version1.Equals( version2 );
 
         /// <summary>
         /// Overloads the less than operator.
@@ -402,7 +402,7 @@ public override int GetHashCode()
         /// <param name="version2">The <see cref="ApiVersion"/> to compare against.</param>
         /// <returns>True the first object is less than the second object; otherwise, false.</returns>
         public static bool operator <( ApiVersion? version1, ApiVersion? version2 ) =>
-            version1 is null ? version2 is object : version1.CompareTo( version2 ) < 0;
+            version1 is null ? version2 is not null : version1.CompareTo( version2 ) < 0;
 
         /// <summary>
         /// Overloads the less than or equal to operator.

src/Microsoft.AspNetCore.Mvc.Versioning/Routing/ApiVersionMatcherPolicy.cs

@@ -192,10 +192,10 @@ bool IsRequestedApiVersionAmbiguous( HttpContext httpContext, out ApiVersion? ap
             }
             catch ( AmbiguousApiVersionException ex )
             {
-                Logger.LogInformation( ex.Message );
+                Logger.ApiVersionAmbiguous( ex );
                 apiVersion = default;
 
-                var handlerContext = new RequestHandlerContext( Options.ErrorResponses )
+                var handlerContext = new RequestHandlerContext( Options.ErrorResponses, ApiVersionReporter )
                 {
                     Code = AmbiguousApiVersion,
                     Message = ex.Message,

src/Microsoft.AspNetCore.Mvc.Versioning/Versioning/ILoggerExtensions.cs

@@ -51,7 +51,7 @@ internal static void ConstraintMismatch( this ILogger logger, string actionName,
         internal static void ApiVersionAmbiguous( this ILogger logger, string[]? apiVersions ) => apiVersionAmbiguous( logger, apiVersions, null );
 
         internal static void ApiVersionAmbiguous( this ILogger logger, AmbiguousApiVersionException exception ) =>
-            logger.ApiVersionAmbiguous( exception.ApiVersions.Select( v => v.ToString() ).ToArray() );
+            logger.ApiVersionAmbiguous( exception.ApiVersions.ToArray() );
 
         internal static void NoActionsMatched( this ILogger logger, IDictionary<string, object?> routeValueDictionary )
         {