Use jdk 5 for-each loopI20190614-0330I20190613-1800

Replace simple uses of Iterator with a corresponding for-each loop. Also
add missing braces on loops as necessary.

Change-Id: Ibde37d56a7962f432ed771f4a215f5903161bde0
Signed-off-by: Carsten Hammer <carsten.hammer@t-online.de>

1 files changed, 7 insertions, 6 deletions

diff --git a/org.eclipse.jface.text.tests/src/org/eclipse/jface/text/tests/TextPresentationTest.java b/org.eclipse.jface.text.tests/src/org/eclipse/jface/text/tests/TextPresentationTest.java
index f8c2518a91b..7e71c22bef2 100644
--- a/org.eclipse.jface.text.tests/src/org/eclipse/jface/text/tests/TextPresentationTest.java
+++ b/org.eclipse.jface.text.tests/src/org/eclipse/jface/text/tests/TextPresentationTest.java
@@ -66,8 +66,9 @@ public class TextPresentationTest {
 	private void setUpTextPresentation() {
 		fTextPresentation= new TextPresentation();
 		fTextPresentation.setDefaultStyleRange(createStyleRange(0, 140, NORMAL));
-		for (int i= 0; i < fAllRanges.length; i++)
-			fTextPresentation.addStyleRange(fAllRanges[i]);
+		for (StyleRange fAllRange : fAllRanges) {
+			fTextPresentation.addStyleRange(fAllRange);
+		}
 	}
 
 	private StyleRange createStyleRange(int start, int end, int style) {
@@ -100,16 +101,16 @@ public class TextPresentationTest {
 
 	private void checkRegions(StyleRange[] expectedAllRanges, StyleRange[] expectedNonDefaultRanges) {
 		Iterator<StyleRange> e= fTextPresentation.getAllStyleRangeIterator();
-		for (int i= 0; i < expectedAllRanges.length; i++) {
+		for (StyleRange expectedAllRange : expectedAllRanges) {
 			assertTrue(e.hasNext());
-			assertEquals(expectedAllRanges[i], e.next());
+			assertEquals(expectedAllRange, e.next());
 		}
 		assertTrue(!e.hasNext());
 
 		e= fTextPresentation.getNonDefaultStyleRangeIterator();
-		for (int i= 0; i < expectedNonDefaultRanges.length; i++) {
+		for (StyleRange expectedNonDefaultRange : expectedNonDefaultRanges) {
 			assertTrue(e.hasNext());
-			assertEquals(expectedNonDefaultRanges[i], e.next());
+			assertEquals(expectedNonDefaultRange, e.next());
 		}
 		assertTrue(!e.hasNext());
 	}