Create SyntaxKindEx helper

dotnet · Apr 5, 2024 · 5705f38 · 5705f38
1 parent 33417fb
commit 5705f38
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 9 deletions.
diff --git a/src/NetAnalyzers/CSharp/Microsoft.NetCore.Analyzers/Performance/CSharpUseSearchValues.cs b/src/NetAnalyzers/CSharp/Microsoft.NetCore.Analyzers/Performance/CSharpUseSearchValues.cs
@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the MIT license.  See License.txt in the project root for license information.
 
 using System.Collections.Generic;
+using Analyzer.Utilities.Lightup;
 using Microsoft.CodeAnalysis;
 using Microsoft.CodeAnalysis.CSharp;
 using Microsoft.CodeAnalysis.CSharp.Syntax;
@@ -14,12 +15,6 @@ namespace Microsoft.NetCore.CSharp.Analyzers.Performance
     [DiagnosticAnalyzer(LanguageNames.CSharp)]
     public sealed class CSharpUseSearchValuesAnalyzer : UseSearchValuesAnalyzer
     {
-        // The referenced SDK version doesn't yet contain these SyntaxKind values
-        // https://github.com/dotnet/roslyn/blob/main/src/Compilers/CSharp/Portable/Syntax/SyntaxKind.cs
-        private const SyntaxKind Utf8StringLiteralToken = (SyntaxKind)8520;
-        private const SyntaxKind Utf8StringLiteralExpression = (SyntaxKind)8756;
-        private const SyntaxKind CollectionExpression = (SyntaxKind)9076;
-
         // char[] myField = new char[] { 'a', 'b', 'c' };
         // char[] myField = new[] { 'a', 'b', 'c' };
         // char[] myField = "abc".ToCharArray();
@@ -118,7 +113,7 @@ internal static bool IsConstantByteOrCharArrayCreationExpression(SemanticModel s
                     return true;
                 }
             }
-            else if (expression.IsKind(CollectionExpression))
+            else if (expression.IsKind(SyntaxKindEx.CollectionExpression))
             {
                 return
                     semanticModel.GetOperation(expression) is { } operation &&
@@ -171,9 +166,9 @@ static bool TryGetByteOrCharLiteral(ExpressionSyntax? expression, out char value
 
         private static bool IsUtf8StringLiteralExpression(ExpressionSyntax expression, out int length)
         {
-            if (expression.IsKind(Utf8StringLiteralExpression) &&
+            if (expression.IsKind(SyntaxKindEx.Utf8StringLiteralExpression) &&
                 expression is LiteralExpressionSyntax literal &&
-                literal.Token.IsKind(Utf8StringLiteralToken) &&
+                literal.Token.IsKind(SyntaxKindEx.Utf8StringLiteralToken) &&
                 literal.Token.Value is string value)
             {
                 length = value.Length;

diff --git a/src/Utilities/Compiler.CSharp/Analyzer.CSharp.Utilities.projitems b/src/Utilities/Compiler.CSharp/Analyzer.CSharp.Utilities.projitems
@@ -10,5 +10,6 @@
   </PropertyGroup>
   <ItemGroup>
     <Compile Include="$(MSBuildThisFileDirectory)Extensions\SyntaxNodeExtensions.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)Lightup\SyntaxKindEx.cs" />
   </ItemGroup>
 </Project>
diff --git a/src/Utilities/Compiler.CSharp/Lightup/SyntaxKindEx.cs b/src/Utilities/Compiler.CSharp/Lightup/SyntaxKindEx.cs
@@ -0,0 +1,14 @@
+// Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the MIT license.  See License.txt in the project root for license information.
+
+using Microsoft.CodeAnalysis.CSharp;
+
+namespace Analyzer.Utilities.Lightup
+{
+    internal static class SyntaxKindEx
+    {
+        // https://github.com/dotnet/roslyn/blob/main/src/Compilers/CSharp/Portable/Syntax/SyntaxKind.cs
+        public const SyntaxKind Utf8StringLiteralToken = (SyntaxKind)8520;
+        public const SyntaxKind Utf8StringLiteralExpression = (SyntaxKind)8756;
+        public const SyntaxKind CollectionExpression = (SyntaxKind)9076;
+    }
+}