Skip to content

Commit e6055ae

Browse files
committed
Use ImmutableArray<T>.Builder instead of ArrayBuilder<T>
1 parent f1b1761 commit e6055ae

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

StyleCop.Analyzers/StyleCop.Analyzers.PrivateCodeFixes/Helpers/AbstractRefactoringHelpers`3.cs

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
using System.Threading;
1616
using System.Threading.Tasks;
1717
using Analyzer.Utilities.Extensions;
18-
using Analyzer.Utilities.PooledObjects;
1918
using Microsoft.CodeAnalysis;
2019
using Microsoft.CodeAnalysis.Text;
2120

@@ -59,7 +58,7 @@ public async Task<ImmutableArray<TSyntaxNode>> GetRelevantNodesAsync<TSyntaxNode
5958
return ImmutableArray<TSyntaxNode>.Empty;
6059
}
6160

62-
var relevantNodesBuilder = ArrayBuilder<TSyntaxNode>.GetInstance();
61+
var relevantNodesBuilder = ImmutableArray.CreateBuilder<TSyntaxNode>();
6362

6463
// Every time a Node is considered an extractNodes method is called to add all nodes around the original one
6564
// that should also be considered.
@@ -130,7 +129,7 @@ public async Task<ImmutableArray<TSyntaxNode>> GetRelevantNodesAsync<TSyntaxNode
130129
}
131130
}
132131

133-
return relevantNodesBuilder.ToImmutableAndFree();
132+
return relevantNodesBuilder.ToImmutable();
134133
}
135134

136135
private static bool IsWantedTypeExpressionLike<TSyntaxNode>() where TSyntaxNode : SyntaxNode
@@ -235,7 +234,7 @@ static bool IsAcceptableLineDistanceAway(
235234
}
236235
}
237236

238-
private void AddNodesForTokenToLeft<TSyntaxNode>(ISyntaxFacts syntaxFacts, ArrayBuilder<TSyntaxNode> relevantNodesBuilder, int location, SyntaxToken tokenToLeft, CancellationToken cancellationToken) where TSyntaxNode : SyntaxNode
237+
private void AddNodesForTokenToLeft<TSyntaxNode>(ISyntaxFacts syntaxFacts, ImmutableArray<TSyntaxNode>.Builder relevantNodesBuilder, int location, SyntaxToken tokenToLeft, CancellationToken cancellationToken) where TSyntaxNode : SyntaxNode
239238
{
240239
// there could be multiple (n) tokens to the left if first n-1 are Empty -> iterate over all of them
241240
while (tokenToLeft != default)
@@ -262,7 +261,7 @@ private void AddNodesForTokenToLeft<TSyntaxNode>(ISyntaxFacts syntaxFacts, Array
262261
}
263262
}
264263

265-
private void AddNodesForTokenToRightOrIn<TSyntaxNode>(ISyntaxFacts syntaxFacts, SyntaxNode root, ArrayBuilder<TSyntaxNode> relevantNodesBuilder, int location, SyntaxToken tokenToRightOrIn, CancellationToken cancellationToken) where TSyntaxNode : SyntaxNode
264+
private void AddNodesForTokenToRightOrIn<TSyntaxNode>(ISyntaxFacts syntaxFacts, SyntaxNode root, ImmutableArray<TSyntaxNode>.Builder relevantNodesBuilder, int location, SyntaxToken tokenToRightOrIn, CancellationToken cancellationToken) where TSyntaxNode : SyntaxNode
266265
{
267266
if (tokenToRightOrIn != default)
268267
{
@@ -300,7 +299,7 @@ private void AddNodesForTokenToRightOrIn<TSyntaxNode>(ISyntaxFacts syntaxFacts,
300299
}
301300
}
302301

303-
private void AddRelevantNodesForSelection<TSyntaxNode>(ISyntaxFacts syntaxFacts, SyntaxNode root, TextSpan selectionTrimmed, ArrayBuilder<TSyntaxNode> relevantNodesBuilder, CancellationToken cancellationToken)
302+
private void AddRelevantNodesForSelection<TSyntaxNode>(ISyntaxFacts syntaxFacts, SyntaxNode root, TextSpan selectionTrimmed, ImmutableArray<TSyntaxNode>.Builder relevantNodesBuilder, CancellationToken cancellationToken)
304303
where TSyntaxNode : SyntaxNode
305304
{
306305
var selectionNode = root.FindNode(selectionTrimmed, getInnermostNodeForTie: true);
@@ -479,7 +478,7 @@ protected virtual IEnumerable<SyntaxNode> ExtractNodesInHeader(SyntaxNode root,
479478

480479
protected virtual async Task AddNodesDeepInAsync<TSyntaxNode>(
481480
Document document, int position,
482-
ArrayBuilder<TSyntaxNode> relevantNodesBuilder,
481+
ImmutableArray<TSyntaxNode>.Builder relevantNodesBuilder,
483482
CancellationToken cancellationToken) where TSyntaxNode : SyntaxNode
484483
{
485484
// If we're deep inside we don't have to deal with being on edges (that gets dealt by TryGetSelectedNodeAsync)
@@ -514,7 +513,7 @@ protected virtual async Task AddNodesDeepInAsync<TSyntaxNode>(
514513
}
515514
}
516515

517-
private static void AddNonHiddenCorrectTypeNodes<TSyntaxNode>(IEnumerable<SyntaxNode> nodes, ArrayBuilder<TSyntaxNode> resultBuilder, CancellationToken cancellationToken)
516+
private static void AddNonHiddenCorrectTypeNodes<TSyntaxNode>(IEnumerable<SyntaxNode> nodes, ImmutableArray<TSyntaxNode>.Builder resultBuilder, CancellationToken cancellationToken)
518517
where TSyntaxNode : SyntaxNode
519518
{
520519
var correctTypeNonHiddenNodes = nodes.OfType<TSyntaxNode>().Where(n => !n.OverlapsHiddenPosition(cancellationToken));

0 commit comments

Comments
 (0)