BUGFIX: Handle empty setter body.

#291
DotNetAnalyzers · Nov 20, 2020 · 0e4ac71 · 0e4ac71
1 parent 6f3016a
commit 0e4ac71
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/ValidCode/Repro/Issue291.cs b/ValidCode/Repro/Issue291.cs
@@ -0,0 +1,13 @@
+namespace ValidCode.Repro
+{
+    using System.Windows.Controls;
+
+    public class Issue291 : Control
+    {
+        public bool IsAcknowledged
+        {
+            get { return false; }
+            set { }
+        }
+    }
+}
diff --git a/WpfAnalyzers/Helpers/DependencyObject.cs b/WpfAnalyzers/Helpers/DependencyObject.cs
@@ -274,7 +274,7 @@ private SetCurrentValue(InvocationExpressionSyntax invocation, IMethodSymbol tar
                         => Match(invocation, semanticModel, cancellationToken),
                     { ExpressionBody: { } expressionBody } => Walk(expressionBody),
                     { Body: { Statements: { } statements } }
-                        when statements.Last() is ReturnStatementSyntax { Expression: InvocationExpressionSyntax invocation }
+                        when statements.LastOrDefault() is ReturnStatementSyntax { Expression: InvocationExpressionSyntax invocation }
                         => Match(invocation, semanticModel, cancellationToken),
                     { Body: { } body } => Walk(body),
                     _ => null,