From 3ae7e2806c07d9e11d29bbdf5f0807a7acdcfa4a Mon Sep 17 00:00:00 2001
From: Maurycy Markowski <maumar@microsoft.com>
Date: Wed, 8 Jan 2020 16:46:59 -0800
Subject: [PATCH] additional code cleanup after null semantics

---
 .../SqlServerQueryTranslationPostprocessor.cs | 28 -----------
 ...verQueryTranslationPostprocessorFactory.cs | 47 -------------------
 2 files changed, 75 deletions(-)
 delete mode 100644 src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessor.cs
 delete mode 100644 src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessorFactory.cs

diff --git a/src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessor.cs b/src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessor.cs
deleted file mode 100644
index f29dd958ed4..00000000000
--- a/src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessor.cs
+++ /dev/null
@@ -1,28 +0,0 @@
-// Copyright (c) .NET Foundation. All rights reserved.
-// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
-
-using System.Linq.Expressions;
-using JetBrains.Annotations;
-using Microsoft.EntityFrameworkCore.Query;
-
-namespace Microsoft.EntityFrameworkCore.SqlServer.Query.Internal
-{
-    public class SqlServerQueryTranslationPostprocessor : RelationalQueryTranslationPostprocessor
-    {
-        public SqlServerQueryTranslationPostprocessor(
-            [NotNull] QueryTranslationPostprocessorDependencies dependencies,
-            [NotNull] RelationalQueryTranslationPostprocessorDependencies relationalDependencies,
-            [NotNull] QueryCompilationContext queryCompilationContext)
-            : base(dependencies, relationalDependencies, queryCompilationContext)
-        {
-        }
-
-        public override Expression Process(Expression query)
-        {
-            query = base.Process(query);
-            query = new SearchConditionConvertingExpressionVisitor(SqlExpressionFactory).Visit(query);
-
-            return query;
-        }
-    }
-}
diff --git a/src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessorFactory.cs b/src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessorFactory.cs
deleted file mode 100644
index bcf96e23df9..00000000000
--- a/src/EFCore.SqlServer/Query/Internal/SqlServerQueryTranslationPostprocessorFactory.cs
+++ /dev/null
@@ -1,47 +0,0 @@
-// Copyright (c) .NET Foundation. All rights reserved.
-// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
-
-using JetBrains.Annotations;
-using Microsoft.EntityFrameworkCore.Query;
-using Microsoft.EntityFrameworkCore.Utilities;
-using Microsoft.Extensions.DependencyInjection;
-
-namespace Microsoft.EntityFrameworkCore.SqlServer.Query.Internal
-{
-    /// <summary>
-    ///     <para>
-    ///         This is an internal API that supports the Entity Framework Core infrastructure and not subject to
-    ///         the same compatibility standards as public APIs. It may be changed or removed without notice in
-    ///         any release. You should only use it directly in your code with extreme caution and knowing that
-    ///         doing so can result in application failures when updating to a new Entity Framework Core release.
-    ///     </para>
-    ///     <para>
-    ///         The service lifetime is <see cref="ServiceLifetime.Singleton" />. This means a single instance
-    ///         is used by many <see cref="DbContext" /> instances. The implementation must be thread-safe.
-    ///         This service cannot depend on services registered as <see cref="ServiceLifetime.Scoped" />.
-    ///     </para>
-    /// </summary>
-    public class SqlServerQueryTranslationPostprocessorFactory : IQueryTranslationPostprocessorFactory
-    {
-        private readonly QueryTranslationPostprocessorDependencies _dependencies;
-        private readonly RelationalQueryTranslationPostprocessorDependencies _relationalDependencies;
-
-        public SqlServerQueryTranslationPostprocessorFactory(
-            [NotNull] QueryTranslationPostprocessorDependencies dependencies,
-            [NotNull] RelationalQueryTranslationPostprocessorDependencies relationalDependencies)
-        {
-            _dependencies = dependencies;
-            _relationalDependencies = relationalDependencies;
-        }
-
-        public virtual QueryTranslationPostprocessor Create(QueryCompilationContext queryCompilationContext)
-        {
-            Check.NotNull(queryCompilationContext, nameof(queryCompilationContext));
-
-            return new SqlServerQueryTranslationPostprocessor(
-                _dependencies,
-                _relationalDependencies,
-                queryCompilationContext);
-        }
-    }
-}