Skip to content

Commit 1af96a9

Browse files
committed
Switch to extension method for GetIdentifierReferences.
1 parent 67561cb commit 1af96a9

File tree

1 file changed

+1
-31
lines changed

1 file changed

+1
-31
lines changed

Rubberduck.CodeAnalysis/Inspections/Concrete/AssignmentNotUsedInspection.cs

Lines changed: 1 addition & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
using Rubberduck.Parsing.VBA;
55
using Rubberduck.Inspections.CodePathAnalysis;
66
using Rubberduck.Parsing.Symbols;
7-
using Rubberduck.Inspections.CodePathAnalysis.Nodes;
87
using Rubberduck.Inspections.CodePathAnalysis.Extensions;
98
using System.Linq;
109
using Rubberduck.Inspections.Results;
@@ -29,41 +28,12 @@ protected override IEnumerable<IInspectionResult> DoGetInspectionResults()
2928
{
3029
var tree = _walker.GenerateTree(variable.ParentScopeDeclaration.Context, variable);
3130

32-
nodes.AddRange(GetIdentifierReferences(tree, variable));
31+
nodes.AddRange(tree.GetIdentifierReferences());
3332
}
3433

3534
return nodes
3635
.Select(issue => new IdentifierReferenceInspectionResult(this, Description, State, issue))
3736
.ToList();
3837
}
39-
40-
private List<IdentifierReference> GetIdentifierReferences(INode node, Declaration declaration)
41-
{
42-
var nodes = new List<IdentifierReference>();
43-
44-
var blockNodes = node.GetNodes(new[] { typeof(BlockNode) });
45-
foreach (var block in blockNodes)
46-
{
47-
INode lastNode = default;
48-
foreach (var flattenedNode in block.GetFlattenedNodes(new[] { typeof(GenericNode), typeof(BlockNode) }))
49-
{
50-
if (flattenedNode is AssignmentNode &&
51-
lastNode is AssignmentNode)
52-
{
53-
nodes.Add(lastNode.Reference);
54-
}
55-
56-
lastNode = flattenedNode;
57-
}
58-
59-
if (lastNode is AssignmentNode &&
60-
block.Children[0].GetFirstNode(new[] { typeof(GenericNode) }) is DeclarationNode)
61-
{
62-
nodes.Add(lastNode.Reference);
63-
}
64-
}
65-
66-
return nodes;
67-
}
6838
}
6939
}

0 commit comments

Comments
 (0)