Merge pull request #5541 from BZngr/UnusedAssignment_FalsePositives

AssignmentNotUsedInspection false positives - incorrect variable scoping and others

Author: MDoerner

.gitignore

@@ -186,6 +186,7 @@ CodeGraphData/
 
 # Generated Artifacts
 Rubberduck.CodeAnalysis.xml
+Rubberduck.Parsing.xml
 
 #Gradle 
 /.gradle/

Rubberduck.CodeAnalysis/CodePathAnalysis/Extensions/NodeExtensions.cs

@@ -55,5 +55,21 @@ public static INode GetFirstNode(this INode node, ICollection<Type> excludedType
 
             return GetFirstNode(node.Children[0], excludedTypes);
         }
+
+        public static bool TryGetAncestorNode<T>(this INode node, out T ancestor) where T: INode
+        {
+            ancestor = default;
+            if (node.Parent is null)
+            {
+                return false;
+            }
+
+            if (node.Parent is T result)
+            {
+                ancestor = result;
+                return true;
+            }
+            return TryGetAncestorNode(node.Parent, out ancestor);
+        }
     }
 }

Rubberduck.CodeAnalysis/CodePathAnalysis/Nodes/Implementations/AssignmentExpressionNode.cs

@@ -0,0 +1,9 @@
+using Antlr4.Runtime.Tree;
+
+namespace Rubberduck.Inspections.CodePathAnalysis.Nodes
+{
+    public class AssignmentExpressionNode : NodeBase
+    {
+        public AssignmentExpressionNode(IParseTree tree) : base(tree) { }
+    }
+}

Rubberduck.CodeAnalysis/CodePathAnalysis/Walker.cs

@@ -33,6 +33,10 @@ public INode GenerateTree(IParseTree tree, Declaration declaration)
                 case VBAParser.BlockContext _:
                     node = new BlockNode(tree);
                     break;
+                case VBAParser.LetStmtContext _:
+                case VBAParser.SetStmtContext _:
+                    node = new AssignmentExpressionNode(tree);
+                    break;
             }
 
             if (declaration.Context == tree)