Remove type check - not needed

SonarSource · Nov 22, 2021 · c729f64 · c729f64
1 parent c3bb22f
commit c729f64
Showing 1 changed file with 0 additions and 8 deletions.
diff --git a/analyzers/src/SonarAnalyzer.CSharp/Rules/UnusedPrivateMember.cs b/analyzers/src/SonarAnalyzer.CSharp/Rules/UnusedPrivateMember.cs
@@ -75,7 +75,6 @@ protected override void Initialize(SonarAnalysisContext context) =>
                             if (namedType.ContainingType != null
                                 // We skip top level statements since they cannot have fields. Other declared types are analyzed separately.
                                 || IsTopLevelProgram(namedType)
-                                || !IsType(namedType)
                                 || namedType.DerivesFromAny(IgnoredTypes))
                             {
                                 return;
@@ -139,13 +138,6 @@ private static bool IsTopLevelProgram(ISymbol symbol) =>
             symbol.Name == "Program"
             && symbol.ContainingNamespace.IsGlobalNamespace;
 
-        private static bool IsType(ITypeSymbol namedType) =>
-            namedType.TypeKind == TypeKind.Struct
-            || namedType.TypeKind == TypeKind.Class
-            || namedType.TypeKind == TypeKind.Delegate
-            || namedType.TypeKind == TypeKind.Enum
-            || namedType.TypeKind == TypeKind.Interface;
-
         private static IEnumerable<Diagnostic> GetDiagnosticsForUnusedPrivateMembers(CSharpSymbolUsageCollector usageCollector,
                                                                                      ISet<ISymbol> removableSymbols,
                                                                                      string accessibility,