Merge pull request #782 from StefanMaron/ResolveNullReferenceException

Resolve NullReferenceException on Rule0024
StefanMaron · Oct 7, 2024 · 44d7f7e · 44d7f7e
2 parents 84d6c4c + 544f812
commit 44d7f7e
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/BusinessCentral.LinterCop/Design/Rule0024SemicolonAfterProcedureDeclaration.cs b/BusinessCentral.LinterCop/Design/Rule0024SemicolonAfterProcedureDeclaration.cs
@@ -1,5 +1,4 @@
-#nullable disable // TODO: Enable nullable and review rule
-using BusinessCentral.LinterCop.AnalysisContextExtension;
+using BusinessCentral.LinterCop.AnalysisContextExtension;
 using Microsoft.Dynamics.Nav.CodeAnalysis;
 using Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics;
 using Microsoft.Dynamics.Nav.CodeAnalysis.Syntax;
@@ -16,10 +15,11 @@ public class Rule0024SemicolonAfterMethodOrTriggerDeclaration : DiagnosticAnalyz
 
         private void AnalyzeSemicolonAfterMethodOrTriggerDeclaration(SyntaxNodeAnalysisContext ctx)
         {
-            if (ctx.IsObsoletePendingOrRemoved()) return;
+            if (ctx.IsObsoletePendingOrRemoved())
+                return;
 
-            MethodOrTriggerDeclarationSyntax syntax = ctx.Node as MethodOrTriggerDeclarationSyntax;
-            if (syntax == null) return;
+            if (ctx.Node is not MethodOrTriggerDeclarationSyntax syntax)
+                return;
 
             if (syntax.SemicolonToken.Kind != SyntaxKind.None)
             {