From dcbef6e90cdeffc88dce50c2a9caceec24b2a51c Mon Sep 17 00:00:00 2001 From: CyrusNajmabadi Date: Sat, 13 Aug 2016 17:53:48 -0400 Subject: [PATCH] Decouple SyntaxList from SyntaxListBuilder. --- src/Compilers/CSharp/Portable/Parser/Lexer.cs | 10 ++++++---- .../Portable/Syntax/InternalSyntax/SyntaxList.cs | 10 ---------- .../Microsoft.CodeAnalysis.PooledObjects.shproj | 2 +- 3 files changed, 7 insertions(+), 15 deletions(-) diff --git a/src/Compilers/CSharp/Portable/Parser/Lexer.cs b/src/Compilers/CSharp/Portable/Parser/Lexer.cs index 6dd6ca118fa..8bb1783aa8d 100644 --- a/src/Compilers/CSharp/Portable/Parser/Lexer.cs +++ b/src/Compilers/CSharp/Portable/Parser/Lexer.cs @@ -303,22 +303,24 @@ internal SyntaxTriviaList LexSyntaxLeadingTrivia() { _leadingTriviaCache.Clear(); this.LexSyntaxTrivia(afterFirstToken: TextWindow.Position > 0, isTrailing: false, triviaList: ref _leadingTriviaCache); - return new SyntaxTriviaList(default(Microsoft.CodeAnalysis.SyntaxToken), SyntaxList.List(_leadingTriviaCache), 0, 0); + return new SyntaxTriviaList(default(Microsoft.CodeAnalysis.SyntaxToken), + _leadingTriviaCache?.ToListNode(), position: 0, index: 0); } internal SyntaxTriviaList LexSyntaxTrailingTrivia() { _trailingTriviaCache.Clear(); this.LexSyntaxTrivia(afterFirstToken: true, isTrailing: true, triviaList: ref _trailingTriviaCache); - return new SyntaxTriviaList(default(Microsoft.CodeAnalysis.SyntaxToken), SyntaxList.List(_trailingTriviaCache), 0, 0); + return new SyntaxTriviaList(default(Microsoft.CodeAnalysis.SyntaxToken), + _trailingTriviaCache?.ToListNode(), position: 0, index: 0); } private SyntaxToken Create(ref TokenInfo info, SyntaxListBuilder leading, SyntaxListBuilder trailing, SyntaxDiagnosticInfo[] errors) { Debug.Assert(info.Kind != SyntaxKind.IdentifierToken || info.StringValue != null); - var leadingNode = SyntaxList.List(leading); - var trailingNode = SyntaxList.List(trailing); + var leadingNode = leading?.ToListNode(); + var trailingNode = trailing?.ToListNode(); SyntaxToken token; if (info.RequiresTextForXmlEntity) diff --git a/src/Compilers/CSharp/Portable/Syntax/InternalSyntax/SyntaxList.cs b/src/Compilers/CSharp/Portable/Syntax/InternalSyntax/SyntaxList.cs index 3145f61cc48..2b67d375a09 100644 --- a/src/Compilers/CSharp/Portable/Syntax/InternalSyntax/SyntaxList.cs +++ b/src/Compilers/CSharp/Portable/Syntax/InternalSyntax/SyntaxList.cs @@ -96,16 +96,6 @@ internal static SyntaxList List(ArrayElement[] children) } } - internal static CSharpSyntaxNode List(SyntaxListBuilder builder) - { - if (builder != null) - { - return builder.ToListNode(); - } - - return null; - } - internal abstract void CopyTo(ArrayElement[] array, int offset); internal static CSharpSyntaxNode Concat(CSharpSyntaxNode left, CSharpSyntaxNode right) diff --git a/src/Dependencies/PooledObjects/Microsoft.CodeAnalysis.PooledObjects.shproj b/src/Dependencies/PooledObjects/Microsoft.CodeAnalysis.PooledObjects.shproj index d00c5194a1a..6274cf7f176 100644 --- a/src/Dependencies/PooledObjects/Microsoft.CodeAnalysis.PooledObjects.shproj +++ b/src/Dependencies/PooledObjects/Microsoft.CodeAnalysis.PooledObjects.shproj @@ -1,7 +1,7 @@  - cd77a8cc-2885-47a7-b99c-fbf13353400b + c1930979-c824-496b-a630-70f5369a636f 14.0 -- GitLab