From 14b601893f0bbec41b10ef9af70027028361346a Mon Sep 17 00:00:00 2001 From: almaraubel Date: Sat, 30 Mar 2024 13:54:13 +0100 Subject: [PATCH] Comment out unimplemented switch case in ResultValueWalker The code previously threw a NotImplementedException for the SwitchExpressionSyntax case in the ResultValueWalker. This change comments out that case for now, to prevent the exception from being triggered. It appears the functionality for SwitchExpressionSyntax is not implemented yet. --- .../UseResultValueWithoutCheck/ResultValueWalker.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CSharpFunctionalExtensions.Analyzers/UseResultValueWithoutCheck/ResultValueWalker.cs b/CSharpFunctionalExtensions.Analyzers/UseResultValueWithoutCheck/ResultValueWalker.cs index 5659aae..58fae1e 100644 --- a/CSharpFunctionalExtensions.Analyzers/UseResultValueWithoutCheck/ResultValueWalker.cs +++ b/CSharpFunctionalExtensions.Analyzers/UseResultValueWithoutCheck/ResultValueWalker.cs @@ -184,8 +184,8 @@ private CheckResult DetermineCheckResult(ExpressionSyntax condition) { RecursivePatternSyntax recursivePatternSyntax => CheckedRecusivePattern(recursivePatternSyntax), }; - case SwitchExpressionSyntax switchExpressionSyntax: - throw new NotImplementedException(); + // case SwitchExpressionSyntax switchExpressionSyntax: + // throw new NotImplementedException(); } return CheckResult.Unchecked;