From 5f85138bf04c274522950109bca3d784a6c5b484 Mon Sep 17 00:00:00 2001 From: Rekkonnect Date: Wed, 25 Dec 2024 04:34:17 +0200 Subject: [PATCH] Fix #90 round 2 --- .../Core/DisplayAnalysis/BaseSyntaxAnalysisNodeCreator.cs | 2 +- .../Core/DisplayAnalysis/CSharpSyntaxAnalysisNodeCreator.cs | 6 +++--- .../DisplayAnalysis/VisualBasicSyntaxAnalysisNodeCreator.cs | 6 +++--- Syndiesis/Core/NodeViewAnalysisExecution.cs | 2 +- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/Syndiesis/Core/DisplayAnalysis/BaseSyntaxAnalysisNodeCreator.cs b/Syndiesis/Core/DisplayAnalysis/BaseSyntaxAnalysisNodeCreator.cs index c377508..1a354a7 100644 --- a/Syndiesis/Core/DisplayAnalysis/BaseSyntaxAnalysisNodeCreator.cs +++ b/Syndiesis/Core/DisplayAnalysis/BaseSyntaxAnalysisNodeCreator.cs @@ -103,7 +103,7 @@ public abstract AnalysisTreeListNode CreateRootNodeList( ; public abstract AnalysisTreeListNode CreateRootTokenList( - SyntaxTokenList list, TDisplayValueSource? valueSource, bool includeChildren = true) + IReadOnlyList list, TDisplayValueSource? valueSource, bool includeChildren = true) where TDisplayValueSource : IDisplayValueSource ; diff --git a/Syndiesis/Core/DisplayAnalysis/CSharpSyntaxAnalysisNodeCreator.cs b/Syndiesis/Core/DisplayAnalysis/CSharpSyntaxAnalysisNodeCreator.cs index 59a979d..926dd15 100644 --- a/Syndiesis/Core/DisplayAnalysis/CSharpSyntaxAnalysisNodeCreator.cs +++ b/Syndiesis/Core/DisplayAnalysis/CSharpSyntaxAnalysisNodeCreator.cs @@ -9,6 +9,7 @@ using System; using System.Collections.Generic; using System.Diagnostics; +using System.Diagnostics.Contracts; using System.Linq; namespace Syndiesis.Core.DisplayAnalysis; @@ -20,6 +21,7 @@ namespace Syndiesis.Core.DisplayAnalysis; using ComplexGroupedRunInline = ComplexGroupedRunInline.Builder; using ReadOnlySyntaxNodeList = IReadOnlyList; +using SyntaxTokenList = IReadOnlyList; public sealed partial class CSharpSyntaxAnalysisNodeCreator : BaseSyntaxAnalysisNodeCreator { @@ -947,9 +949,7 @@ private static string EndOfLineTriviaText(SyntaxTrivia trivia) private static string DisabledTextTriviaText(SyntaxTrivia trivia) { - // https://github.com/dotnet/roslyn/issues/76441 - if (trivia.SyntaxTree is null) - return "[Unknown line bounds]"; + Contract.Assert(trivia.SyntaxTree is not null); var span = trivia.Span; var lineSpan = trivia.SyntaxTree.GetLineSpan(span).Span; diff --git a/Syndiesis/Core/DisplayAnalysis/VisualBasicSyntaxAnalysisNodeCreator.cs b/Syndiesis/Core/DisplayAnalysis/VisualBasicSyntaxAnalysisNodeCreator.cs index 06f7c58..1c4de2d 100644 --- a/Syndiesis/Core/DisplayAnalysis/VisualBasicSyntaxAnalysisNodeCreator.cs +++ b/Syndiesis/Core/DisplayAnalysis/VisualBasicSyntaxAnalysisNodeCreator.cs @@ -10,6 +10,7 @@ using System.Diagnostics; using System.Linq; using Garyon.Reflection; +using System.Diagnostics.Contracts; namespace Syndiesis.Core.DisplayAnalysis; @@ -20,6 +21,7 @@ namespace Syndiesis.Core.DisplayAnalysis; using ComplexGroupedRunInline = ComplexGroupedRunInline.Builder; using ReadOnlySyntaxNodeList = IReadOnlyList; +using SyntaxTokenList = IReadOnlyList; public sealed partial class VisualBasicSyntaxAnalysisNodeCreator : BaseSyntaxAnalysisNodeCreator { @@ -953,9 +955,7 @@ private static string EndOfLineTriviaText(SyntaxTrivia trivia) private static string DisabledTextTriviaText(SyntaxTrivia trivia) { - // https://github.com/dotnet/roslyn/issues/76441 - if (trivia.SyntaxTree is null) - return "[Unknown line bounds]"; + Contract.Assert(trivia.SyntaxTree is not null); var span = trivia.Span; var lineSpan = trivia.SyntaxTree.GetLineSpan(span).Span; diff --git a/Syndiesis/Core/NodeViewAnalysisExecution.cs b/Syndiesis/Core/NodeViewAnalysisExecution.cs index 62d1761..b44fc43 100644 --- a/Syndiesis/Core/NodeViewAnalysisExecution.cs +++ b/Syndiesis/Core/NodeViewAnalysisExecution.cs @@ -236,7 +236,7 @@ private static ComplexDisplayValueSource ConstructSemanticModelValueSource( return null; return _container.SyntaxCreator.CreateRootTokenList( - new SyntaxTokenList(_node?.ChildTokens() ?? []), + _node?.ChildTokens().ToList() ?? [], _childTokensValueSource); }