Update src/Features/CSharp/Portable/Copilot/CSharpImplementNotImplementedExceptionDiagnosticAnalyzer.cs

maryamariyan · CyrusNajmabadi · web-flow · commit 67c82fc33f18 · 2025-03-26T16:33:03.000-07:00
Co-authored-by: Cyrus Najmabadi &lt;cyrus.najmabadi@gmail.com&gt;
diff --git a/src/Features/CSharp/Portable/Copilot/CSharpImplementNotImplementedExceptionDiagnosticAnalyzer.cs b/src/Features/CSharp/Portable/Copilot/CSharpImplementNotImplementedExceptionDiagnosticAnalyzer.cs
@@ -103,7 +103,7 @@ private static bool ShouldReportAsTopLevel(IOperation block, IOperation operatio
                 // Excluding property declarations with expression bodies
                 // Because their location is an exact match with the throw operation
                 // and we don't want to report the same location twice
-                !(operation.Syntax.Parent is ArrowExpressionClauseSyntax { Parent: PropertyDeclarationSyntax }))
+                operation.Syntax.Parent is not ArrowExpressionClauseSyntax { Parent: PropertyDeclarationSyntax })
             {
                 // Include expression-bodied methods and get accessors
                 return true;

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ private static bool ShouldReportAsTopLevel(IOperation block, IOperation operatio`
`103`	`103`	`// Excluding property declarations with expression bodies`
`104`	`104`	`// Because their location is an exact match with the throw operation`
`105`	`105`	`// and we don't want to report the same location twice`
`106`		`- !(operation.Syntax.Parent is ArrowExpressionClauseSyntax { Parent: PropertyDeclarationSyntax }))`
	`106`	`+ operation.Syntax.Parent is not ArrowExpressionClauseSyntax { Parent: PropertyDeclarationSyntax })`
`107`	`107`	`{`
`108`	`108`	`// Include expression-bodied methods and get accessors`
`109`	`109`	`return true;`