Dispose some pooled object instances

We Dispose()/Free() other instances of these classes, but not these specific ones. Fix that.
dotnet · Jun 14, 2024 · b054267 · b054267
1 parent 52913b2
commit b054267
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/...t.CodeQuality.Analyzers/ApiDesignGuidelines/CollectionsShouldImplementGenericInterface.cs b/...t.CodeQuality.Analyzers/ApiDesignGuidelines/CollectionsShouldImplementGenericInterface.cs
@@ -68,7 +68,7 @@ public override void Initialize(AnalysisContext context)
 
                    if (wellKnownTypeProvider.TryGetOrCreateTypeByMetadataName(WellKnownTypeNames.SystemCollectionsIList, out var iListType))
                    {
-                       var builder = ArrayBuilder<INamedTypeSymbol>.GetInstance();
+                       using var builder = ArrayBuilder<INamedTypeSymbol>.GetInstance();
                        builder.AddIfNotNull(wellKnownTypeProvider.GetOrCreateTypeByMetadataName(WellKnownTypeNames.SystemCollectionsGenericIList1));
                        builder.AddIfNotNull(wellKnownTypeProvider.GetOrCreateTypeByMetadataName(WellKnownTypeNames.SystemCollectionsGenericIReadOnlyList1));
                        if (builder.Count > 0)
@@ -79,7 +79,7 @@ public override void Initialize(AnalysisContext context)
 
                    if (wellKnownTypeProvider.TryGetOrCreateTypeByMetadataName(WellKnownTypeNames.SystemCollectionsICollection, out var iCollectionType))
                    {
-                       var builder = ArrayBuilder<INamedTypeSymbol>.GetInstance();
+                       using var builder = ArrayBuilder<INamedTypeSymbol>.GetInstance();
                        builder.AddIfNotNull(wellKnownTypeProvider.GetOrCreateTypeByMetadataName(WellKnownTypeNames.SystemCollectionsGenericICollection1));
                        builder.AddIfNotNull(wellKnownTypeProvider.GetOrCreateTypeByMetadataName(WellKnownTypeNames.SystemCollectionsGenericIReadOnlyCollection1));
                        if (builder.Count > 0)

diff --git a/...alyzers/Core/Microsoft.NetCore.Analyzers/InteropServices/PlatformCompatibilityAnalyzer.cs b/...alyzers/Core/Microsoft.NetCore.Analyzers/InteropServices/PlatformCompatibilityAnalyzer.cs
@@ -1822,7 +1822,7 @@ static void MergePlatformAttributes(ImmutableArray<AttributeData> immediateAttri
                 var pAttributes = parentAttributes.Platforms;
                 if (pAttributes != null && !pAttributes.IsEmpty)
                 {
-                    var notFoundPlatforms = PooledHashSet<string>.GetInstance();
+                    using var notFoundPlatforms = PooledHashSet<string>.GetInstance();
                     bool supportFound = false;
                     foreach (var (platform, attributes) in pAttributes)
                     {

diff --git a/...ties/FlowAnalysis/FlowAnalysis/Analysis/ValueContentAnalysis/ValueContentAbstractValue.cs b/...ties/FlowAnalysis/FlowAnalysis/Analysis/ValueContentAnalysis/ValueContentAbstractValue.cs
@@ -261,7 +261,7 @@ internal ValueContentAbstractValue MergeBinaryOperation(
             }
 
             // Merge Literals
-            var builder = PooledHashSet<object?>.GetInstance();
+            using var builder = PooledHashSet<object?>.GetInstance();
             foreach (var leftLiteral in LiteralValues)
             {
                 foreach (var rightLiteral in otherState.LiteralValues)
@@ -275,7 +275,7 @@ internal ValueContentAbstractValue MergeBinaryOperation(
                 }
             }
 
-            ImmutableHashSet<object?> mergedLiteralValues = builder.ToImmutableAndFree();
+            ImmutableHashSet<object?> mergedLiteralValues = builder.ToImmutable();
             ValueContainsNonLiteralState mergedNonLiteralState = Merge(NonLiteralState, otherState.NonLiteralState);
 
             return Create(mergedLiteralValues, mergedNonLiteralState);