Switch to ToImmutableAndClear

src/Analyzers/CSharp/Analyzers/UseCollectionExpression/UseCollectionExpressionHelpers.cs

@@ -906,7 +906,7 @@ public static ImmutableArray<CollectionExpressionMatch<StatementSyntax>> TryGetM
             return default;
         }
 
-        return matches.ToImmutable();
+        return matches.ToImmutableAndClear();
     }
 
     public static bool IsCollectionFactoryCreate(

src/Analyzers/CSharp/CodeFixes/ConvertToRecord/ConvertToRecordHelpers.cs

@@ -533,7 +533,7 @@ private static ImmutableArray<IFieldSymbol> GetEqualizedFields(
                     value, successRequirement: true, type, fields, out var _2))
             {
                 // we're done, no more statements to check
-                return fields.ToImmutable();
+                return fields.ToImmutableAndClear();
             }
             // check for the first statement as an explicit cast to a variable declaration
             // like: var otherC = other as C;
@@ -557,7 +557,7 @@ private static ImmutableArray<IFieldSymbol> GetEqualizedFields(
             return [];
         }
 
-        return fields.ToImmutable();
+        return fields.ToImmutableAndClear();
     }
 
     /// <summary>

src/Analyzers/CSharp/CodeFixes/ConvertToRecord/PositionalParameterInfo.cs

@@ -68,7 +68,7 @@ public static ImmutableArray<PositionalParameterInfo> GetPropertiesForPositional
                 _ => throw ExceptionUtilities.Unreachable(),
             }).WhereNotNull());
 
-        return resultBuilder.ToImmutable();
+        return resultBuilder.ToImmutableAndClear();
     }
 
     public static ImmutableArray<IPropertySymbol> GetInheritedPositionalParams(

src/Analyzers/CSharp/CodeFixes/MisplacedUsingDirectives/MisplacedUsingDirectivesCodeFixProvider.cs

@@ -117,7 +117,7 @@ private static ImmutableArray<UsingDirectiveSyntax> GetAllUsingDirectives(Compil
 
         Recurse(compilationUnit.Members);
 
-        return result.ToImmutable();
+        return result.ToImmutableAndClear();
 
         void Recurse(SyntaxList<MemberDeclarationSyntax> members)
         {

src/Analyzers/CSharp/CodeFixes/UseSimpleUsingStatement/UseSimpleUsingStatementCodeFixProvider.cs

@@ -104,7 +104,7 @@ private static ImmutableArray<StatementSyntax> Expand(UsingStatementSyntax using
         for (int i = 0, n = result.Count; i < n; i++)
             result[i] = result[i].WithAdditionalAnnotations(Formatter.Annotation);
 
-        return result.ToImmutable();
+        return result.ToImmutableAndClear();
     }
 
     private static SyntaxTriviaList Expand(ArrayBuilder<StatementSyntax> result, UsingStatementSyntax usingStatement)

src/Analyzers/CSharp/Tests/UseCollectionExpression/UseCollectionExpressionForBuilderTests.cs

@@ -49,19 +49,19 @@ public void Clear() { }
         }
         """;
 
-    public static readonly IEnumerable<object[]> FailureCreationPatterns = new[]
-    {
-        new[] {"var builder = ImmutableArray.CreateBuilder<int>();" },
-        new[] {"var builder = ArrayBuilder<int>.GetInstance();" },
-        new[] {"using var _ = ArrayBuilder<int>.GetInstance(out var builder);" },
-    };
-
-    public static readonly IEnumerable<object[]> SuccessCreationPatterns = new[]
-    {
-        new[] {"[|var builder = ImmutableArray.[|CreateBuilder|]<int>();|]" },
-        new[] {"[|var builder = ArrayBuilder<int>.[|GetInstance|]();|]" },
-        new[] {"[|using var _ = ArrayBuilder<int>.[|GetInstance|](out var builder);|]" },
-    };
+    public static readonly IEnumerable<object[]> FailureCreationPatterns =
+    [
+        ["var builder = ImmutableArray.CreateBuilder<int>();"],
+        ["var builder = ArrayBuilder<int>.GetInstance();"],
+        ["using var _ = ArrayBuilder<int>.GetInstance(out var builder);"],
+    ];
+
+    public static readonly IEnumerable<object[]> SuccessCreationPatterns =
+    [
+        ["[|var builder = ImmutableArray.[|CreateBuilder|]<int>();|]"],
+        ["[|var builder = ArrayBuilder<int>.[|GetInstance|]();|]"],
+        ["[|using var _ = ArrayBuilder<int>.[|GetInstance|](out var builder);|]"],
+    ];
 
     [Theory, MemberData(nameof(FailureCreationPatterns))]
     public async Task TestNotInCSharp11(string pattern)

src/Analyzers/Core/Analyzers/UseCollectionInitializer/AbstractObjectCreationExpressionAnalyzer.cs

@@ -84,7 +84,7 @@ protected ImmutableArray<TMatch> AnalyzeWorker(CancellationToken cancellationTok
         if (!TryAddMatches(matches, cancellationToken))
             return default;
 
-        return matches.ToImmutable();
+        return matches.ToImmutableAndClear();
     }
 
     protected UpdateExpressionState<TExpressionSyntax, TStatementSyntax>? TryInitializeState(

src/Analyzers/Core/CodeFixes/AddExplicitCast/Fixer.cs

@@ -73,7 +73,7 @@ protected abstract class Fixer<TArgumentSyntax, TArgumentListSyntax, TInvocation
             // operations are in order and user can choose least specific types(more accurate)
             mutablePotentialConversionTypes.Sort(new InheritanceDistanceComparer<TExpressionSyntax>(semanticModel));
 
-            return mutablePotentialConversionTypes.ToImmutable();
+            return mutablePotentialConversionTypes.ToImmutableAndClear();
         }
 
         /// <summary>

src/Analyzers/Core/CodeFixes/MatchFolderAndNamespace/AbstractChangeNamespaceToMatchFolderCodeFixProvider.CustomFixAllProvider.cs

@@ -63,7 +63,7 @@ static async Task<ImmutableArray<Diagnostic>> GetSolutionDiagnosticsAsync(FixAll
                     diagnostics.AddRange(projectDiagnostics);
                 }
 
-                return diagnostics.ToImmutable();
+                return diagnostics.ToImmutableAndClear();
             }
         }
 

src/EditorFeatures/CSharp/StringCopyPaste/KnownSourcePasteProcessor.cs

@@ -221,7 +221,7 @@ private ImmutableArray<TextChange> DetermineTotalEditsToMakeToRawString(
         if (quotesToAdd != null)
             edits.Add(new TextChange(new TextSpan(StringExpressionBeforePasteInfo.EndDelimiterSpanWithoutSuffix.End, 0), quotesToAdd));
 
-        return edits.ToImmutable();
+        return edits.ToImmutableAndClear();
     }
 
     private void UpdateExistingInterpolationBraces(

src/EditorFeatures/CSharp/StringCopyPaste/UnknownSourcePasteProcessor.cs

@@ -141,7 +141,7 @@ private ImmutableArray<TextChange> GetEditsForRawString()
         if (quotesToAdd != null)
             edits.Add(new TextChange(new TextSpan(StringExpressionBeforePasteInfo.EndDelimiterSpanWithoutSuffix.End, 0), quotesToAdd));
 
-        return edits.ToImmutable();
+        return edits.ToImmutableAndClear();
     }
 
     /// <inheritdoc cref="AbstractPasteProcessor.GetQuotesToAddToRawString(SourceText, ImmutableArray{TextSpan})" />

src/EditorFeatures/Core.Wpf/Peek/PeekableItemFactory.cs

@@ -94,7 +94,7 @@ public async Task<IEnumerable<IPeekableItem>> GetPeekableItemsAsync(
                 }
             }
 
-            return results.ToImmutable();
+            return results.ToImmutableAndClear();
         }
     }
 }

src/EditorFeatures/Core.Wpf/StringIndentation/StringIndentationAdornmentManager.VisibleBlock.cs

@@ -163,7 +163,7 @@ private VisibleBlock(double x, ImmutableArray<(double start, double end)> ySegme
                 if ((currentSegmentBottom - currentSegmentTop) >= MinLineHeight)
                     segments.Add((currentSegmentTop, currentSegmentBottom));
 
-                return segments.ToImmutable();
+                return segments.ToImmutableAndClear();
             }
 
             private static bool IsInHole(ImmutableArray<SnapshotSpan> orderedHoleSpans, ITextViewLine line)

src/EditorFeatures/Core/CodeDefinitionWindow/DefinitionContextTracker.cs

@@ -171,7 +171,7 @@ internal async Task<ImmutableArray<CodeDefinitionWindowLocation>> GetContextFrom
                 }
             }
 
-            return builder.ToImmutable();
+            return builder.ToImmutableAndClear();
         }
 
         // We didn't have regular source references, but possibly:

src/EditorFeatures/Core/Extensibility/NavigationBar/WrappedNavigationBarItem.cs

@@ -39,7 +39,7 @@ private static ImmutableArray<TextSpan> GetSpans(RoslynNavigationBarItem underly
         using var _ = ArrayBuilder<TextSpan>.GetInstance(out var spans);
         AddSpans(underlyingItem, spans);
         spans.SortAndRemoveDuplicates(Comparer<TextSpan>.Default);
-        return spans.ToImmutable();
+        return spans.ToImmutableAndClear();
 
         static void AddSpans(RoslynNavigationBarItem underlyingItem, ArrayBuilder<TextSpan> spans)
         {

src/EditorFeatures/Core/ExternalAccess/IntelliCode/IntentProcessor.cs

@@ -88,7 +88,7 @@ public async Task<ImmutableArray<IntentSource>> ComputeIntentsAsync(IntentReques
             convertedResults.AddIfNotNull(convertedIntent);
         }
 
-        return convertedResults.ToImmutable();
+        return convertedResults.ToImmutableAndClear();
     }
 
     private static async Task<IntentSource?> ConvertToIntelliCodeResultAsync(

src/EditorFeatures/Core/IntelliSense/AsyncCompletion/FilterSet.cs

@@ -191,7 +191,7 @@ public ImmutableArray<CompletionFilterWithState> GetFilterStatesInSet()
             }
         }
 
-        return builder.ToImmutable();
+        return builder.ToImmutableAndClear();
     }
 
     /// <summary>
@@ -218,7 +218,7 @@ public static ImmutableArray<CompletionFilterWithState> CombineFilterStates(Immu
             }
         }
 
-        return builder.ToImmutable();
+        return builder.ToImmutableAndClear();
 
         void AddFilterState(ImmutableArray<CompletionFilterWithState> filterStates)
         {

src/EditorFeatures/Core/ReferenceHighlighting/NavigateToHighlightReferenceCommandHandler.cs

@@ -87,7 +87,7 @@ private static ImmutableArray<SnapshotSpan> GetTags(
             tags.AddRange(tag.Span.GetSpans(span.Snapshot.TextBuffer));
 
         tags.Sort(static (ss1, ss2) => ss1.Start - ss2.Start);
-        return tags.ToImmutable();
+        return tags.ToImmutableAndClear();
     }
 
     private static SnapshotSpan GetDestinationTag(

src/EditorFeatures/Test/CodeGeneration/CodeGenerationTests.cs

@@ -956,7 +956,7 @@ public ImmutableArray<SyntaxNode> ParseStatements(string statements)
                     }
                 }
 
-                return list.ToImmutable();
+                return list.ToImmutableAndClear();
             }
 
             public void Dispose()

src/EditorFeatures/Test/Snippets/RoslynLSPSnippetConvertTests.cs

@@ -515,7 +515,7 @@ private static ImmutableArray<SnippetPlaceholder> GetSnippetPlaceholders(string
                 }
             }
 
-            return arrayBuilder.ToImmutable();
+            return arrayBuilder.ToImmutableAndClear();
         }
     }
 }

src/Features/CSharp/Portable/BraceCompletion/AbstractCurlyBraceOrBracketCompletionService.cs

@@ -200,7 +200,7 @@ static ImmutableArray<TextChange> GetMergedChanges(TextChange? newLineEdit, Immu
                 mergedChanges.Add(new TextChange(newTextChangeSpan, newTextChangeText));
             }
 
-            return mergedChanges.ToImmutable();
+            return mergedChanges.ToImmutableAndClear();
         }
     }
 

src/Features/CSharp/Portable/ChangeSignature/CSharpChangeSignatureService.cs

@@ -763,7 +763,7 @@ private ImmutableArray<T> TransferLeadingWhitespaceTrivia<T, U>(IEnumerable<T> n
             index++;
         }
 
-        return result.ToImmutable();
+        return result.ToImmutableAndClear();
     }
 
     private async ValueTask<ImmutableArray<SyntaxTrivia>> UpdateParamTagsInLeadingTriviaAsync(

src/Features/CSharp/Portable/CodeLens/CSharpCodeLensMemberFinder.cs

@@ -33,7 +33,7 @@ public async Task<ImmutableArray<CodeLensMember>> GetCodeLensMembersAsync(Docume
 
         visitor.Visit(root);
 
-        return codeLensNodes.ToImmutable();
+        return codeLensNodes.ToImmutableAndClear();
     }
 
     private sealed class CSharpCodeLensVisitor(ArrayBuilder<CodeLensMember> memberBuilder) : CSharpSyntaxWalker

src/Features/CSharp/Portable/Completion/CompletionProviders/DeclarationName/DeclarationNameRecommender.cs

@@ -60,7 +60,7 @@ public DeclarationNameRecommender()
             GetRecommendedNames(names, nameInfo, context, result, namingStyleOptions, cancellationToken);
         }
 
-        return result.ToImmutable();
+        return result.ToImmutableAndClear();
     }
 
     private ImmutableArray<ImmutableArray<string>> GetBaseNames(SemanticModel semanticModel, NameDeclarationInfo nameInfo)

src/Features/CSharp/Portable/ConvertProgram/ConvertProgramTransform_ProgramMain.cs

@@ -146,7 +146,7 @@ private static ImmutableArray<StatementSyntax> GenerateProgramMainStatements(
             }
         }
 
-        return statements.ToImmutable();
+        return statements.ToImmutableAndClear();
     }
 
     private static TSyntaxNode FixupComments<TSyntaxNode>(TSyntaxNode node) where TSyntaxNode : SyntaxNode

src/Features/CSharp/Portable/ConvertProgram/ConvertProgramTransform_TopLevelStatements.cs

@@ -231,7 +231,7 @@ private static ImmutableArray<GlobalStatementSyntax> GetGlobalStatements(
         foreach (var statement in statements)
             globalStatements.Add(GlobalStatement(statement).WithAdditionalAnnotations(Formatter.Annotation));
 
-        return globalStatements.ToImmutable();
+        return globalStatements.ToImmutableAndClear();
     }
 
     private static VariableDeclarationSyntax ConvertDeclaration(

src/Features/CSharp/Portable/DocumentHighlighting/CSharpDocumentHighlightsService.cs

@@ -67,6 +67,6 @@ protected override async Task<ImmutableArray<Location>> GetAdditionalReferencesA
             }
         }
 
-        return results.ToImmutable();
+        return results.ToImmutableAndClear();
     }
 }

src/Features/CSharp/Portable/ExtractMethod/CSharpMethodExtractor.CSharpCodeGenerator.MultipleStatementsCodeGenerator.cs

@@ -60,7 +60,7 @@ protected override ImmutableArray<StatementSyntax> GetInitialStatementsForMethod
                     }
                 }
 
-                return list.ToImmutable();
+                return list.ToImmutableAndClear();
             }
 
             private static IEnumerable<StatementSyntax> GetStatementsFromContainer(SyntaxNode node)

src/Features/CSharp/Portable/ExtractMethod/CSharpMethodExtractor.CSharpCodeGenerator.cs

@@ -408,7 +408,7 @@ private ImmutableArray<StatementSyntax> MoveDeclarationOutFromMethodDefinition(
                 result.AddRange(expressionStatements);
             }
 
-            return result.ToImmutable();
+            return result.ToImmutableAndClear();
         }
 
         /// <summary>

src/Features/CSharp/Portable/ExtractMethod/CSharpMethodExtractor.PostProcessor.cs

@@ -108,7 +108,7 @@ private ImmutableArray<StatementSyntax> MergeDeclarationStatementsWorker(Immutab
             if (map.Count > 0)
                 result.AddRange(GetMergedDeclarationStatements(map));
 
-            return result.ToImmutable();
+            return result.ToImmutableAndClear();
         }
 
         private void AppendDeclarationStatementToMap(

src/Features/CSharp/Portable/GenerateMember/GenerateParameterizedMember/CSharpGenerateParameterizedMemberService.cs

@@ -99,7 +99,7 @@ protected override ImmutableArray<ITypeParameterSymbol> GenerateTypeParameters(C
                     list.Add(typeParameter);
                 }
 
-                return list.ToImmutable();
+                return list.ToImmutableAndClear();
             }
         }
 
@@ -158,7 +158,7 @@ protected override ImmutableArray<ITypeSymbol> DetermineTypeArguments(Cancellati
                 }
             }
 
-            return result.ToImmutable();
+            return result.ToImmutableAndClear();
         }
     }
 }

src/Features/CSharp/Portable/LanguageServices/CSharpStructuralTypeDisplayService.cs

@@ -57,6 +57,6 @@ protected override ImmutableArray<SymbolDisplayPart> GetNormalAnonymousTypeParts
         members.AddRange(Space());
         members.Add(Punctuation(SyntaxFacts.GetText(SyntaxKind.CloseBraceToken)));
 
-        return members.ToImmutable();
+        return members.ToImmutableAndClear();
     }
 }

src/Features/CSharp/Portable/LineSeparators/CSharpLineSeparatorService.cs

@@ -64,7 +64,7 @@ public async Task<ImmutableArray<TextSpan>> GetLineSeparatorsAsync(
             }
         }
 
-        return spans.ToImmutable();
+        return spans.ToImmutableAndClear();
     }
 
     /// <summary>Node types that are interesting for line separation.</summary>

src/Features/CSharp/Portable/NavigationBar/CSharpNavigationBarItemService.cs

@@ -108,7 +108,7 @@ private static ImmutableArray<RoslynNavigationBarItem> GetMembersInTypes(
             }
 
             items.Sort((x1, x2) => x1.Text.CompareTo(x2.Text));
-            return items.ToImmutable();
+            return items.ToImmutableAndClear();
         }
     }
 

src/Features/CSharp/Portable/ReplacePropertyWithMethods/CSharpReplacePropertyWithMethodsService.cs

@@ -100,7 +100,7 @@ private static ImmutableArray<SyntaxNode> ConvertPropertyToMembers(
                 cancellationToken));
         }
 
-        return result.ToImmutable();
+        return result.ToImmutableAndClear();
     }
 
     private static SyntaxNode GetSetMethod(

src/Features/CSharp/Portable/Snippets/CSharpForEachLoopSnippetProvider.cs

@@ -113,7 +113,7 @@ protected override ImmutableArray<SnippetPlaceholder> GetPlaceHolderLocationsLis
         if (!ConstructedFromInlineExpression)
             arrayBuilder.Add(new SnippetPlaceholder(node.Expression.ToString(), node.Expression.SpanStart));
 
-        return arrayBuilder.ToImmutable();
+        return arrayBuilder.ToImmutableAndClear();
     }
 
     protected override int GetTargetCaretPosition(ForEachStatementSyntax forEachStatement, SourceText sourceText)

src/Features/CSharp/Portable/StringIndentation/CSharpStringIndentationService.cs

@@ -37,7 +37,7 @@ public async Task<ImmutableArray<StringIndentationRegion>> GetStringIndentationR
 
         Recurse(text, root, textSpan, result, cancellationToken);
 
-        return result.ToImmutable();
+        return result.ToImmutableAndClear();
     }
 
     private static void Recurse(

src/Features/CSharp/Portable/UseExpressionBodyForLambda/UseExpressionBodyForLambdaCodeRefactoringProvider.cs

@@ -194,7 +194,7 @@ private static async Task<ImmutableArray<CodeAction>> ComputeRefactoringsAsync(
                 title));
         }
 
-        return result.ToImmutable();
+        return result.ToImmutableAndClear();
     }
 
     private static async Task<Document> UpdateDocumentAsync(

src/Features/CSharpTest/Diagnostics/Suppression/RemoveUnnecessaryPragmaSuppressionsTests.cs

@@ -170,7 +170,7 @@ protected override ImmutableArray<string> UnsupportedDiagnosticIds
                             }
                         }
 
-                        return builder.ToImmutable();
+                        return builder.ToImmutableAndClear();
                     }
                 }
             }