diff options
| author | Mike Krüger <mkrueger@xamarin.com> | 2015-12-03 20:00:36 +0300 |
|---|---|---|
| committer | Mike Krüger <mkrueger@xamarin.com> | 2015-12-03 20:00:36 +0300 |
| commit | a2b55de351be2119b6f0c3a17c36b5b9adbd7c59 (patch) | |
| tree | 321f878b33329ca101e0233b9a280cb86c2ba4f0 | |
| parent | 07ee3cddfc192daa67238811d140f7e5aa226fb9 (diff) | |
Fixed unit test.
| -rw-r--r-- | ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/RedundantAssignmentIssue.cs | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/RedundantAssignmentIssue.cs b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/RedundantAssignmentIssue.cs index 77a6c943..ce497098 100644 --- a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/RedundantAssignmentIssue.cs +++ b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/RedundantAssignmentIssue.cs @@ -441,16 +441,17 @@ namespace ICSharpCode.NRefactory.CSharp.Refactoring void ProcessNode(VariableReferenceNode node, bool addIssue, IDictionary<VariableReferenceNode, NodeState> nodeStates) { + if (!visitedNodes.Add (node)) + return; + if (nodeStates [node] == NodeState.None) nodeStates [node] = NodeState.Processing; bool? reachable = false; foreach (var nextNode in node.NextNodes) { - if (!visitedNodes.Add (nextNode)) - continue; - if (nodeStates [nextNode] == NodeState.None) - ProcessNode(nextNode, false, nodeStates); - + if (nodeStates [nextNode] == NodeState.None) { + ProcessNode (nextNode, false, nodeStates); + } if (nodeStates [nextNode] == NodeState.UsageReachable) { reachable = true; break; |