Use ArrayDeque in StringSubstitutionEngine instead of Stack

Change-Id: I65b227abde82a39cde30834960344c9930663d20
Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/org.eclipse.core.variables/src/org/eclipse/core/internal/variables/StringSubstitutionEngine.java b/org.eclipse.core.variables/src/org/eclipse/core/internal/variables/StringSubstitutionEngine.java
index 774c50d69..55ac7c851 100644
--- a/org.eclipse.core.variables/src/org/eclipse/core/internal/variables/StringSubstitutionEngine.java
+++ b/org.eclipse.core.variables/src/org/eclipse/core/internal/variables/StringSubstitutionEngine.java
@@ -13,10 +13,11 @@
  *******************************************************************************/
 package org.eclipse.core.internal.variables;
 
+import java.util.ArrayDeque;
 import java.util.ArrayList;
+import java.util.Deque;
 import java.util.HashSet;
 import java.util.List;
-import java.util.Stack;
 
 import org.eclipse.core.runtime.CoreException;
 import org.eclipse.core.runtime.IStatus;
@@ -53,7 +54,7 @@ public class StringSubstitutionEngine {
 	/**
 	 * Stack of variables to resolve
 	 */
-	private Stack<VariableReference> fStack;
+	private Deque<VariableReference> fStack;
 
 	class VariableReference {
 
@@ -138,7 +139,7 @@ public class StringSubstitutionEngine {
 	 */
 	private HashSet<String> substitute(String expression, boolean reportUndefinedVariables, boolean resolveVariables, IStringVariableManager manager) throws CoreException {
 		fResult = new StringBuilder(expression.length());
-		fStack = new Stack<>();
+		fStack = new ArrayDeque<>();
 		fSubs = false;
 
 		HashSet<String> resolvedVariables = new HashSet<>();