Merge pull request #41512 from CyrusNajmabadi/fixAllCleaup

mavasani · web-flow · commit 1a4de486531e · 2020-02-10T13:35:31.000-08:00
Cleanup fix all code.
diff --git a/src/Features/Core/Portable/CodeFixes/Suppression/AbstractSuppressionCodeFixProvider.FixAllProvider.cs b/src/Features/Core/Portable/CodeFixes/Suppression/AbstractSuppressionCodeFixProvider.FixAllProvider.cs
@@ -26,40 +26,34 @@ public async override Task<CodeAction> GetFixAsync(FixAllContext fixAllContext)
                     return null;
                 }
 
-                var batchFixer = WellKnownFixAllProviders.BatchFixer;
                 var suppressionFixer = (AbstractSuppressionCodeFixProvider)((WrapperCodeFixProvider)fixAllContext.CodeFixProvider).SuppressionFixProvider;
-                var isGlobalSuppression = NestedSuppressionCodeAction.IsEquivalenceKeyForGlobalSuppression(fixAllContext.CodeActionEquivalenceKey);
-                if (!isGlobalSuppression)
-                {
-                    var isPragmaWarningSuppression = NestedSuppressionCodeAction.IsEquivalenceKeyForPragmaWarning(fixAllContext.CodeActionEquivalenceKey);
-                    Contract.ThrowIfFalse(isPragmaWarningSuppression || NestedSuppressionCodeAction.IsEquivalenceKeyForRemoveSuppression(fixAllContext.CodeActionEquivalenceKey));
 
-                    batchFixer = isPragmaWarningSuppression ?
-                        new PragmaWarningBatchFixAllProvider(suppressionFixer) :
-                        RemoveSuppressionCodeAction.GetBatchFixer(suppressionFixer);
+                if (NestedSuppressionCodeAction.IsEquivalenceKeyForGlobalSuppression(fixAllContext.CodeActionEquivalenceKey))
+                {
+                    // For global suppressions, we defer to the global suppression system to handle directly.
+                    var title = fixAllContext.CodeActionEquivalenceKey;
+                    return fixAllContext.Document != null
+                        ? GlobalSuppressMessageFixAllCodeAction.Create(
+                            title, suppressionFixer, fixAllContext.Document,
+                            await fixAllContext.GetDocumentDiagnosticsToFixAsync().ConfigureAwait(false))
+                        : GlobalSuppressMessageFixAllCodeAction.Create(
+                            title, suppressionFixer, fixAllContext.Project,
+                            await fixAllContext.GetProjectDiagnosticsToFixAsync().ConfigureAwait(false));
                 }
 
-                var title = fixAllContext.CodeActionEquivalenceKey;
-                if (fixAllContext.Document != null)
+                if (NestedSuppressionCodeAction.IsEquivalenceKeyForPragmaWarning(fixAllContext.CodeActionEquivalenceKey))
                 {
-                    var documentsAndDiagnosticsToFixMap =
-                        await fixAllContext.GetDocumentDiagnosticsToFixAsync().ConfigureAwait(false);
-
-                    return !isGlobalSuppression
-                        ? await batchFixer.GetFixAsync(
-                            documentsAndDiagnosticsToFixMap, fixAllContext.State, fixAllContext.CancellationToken).ConfigureAwait(false)
-                        : GlobalSuppressMessageFixAllCodeAction.Create(title, suppressionFixer, fixAllContext.Document, documentsAndDiagnosticsToFixMap);
+                    var batchFixer = new PragmaWarningBatchFixAllProvider(suppressionFixer);
+                    return await batchFixer.GetFixAsync(fixAllContext).ConfigureAwait(false);
                 }
-                else
-                {
-                    var projectsAndDiagnosticsToFixMap =
-                        await fixAllContext.GetProjectDiagnosticsToFixAsync().ConfigureAwait(false);
 
-                    return !isGlobalSuppression
-                        ? await batchFixer.GetFixAsync(
-                            projectsAndDiagnosticsToFixMap, fixAllContext.State, fixAllContext.CancellationToken).ConfigureAwait(false)
-                        : GlobalSuppressMessageFixAllCodeAction.Create(title, suppressionFixer, fixAllContext.Project, projectsAndDiagnosticsToFixMap);
+                if (NestedSuppressionCodeAction.IsEquivalenceKeyForRemoveSuppression(fixAllContext.CodeActionEquivalenceKey))
+                {
+                    var batchFixer = RemoveSuppressionCodeAction.GetBatchFixer(suppressionFixer);
+                    return await batchFixer.GetFixAsync(fixAllContext).ConfigureAwait(false);
                 }
+
+                throw ExceptionUtilities.Unreachable;
             }
         }
     }
diff --git a/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/BatchFixAllProvider.cs b/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/BatchFixAllProvider.cs
@@ -46,7 +46,7 @@ public override async Task<CodeAction> GetFixAsync(FixAllContext fixAllContext)
 
         #endregion
 
-        internal override async Task<CodeAction> GetFixAsync(
+        private async Task<CodeAction> GetFixAsync(
             ImmutableDictionary<Document, ImmutableArray<Diagnostic>> documentsAndDiagnosticsToFixMap,
             FixAllState fixAllState, CancellationToken cancellationToken)
         {
@@ -131,7 +131,7 @@ protected async virtual Task AddDocumentFixesAsync(
             await Task.WhenAll(fixerTasks).ConfigureAwait(false);
         }
 
-        internal override async Task<CodeAction> GetFixAsync(
+        private async Task<CodeAction> GetFixAsync(
             ImmutableDictionary<Project, ImmutableArray<Diagnostic>> projectsAndDiagnosticsToFixMap,
             FixAllState fixAllState, CancellationToken cancellationToken)
         {
diff --git a/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllContext.DiagnosticProvider.cs b/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllContext.DiagnosticProvider.cs
@@ -24,8 +24,6 @@ public partial class FixAllContext
         /// </summary>
         public abstract class DiagnosticProvider
         {
-            internal virtual bool IsFixMultiple => false;
-
             /// <summary>
             /// Gets all the diagnostics to fix in the given document in a <see cref="FixAllContext"/>.
             /// </summary>
@@ -42,27 +40,30 @@ public abstract class DiagnosticProvider
             /// </summary>
             public abstract Task<IEnumerable<Diagnostic>> GetAllDiagnosticsAsync(Project project, CancellationToken cancellationToken);
 
-            internal async Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixAsync(
-                FixAllContext fixAllContext)
+            internal async Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixAsync(FixAllContext fixAllContext)
             {
                 var result = await GetDocumentDiagnosticsToFixWorkerAsync(fixAllContext).ConfigureAwait(false);
 
                 // Filter out any documents that we don't have any diagnostics for.
                 return result.Where(kvp => !kvp.Value.IsDefaultOrEmpty).ToImmutableDictionary();
-            }
 
-            internal virtual async Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixWorkerAsync(
-                FixAllContext fixAllContext)
-            {
-                using (Logger.LogBlock(
-                    FunctionId.CodeFixes_FixAllOccurrencesComputation_Document_Diagnostics,
-                    FixAllLogger.CreateCorrelationLogMessage(fixAllContext.State.CorrelationId),
-                    fixAllContext.CancellationToken))
+                static async Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixWorkerAsync(FixAllContext fixAllContext)
                 {
-                    return await FixAllContextHelper.GetDocumentDiagnosticsToFixAsync(
-                        fixAllContext,
-                        fixAllContext.ProgressTracker,
-                        (document, cancellationToken) => document.IsGeneratedCode(cancellationToken)).ConfigureAwait(false);
+                    if (fixAllContext.State.DiagnosticProvider is FixAllState.FixMultipleDiagnosticProvider fixMultipleDiagnosticProvider)
+                    {
+                        return fixMultipleDiagnosticProvider.DocumentDiagnosticsMap;
+                    }
+
+                    using (Logger.LogBlock(
+                            FunctionId.CodeFixes_FixAllOccurrencesComputation_Document_Diagnostics,
+                            FixAllLogger.CreateCorrelationLogMessage(fixAllContext.State.CorrelationId),
+                            fixAllContext.CancellationToken))
+                    {
+                        return await FixAllContextHelper.GetDocumentDiagnosticsToFixAsync(
+                            fixAllContext,
+                            fixAllContext.ProgressTracker,
+                            (d, c) => DocumentExtensions.IsGeneratedCode(d, c)).ConfigureAwait(false);
+                    }
                 }
             }
 
diff --git a/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllProvider.cs b/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllProvider.cs
@@ -42,19 +42,5 @@ public virtual IEnumerable<string> GetSupportedFixAllDiagnosticIds(CodeFixProvid
         /// Gets fix all occurrences fix for the given fixAllContext.
         /// </summary>
         public abstract Task<CodeAction> GetFixAsync(FixAllContext fixAllContext);
-
-        internal virtual Task<CodeAction> GetFixAsync(
-            ImmutableDictionary<Document, ImmutableArray<Diagnostic>> documentsAndDiagnosticsToFixMap,
-            FixAllState fixAllState, CancellationToken cancellationToken)
-        {
-            return Task.FromResult<CodeAction>(null);
-        }
-
-        internal virtual Task<CodeAction> GetFixAsync(
-            ImmutableDictionary<Project, ImmutableArray<Diagnostic>> projectsAndDiagnosticsToFixMap,
-            FixAllState fixAllState, CancellationToken cancellationToken)
-        {
-            return Task.FromResult<CodeAction>(null);
-        }
     }
 }
diff --git a/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllState.FixMultipleDiagnosticProvider.cs b/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllState.FixMultipleDiagnosticProvider.cs
@@ -18,51 +18,37 @@ internal partial class FixAllState
         /// </summary>
         internal sealed class FixMultipleDiagnosticProvider : FixAllContext.DiagnosticProvider
         {
-            internal override bool IsFixMultiple => true;
-
-            private readonly ImmutableDictionary<Document, ImmutableArray<Diagnostic>> _documentDiagnosticsMap;
-            private readonly ImmutableDictionary<Project, ImmutableArray<Diagnostic>> _projectDiagnosticsMap;
+            public ImmutableDictionary<Document, ImmutableArray<Diagnostic>> DocumentDiagnosticsMap { get; }
+            public ImmutableDictionary<Project, ImmutableArray<Diagnostic>> ProjectDiagnosticsMap { get; }
 
             public FixMultipleDiagnosticProvider(ImmutableDictionary<Document, ImmutableArray<Diagnostic>> diagnosticsMap)
             {
-                _documentDiagnosticsMap = diagnosticsMap;
-                _projectDiagnosticsMap = ImmutableDictionary<Project, ImmutableArray<Diagnostic>>.Empty;
+                DocumentDiagnosticsMap = diagnosticsMap;
+                ProjectDiagnosticsMap = ImmutableDictionary<Project, ImmutableArray<Diagnostic>>.Empty;
             }
 
             public FixMultipleDiagnosticProvider(ImmutableDictionary<Project, ImmutableArray<Diagnostic>> diagnosticsMap)
             {
-                _projectDiagnosticsMap = diagnosticsMap;
-                _documentDiagnosticsMap = ImmutableDictionary<Document, ImmutableArray<Diagnostic>>.Empty;
-            }
-
-            internal override Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixWorkerAsync(
-                FixAllContext context)
-            {
-                return Task.FromResult(_documentDiagnosticsMap);
-            }
-
-            internal override Task<ImmutableDictionary<Project, ImmutableArray<Diagnostic>>> GetProjectDiagnosticsToFixAsync(
-                FixAllContext context)
-            {
-                return Task.FromResult(_projectDiagnosticsMap);
+                ProjectDiagnosticsMap = diagnosticsMap;
+                DocumentDiagnosticsMap = ImmutableDictionary<Document, ImmutableArray<Diagnostic>>.Empty;
             }
 
             public override Task<IEnumerable<Diagnostic>> GetAllDiagnosticsAsync(Project project, CancellationToken cancellationToken)
             {
                 var allDiagnosticsBuilder = ArrayBuilder<Diagnostic>.GetInstance();
                 ImmutableArray<Diagnostic> diagnostics;
-                if (!_documentDiagnosticsMap.IsEmpty)
+                if (!DocumentDiagnosticsMap.IsEmpty)
                 {
                     foreach (var document in project.Documents)
                     {
-                        if (_documentDiagnosticsMap.TryGetValue(document, out diagnostics))
+                        if (DocumentDiagnosticsMap.TryGetValue(document, out diagnostics))
                         {
                             allDiagnosticsBuilder.AddRange(diagnostics);
                         }
                     }
                 }
 
-                if (_projectDiagnosticsMap.TryGetValue(project, out diagnostics))
+                if (ProjectDiagnosticsMap.TryGetValue(project, out diagnostics))
                 {
                     allDiagnosticsBuilder.AddRange(diagnostics);
                 }
@@ -72,7 +58,7 @@ public override Task<IEnumerable<Diagnostic>> GetAllDiagnosticsAsync(Project pro
 
             public override Task<IEnumerable<Diagnostic>> GetDocumentDiagnosticsAsync(Document document, CancellationToken cancellationToken)
             {
-                if (_documentDiagnosticsMap.TryGetValue(document, out var diagnostics))
+                if (DocumentDiagnosticsMap.TryGetValue(document, out var diagnostics))
                 {
                     return Task.FromResult<IEnumerable<Diagnostic>>(diagnostics);
                 }
@@ -82,7 +68,7 @@ public override Task<IEnumerable<Diagnostic>> GetDocumentDiagnosticsAsync(Docume
 
             public override Task<IEnumerable<Diagnostic>> GetProjectDiagnosticsAsync(Project project, CancellationToken cancellationToken)
             {
-                if (_projectDiagnosticsMap.TryGetValue(project, out var diagnostics))
+                if (ProjectDiagnosticsMap.TryGetValue(project, out var diagnostics))
                 {
                     return Task.FromResult<IEnumerable<Diagnostic>>(diagnostics);
                 }
diff --git a/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllState.cs b/src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllState.cs
@@ -91,7 +91,7 @@ private FixAllState(
             this.DiagnosticProvider = fixAllDiagnosticProvider ?? throw new ArgumentNullException(nameof(fixAllDiagnosticProvider));
         }
 
-        internal bool IsFixMultiple => this.DiagnosticProvider.IsFixMultiple;
+        internal bool IsFixMultiple => this.DiagnosticProvider is FixMultipleDiagnosticProvider;
 
         public FixAllState WithScopeAndEquivalenceKey(FixAllScope scope, string codeActionEquivalenceKey)
         {
diff --git a/src/Workspaces/Core/Portable/CodeFixes/SyntaxEditorBasedCodeFixProvider.FixAllProvider.cs b/src/Workspaces/Core/Portable/CodeFixes/SyntaxEditorBasedCodeFixProvider.FixAllProvider.cs
@@ -8,6 +8,8 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.CodeAnalysis.CodeActions;
+using Microsoft.CodeAnalysis.CodeStyle;
+using Microsoft.CodeAnalysis.Shared.Extensions;
 
 namespace Microsoft.CodeAnalysis.CodeFixes
 {
@@ -34,7 +36,23 @@ public sealed override async Task<CodeAction> GetFixAsync(FixAllContext fixAllCo
                 return await GetFixAsync(documentsAndDiagnosticsToFixMap, fixAllContext.State, fixAllContext.CancellationToken).ConfigureAwait(false);
             }
 
-            internal sealed override async Task<CodeAction> GetFixAsync(
+            private async Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixAsync(FixAllContext fixAllContext)
+            {
+                var result = await GetDocumentDiagnosticsToFixWorkerAsync(fixAllContext).ConfigureAwait(false);
+
+                // Filter out any documents that we don't have any diagnostics for.
+                return result.Where(kvp => !kvp.Value.IsDefaultOrEmpty).ToImmutableDictionary();
+
+                static async Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixWorkerAsync(FixAllContext fixAllContext)
+                {
+                    return await FixAllContextHelper.GetDocumentDiagnosticsToFixAsync(
+                        fixAllContext,
+                        progressTrackerOpt: null,
+                        (d, c) => DocumentExtensions.IsGeneratedCode(d, c)).ConfigureAwait(false);
+                }
+            }
+
+            private async Task<CodeAction> GetFixAsync(
                 ImmutableDictionary<Document, ImmutableArray<Diagnostic>> documentsAndDiagnosticsToFixMap,
                 FixAllState fixAllState, CancellationToken cancellationToken)
             {

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ public override async Task<CodeAction> GetFixAsync(FixAllContext fixAllContext)`
`46`	`46`
`47`	`47`	`#endregion`
`48`	`48`
`49`		`- internal override async Task<CodeAction> GetFixAsync(`
	`49`	`+ private async Task<CodeAction> GetFixAsync(`
`50`	`50`	`ImmutableDictionary<Document, ImmutableArray<Diagnostic>> documentsAndDiagnosticsToFixMap,`
`51`	`51`	`FixAllState fixAllState, CancellationToken cancellationToken)`
`52`	`52`	`{`
`@@ -131,7 +131,7 @@ protected async virtual Task AddDocumentFixesAsync(`
`131`	`131`	`await Task.WhenAll(fixerTasks).ConfigureAwait(false);`
`132`	`132`	`}`
`133`	`133`
`134`		`- internal override async Task<CodeAction> GetFixAsync(`
	`134`	`+ private async Task<CodeAction> GetFixAsync(`
`135`	`135`	`ImmutableDictionary<Project, ImmutableArray<Diagnostic>> projectsAndDiagnosticsToFixMap,`
`136`	`136`	`FixAllState fixAllState, CancellationToken cancellationToken)`
`137`	`137`	`{`
Original file line number	Diff line number	Diff line change
`@@ -18,51 +18,37 @@ internal partial class FixAllState`
`18`	`18`	`/// </summary>`
`19`	`19`	`internal sealed class FixMultipleDiagnosticProvider : FixAllContext.DiagnosticProvider`
`20`	`20`	`{`
`21`		`- internal override bool IsFixMultiple => true;`
`22`		`-`
`23`		`- private readonly ImmutableDictionary<Document, ImmutableArray<Diagnostic>> _documentDiagnosticsMap;`
`24`		`- private readonly ImmutableDictionary<Project, ImmutableArray<Diagnostic>> _projectDiagnosticsMap;`
	`21`	`+ public ImmutableDictionary<Document, ImmutableArray<Diagnostic>> DocumentDiagnosticsMap { get; }`
	`22`	`+ public ImmutableDictionary<Project, ImmutableArray<Diagnostic>> ProjectDiagnosticsMap { get; }`
`25`	`23`
`26`	`24`	`public FixMultipleDiagnosticProvider(ImmutableDictionary<Document, ImmutableArray<Diagnostic>> diagnosticsMap)`
`27`	`25`	`{`
`28`		`- _documentDiagnosticsMap = diagnosticsMap;`
`29`		`- _projectDiagnosticsMap = ImmutableDictionary<Project, ImmutableArray<Diagnostic>>.Empty;`
	`26`	`+ DocumentDiagnosticsMap = diagnosticsMap;`
	`27`	`+ ProjectDiagnosticsMap = ImmutableDictionary<Project, ImmutableArray<Diagnostic>>.Empty;`
`30`	`28`	`}`
`31`	`29`
`32`	`30`	`public FixMultipleDiagnosticProvider(ImmutableDictionary<Project, ImmutableArray<Diagnostic>> diagnosticsMap)`
`33`	`31`	`{`
`34`		`- _projectDiagnosticsMap = diagnosticsMap;`
`35`		`- _documentDiagnosticsMap = ImmutableDictionary<Document, ImmutableArray<Diagnostic>>.Empty;`
`36`		`- }`
`37`		`-`
`38`		`- internal override Task<ImmutableDictionary<Document, ImmutableArray<Diagnostic>>> GetDocumentDiagnosticsToFixWorkerAsync(`
`39`		`- FixAllContext context)`
`40`		`- {`
`41`		`- return Task.FromResult(_documentDiagnosticsMap);`
`42`		`- }`
`43`		`-`
`44`		`- internal override Task<ImmutableDictionary<Project, ImmutableArray<Diagnostic>>> GetProjectDiagnosticsToFixAsync(`
`45`		`- FixAllContext context)`
`46`		`- {`
`47`		`- return Task.FromResult(_projectDiagnosticsMap);`
	`32`	`+ ProjectDiagnosticsMap = diagnosticsMap;`
	`33`	`+ DocumentDiagnosticsMap = ImmutableDictionary<Document, ImmutableArray<Diagnostic>>.Empty;`
`48`	`34`	`}`
`49`	`35`
`50`	`36`	`public override Task<IEnumerable<Diagnostic>> GetAllDiagnosticsAsync(Project project, CancellationToken cancellationToken)`
`51`	`37`	`{`
`52`	`38`	`var allDiagnosticsBuilder = ArrayBuilder<Diagnostic>.GetInstance();`
`53`	`39`	`ImmutableArray<Diagnostic> diagnostics;`
`54`		`- if (!_documentDiagnosticsMap.IsEmpty)`
	`40`	`+ if (!DocumentDiagnosticsMap.IsEmpty)`
`55`	`41`	`{`
`56`	`42`	`foreach (var document in project.Documents)`
`57`	`43`	`{`
`58`		`- if (_documentDiagnosticsMap.TryGetValue(document, out diagnostics))`
	`44`	`+ if (DocumentDiagnosticsMap.TryGetValue(document, out diagnostics))`
`59`	`45`	`{`
`60`	`46`	`allDiagnosticsBuilder.AddRange(diagnostics);`
`61`	`47`	`}`
`62`	`48`	`}`
`63`	`49`	`}`
`64`	`50`
`65`		`- if (_projectDiagnosticsMap.TryGetValue(project, out diagnostics))`
	`51`	`+ if (ProjectDiagnosticsMap.TryGetValue(project, out diagnostics))`
`66`	`52`	`{`
`67`	`53`	`allDiagnosticsBuilder.AddRange(diagnostics);`
`68`	`54`	`}`
`@@ -72,7 +58,7 @@ public override Task<IEnumerable<Diagnostic>> GetAllDiagnosticsAsync(Project pro`
`72`	`58`
`73`	`59`	`public override Task<IEnumerable<Diagnostic>> GetDocumentDiagnosticsAsync(Document document, CancellationToken cancellationToken)`
`74`	`60`	`{`
`75`		`- if (_documentDiagnosticsMap.TryGetValue(document, out var diagnostics))`
	`61`	`+ if (DocumentDiagnosticsMap.TryGetValue(document, out var diagnostics))`
`76`	`62`	`{`
`77`	`63`	`return Task.FromResult<IEnumerable<Diagnostic>>(diagnostics);`
`78`	`64`	`}`
`@@ -82,7 +68,7 @@ public override Task<IEnumerable<Diagnostic>> GetDocumentDiagnosticsAsync(Docume`
`82`	`68`
`83`	`69`	`public override Task<IEnumerable<Diagnostic>> GetProjectDiagnosticsAsync(Project project, CancellationToken cancellationToken)`
`84`	`70`	`{`
`85`		`- if (_projectDiagnosticsMap.TryGetValue(project, out var diagnostics))`
	`71`	`+ if (ProjectDiagnosticsMap.TryGetValue(project, out var diagnostics))`
`86`	`72`	`{`
`87`	`73`	`return Task.FromResult<IEnumerable<Diagnostic>>(diagnostics);`
`88`	`74`	`}`
Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ private FixAllState(`
`91`	`91`	`this.DiagnosticProvider = fixAllDiagnosticProvider ?? throw new ArgumentNullException(nameof(fixAllDiagnosticProvider));`
`92`	`92`	`}`
`93`	`93`
`94`		`- internal bool IsFixMultiple => this.DiagnosticProvider.IsFixMultiple;`
	`94`	`+ internal bool IsFixMultiple => this.DiagnosticProvider is FixMultipleDiagnosticProvider;`
`95`	`95`
`96`	`96`	`public FixAllState WithScopeAndEquivalenceKey(FixAllScope scope, string codeActionEquivalenceKey)`
`97`	`97`	`{`