Permalink
Browse files

Non-sprint task - fix for issue http://bugs.eclipse.org/396763 review…

…ed and applied.
  • Loading branch information...
1 parent 5f46a27 commit 5e418d6c36a538fddc42be86387e82f8e3480826 Markus Tiede committed Dec 17, 2012
@@ -10,6 +10,8 @@
*******************************************************************************/
package org.eclipse.jubula.client.ui.provider.labelprovider.decorators;
+import java.util.List;
+
import org.apache.commons.lang.StringUtils;
import org.eclipse.jface.viewers.IDecoration;
import org.eclipse.jubula.client.core.model.TestResultNode;
@@ -36,7 +38,10 @@ public void decorate(Object element, IDecoration decoration) {
if (element instanceof TestResultNode) {
TestResultNode testResult = (TestResultNode)element;
StringBuilder paramValueBuilder = new StringBuilder();
- for (TestResultParameter parameter : testResult.getParameters()) {
+ List<TestResultParameter> parameters = testResult.getParameters();
+ // use index based loop to avoid ConcurrentModificationException
+ for (int index = 0; index < parameters.size(); index++) {
+ TestResultParameter parameter = parameters.get(index);
paramValueBuilder
.append(StringUtils.defaultString(parameter.getValue()))
.append(SEPARATOR);

0 comments on commit 5e418d6

Please sign in to comment.