Bug 519785 - Use enhanced for loops in org.eclipse.searchI20170808-2000I20170807-2000

Change-Id: I3b6d471d4a889bf7769cd9d3d1cd9bbba3447361
Signed-off-by: Christoph Stanik <stanik@informatik.uni-hamburg.de>

22 files changed, 84 insertions, 108 deletions

diff --git a/org.eclipse.search/search/org/eclipse/search/internal/core/text/FileNamePatternSearchScope.java b/org.eclipse.search/search/org/eclipse/search/internal/core/text/FileNamePatternSearchScope.java
index a93aa568d8d..0d2e83ffdfb 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/core/text/FileNamePatternSearchScope.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/core/text/FileNamePatternSearchScope.java
@@ -156,8 +156,7 @@ public class FileNamePatternSearchScope extends TextSearchScope {
 
 	private static IResource[] removeRedundantEntries(IResource[] elements, boolean includeDerived) {
 		ArrayList<IResource> res= new ArrayList<>();
-		for (int i= 0; i < elements.length; i++) {
-			IResource curr= elements[i];
+		for (IResource curr : elements) {
 			addToList(res, curr, includeDerived);
 		}
 		return res.toArray(new IResource[res.size()]);
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/core/text/FilesOfScopeCalculator.java b/org.eclipse.search/search/org/eclipse/search/internal/core/text/FilesOfScopeCalculator.java
index b4b2ec1c7ac..91ace2479e6 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/core/text/FilesOfScopeCalculator.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/core/text/FilesOfScopeCalculator.java
@@ -47,9 +47,8 @@ public class FilesOfScopeCalculator implements IResourceProxyVisitor {
 		fFiles= new ArrayList<>();
 		try {
 			IResource[] roots= fScope.getRoots();
-			for (int i= 0; i < roots.length; i++) {
+			for (IResource resource : roots) {
 				try {
-					IResource resource= roots[i];
 					if (resource.isAccessible()) {
 						resource.accept(this, 0);
 					}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchEngineRegistry.java b/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchEngineRegistry.java
index 20ab53caaf3..db5acafa05a 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchEngineRegistry.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchEngineRegistry.java
@@ -82,8 +82,7 @@ public class TextSearchEngineRegistry {
 			@Override
 			public void run() throws Exception {
 				IConfigurationElement[] extensions= Platform.getExtensionRegistry().getConfigurationElementsFor(EXTENSION_POINT_ID);
-				for (int i= 0; i < extensions.length; i++) {
-					IConfigurationElement curr= extensions[i];
+				for (IConfigurationElement curr : extensions) {
 					if (ENGINE_NODE_NAME.equals(curr.getName()) && id.equals(curr.getAttribute(ATTRIB_ID))) {
 						res[0]= (TextSearchEngine) curr.createExecutableExtension(ATTRIB_CLASS);
 						return;
@@ -104,8 +103,7 @@ public class TextSearchEngineRegistry {
 		res.add(new String[] { SearchMessages.TextSearchEngineRegistry_defaulttextsearch_label, "" }); //$NON-NLS-1$
 
 		IConfigurationElement[] extensions= Platform.getExtensionRegistry().getConfigurationElementsFor(EXTENSION_POINT_ID);
-		for (int i= 0; i < extensions.length; i++) {
-			IConfigurationElement engine= extensions[i];
+		for (IConfigurationElement engine : extensions) {
 			if (ENGINE_NODE_NAME.equals(engine.getName())) {
 				res.add(new String[] { engine.getAttribute(ATTRIB_LABEL), engine.getAttribute(ATTRIB_ID) });
 			}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchVisitor.java b/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchVisitor.java
index 552c2411145..63a10cbc72d 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchVisitor.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/core/text/TextSearchVisitor.java
@@ -438,12 +438,12 @@ public class TextSearchVisitor {
 		Map<IFile, IDocument> result= new HashMap<>();
 		IWorkbench workbench= SearchPlugin.getDefault().getWorkbench();
 		IWorkbenchWindow[] windows= workbench.getWorkbenchWindows();
-		for (int i= 0; i < windows.length; i++) {
-			IWorkbenchPage[] pages= windows[i].getPages();
-			for (int x= 0; x < pages.length; x++) {
-				IEditorReference[] editorRefs= pages[x].getEditorReferences();
-				for (int z= 0; z < editorRefs.length; z++) {
-					IEditorPart ep= editorRefs[z].getEditor(false);
+		for (IWorkbenchWindow window : windows) {
+			IWorkbenchPage[] pages= window.getPages();
+			for (IWorkbenchPage page : pages) {
+				IEditorReference[] editorRefs= page.getEditorReferences();
+				for (IEditorReference editorRef : editorRefs) {
+					IEditorPart ep= editorRef.getEditor(false);
 					if (ep instanceof ITextEditor && ep.isDirty()) { // only dirty editors
 						evaluateTextEditor(result, ep);
 					}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/OpenSearchDialogPageAction.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/OpenSearchDialogPageAction.java
index 33ebb3c1244..ade38dfb8c4 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/OpenSearchDialogPageAction.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/OpenSearchDialogPageAction.java
@@ -11,7 +11,6 @@
  *******************************************************************************/
 package org.eclipse.search.internal.ui;
 
-import java.util.Iterator;
 import java.util.List;
 
 import org.eclipse.swt.events.MenuAdapter;
@@ -77,8 +76,7 @@ public class OpenSearchDialogPageAction implements IWorkbenchWindowPulldownDeleg
 	private void fillMenu(final Menu localMenu) {
 		List<SearchPageDescriptor> pageDescriptors= SearchPlugin.getDefault().getSearchPageDescriptors();
 		int accelerator= 1;
-		for (Iterator<SearchPageDescriptor> iter= pageDescriptors.iterator(); iter.hasNext();) {
-			SearchPageDescriptor desc= iter.next();
+		for (SearchPageDescriptor desc : pageDescriptors) {
 			if (!WorkbenchActivityHelper.filterItem(desc) && desc.isEnabled()) {
 				SearchPageAction action= new SearchPageAction(fWorkbenchWindow, desc);
 				addToMenu(localMenu, action, accelerator++);
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/ScopePart.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/ScopePart.java
index bc8c9d7be8c..63686b457f9 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/ScopePart.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/ScopePart.java
@@ -126,8 +126,8 @@ public class ScopePart {
 		IWorkingSetManager workingSetManager= PlatformUI.getWorkbench().getWorkingSetManager();
 		if (lruWorkingSetNames != null) {
 			Set<IWorkingSet> existingWorkingSets= new HashSet<>(lruWorkingSetNames.length);
-			for (int i= 0; i < lruWorkingSetNames.length; i++) {
-				IWorkingSet workingSet= getWorkingSet(workingSetManager, lruWorkingSetNames[i]);
+			for (String lruWorkingSetName : lruWorkingSetNames) {
+				IWorkingSet workingSet= getWorkingSet(workingSetManager, lruWorkingSetName);
 				if (workingSet != null) {
 					existingWorkingSets.add(workingSet);
 				}
@@ -231,8 +231,8 @@ public class ScopePart {
 		setSelectedScope(ISearchPageContainer.WORKING_SET_SCOPE);
 		fWorkingSets= null;
 		Set<IWorkingSet> existingWorkingSets= new HashSet<>(workingSets.length);
-		for (int i= 0; i < workingSets.length; i++) {
-			String name= workingSets[i].getName();
+		for (IWorkingSet workingSet2 : workingSets) {
+			String name= workingSet2.getName();
 			IWorkingSet workingSet= PlatformUI.getWorkbench().getWorkingSetManager().getWorkingSet(name);
 			if (workingSet != null)
 				existingWorkingSets.add(workingSet);
@@ -417,8 +417,8 @@ public class ScopePart {
 		if (workingSets != null && workingSets.length > 0) {
 			Arrays.sort(workingSets, new WorkingSetComparator());
 			boolean firstFound= false;
-			for (int i= 0; i < workingSets.length; i++) {
-				String workingSetName= workingSets[i].getLabel();
+			for (IWorkingSet workingSet : workingSets) {
+				String workingSetName= workingSet.getLabel();
 				if (firstFound)
 					result= Messages.format(SearchMessages.ScopePart_workingSetConcatenation, new String[] { result, workingSetName });
 				else {
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchDialog.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchDialog.java
index 3875d25efc3..d0503cad20d 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchDialog.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchDialog.java
@@ -107,8 +107,8 @@ public class SearchDialog extends ExtendedDialogWindow implements ISearchPageCon
 			int x= 0;
 			int y= 0;
 			Control[] children= composite.getChildren();
-			for (int i= 0; i < children.length; i++) {
-				Point size= children[i].computeSize(SWT.DEFAULT, SWT.DEFAULT, flushCache);
+			for (Control element : children) {
+				Point size= element.computeSize(SWT.DEFAULT, SWT.DEFAULT, flushCache);
 				x= Math.max(x, size.x);
 				y= Math.max(y, size.y);
 			}
@@ -128,8 +128,8 @@ public class SearchDialog extends ExtendedDialogWindow implements ISearchPageCon
 			Rectangle rect= composite.getClientArea();
 
 			Control[] children= composite.getChildren();
-			for (int i= 0; i < children.length; i++) {
-				children[i].setBounds(rect);
+			for (Control element : children) {
+				element.setBounds(rect);
 			}
 		}
 	}
@@ -204,16 +204,15 @@ public class SearchDialog extends ExtendedDialogWindow implements ISearchPageCon
 					IWorkingSet workingSet= (IWorkingSet) curr;
 					if (workingSet.isAggregateWorkingSet() && workingSet.isEmpty()) {
 						IProject[] projects= ResourcesPlugin.getWorkspace().getRoot().getProjects();
-						for (int i= 0; i < projects.length; i++) {
-							IProject proj= projects[i];
+						for (IProject proj : projects) {
 							if (proj.isOpen()) {
 								res.add(proj.getName());
 							}
 						}
 					} else {
 						IAdaptable[] elements= workingSet.getElements();
-						for (int i= 0; i < elements.length; i++) {
-							String name= evaluateEnclosingProject(elements[i]);
+						for (IAdaptable element : elements) {
+							String name= evaluateEnclosingProject(element);
 							if (name != null) {
 								res.add(name);
 							}
@@ -395,8 +394,7 @@ public class SearchDialog extends ExtendedDialogWindow implements ISearchPageCon
 
 	private List<SearchPageDescriptor> filterByActivities(List<SearchPageDescriptor> input) {
 		ArrayList<SearchPageDescriptor> filteredList= new ArrayList<>(input.size());
-		for (Iterator<SearchPageDescriptor> descriptors= input.iterator(); descriptors.hasNext();) {
-			SearchPageDescriptor descriptor= descriptors.next();
+		for (SearchPageDescriptor descriptor : input) {
 			if (!WorkbenchActivityHelper.filterItem(descriptor))
 			    filteredList.add(descriptor);
 
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchManager.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchManager.java
index 0eb749fcc08..35c7f62b93c 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchManager.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchManager.java
@@ -373,8 +373,8 @@ public class SearchManager implements IResourceChangeListener {
 		while (iter.hasNext())
 			iter.next().getControl().setRedraw(false);
 
-		for (int i=0; i < markerDeltas.length; i++) {
-			handleSearchMarkerChanged(markerDeltas[i]);
+		for (IMarkerDelta markerDelta : markerDeltas) {
+			handleSearchMarkerChanged(markerDelta);
 		}
 
 		iter= fListeners.iterator();
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPageDescriptor.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPageDescriptor.java
index 3c59e8b5d72..8716b96223f 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPageDescriptor.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPageDescriptor.java
@@ -233,9 +233,9 @@ class SearchPageDescriptor implements IPluginContribution, Comparable<SearchPage
 
 	static void setEnabled(Object[] enabledDescriptors) {
 		fgEnabledPageIds= new ArrayList<>(5);
-		for (int i= 0; i < enabledDescriptors.length; i++) {
-			if (enabledDescriptors[i] instanceof SearchPageDescriptor)
-				fgEnabledPageIds.add(((SearchPageDescriptor)enabledDescriptors[i]).getId());
+		for (Object enabledDescriptor : enabledDescriptors) {
+			if (enabledDescriptor instanceof SearchPageDescriptor)
+				fgEnabledPageIds.add(((SearchPageDescriptor)enabledDescriptor).getId());
 		}
 		storeEnabledPageIds();
 	}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPlugin.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPlugin.java
index 1d2766069d3..74e11066afc 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPlugin.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPlugin.java
@@ -143,8 +143,8 @@ public class SearchPlugin extends AbstractUIPlugin {
 			shell= shell.getParent();
 		}
 		Shell shells[]= display.getShells();
-		for (int i= 0; i < shells.length; i++) {
-			Object data= shells[i].getData();
+		for (Shell shell2 : shells) {
+			Object data= shell2.getData();
 			if (data instanceof IWorkbenchWindow) {
 				windowRef.window= (IWorkbenchWindow)data;
 				return;
@@ -282,8 +282,7 @@ public class SearchPlugin extends AbstractUIPlugin {
 	 */
 	private List<SearchPageDescriptor> createSearchPageDescriptors(IConfigurationElement[] elements) {
 		List<SearchPageDescriptor> result= new ArrayList<>(5);
-		for (int i= 0; i < elements.length; i++) {
-			IConfigurationElement element= elements[i];
+		for (IConfigurationElement element : elements) {
 			if (SearchPageDescriptor.PAGE_TAG.equals(element.getName())) {
 				SearchPageDescriptor desc= new SearchPageDescriptor(element);
 				result.add(desc);
@@ -326,8 +325,7 @@ public class SearchPlugin extends AbstractUIPlugin {
 	 */
 	private List<SorterDescriptor> createSorterDescriptors(IConfigurationElement[] elements) {
 		List<SorterDescriptor> result= new ArrayList<>(5);
-		for (int i= 0; i < elements.length; i++) {
-			IConfigurationElement element= elements[i];
+		for (IConfigurationElement element : elements) {
 			if (SorterDescriptor.SORTER_TAG.equals(element.getName()))
 				result.add(new SorterDescriptor(element));
 		}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchResultViewer.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchResultViewer.java
index 04dc2922634..9df5c73eef4 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchResultViewer.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchResultViewer.java
@@ -678,8 +678,7 @@ public class SearchResultViewer extends TableViewer {
 		Object[] changed= event.getElements();
 		if (changed != null && !fResourceToItemsMapper.isEmpty()) {
 			ArrayList<Object> others= new ArrayList<>(changed.length);
-			for (int i= 0; i < changed.length; i++) {
-				Object curr= changed[i];
+			for (Object curr : changed) {
 				if (curr instanceof IResource)
 					fResourceToItemsMapper.resourceChanged((IResource) curr);
 				else if (curr instanceof IAdaptable) {
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/SelectAllAction.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/SelectAllAction.java
index 1e497535be0..d78545c22fc 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/SelectAllAction.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/SelectAllAction.java
@@ -45,8 +45,7 @@ public class SelectAllAction extends Action {
 	}
 
 	private void collectExpandedAndVisible(TreeItem[] items, List<TreeItem> result) {
-		for (int i= 0; i < items.length; i++) {
-			TreeItem item= items[i];
+		for (TreeItem item : items) {
 			result.add(item);
 			if (item.getExpanded()) {
 				collectExpandedAndVisible(item.getItems(), result);
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/SortDropDownAction.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/SortDropDownAction.java
index 0130c9cc70c..30246108e82 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/SortDropDownAction.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/SortDropDownAction.java
@@ -190,9 +190,9 @@ class SortDropDownAction extends Action implements IMenuCreator {
 		if (memento == null)
 			return;
 		IMemento[] mementoElements= memento.getChildren(TAG_ELEMENT);
-		for (int i= 0; i < mementoElements.length; i++) {
-			String pageId= mementoElements[i].getString(TAG_PAGE_ID);
-			String sorterId= mementoElements[i].getString(TAG_SORTER_ID);
+		for (IMemento mementoElement : mementoElements) {
+			String pageId= mementoElement.getString(TAG_PAGE_ID);
+			String sorterId= mementoElement.getString(TAG_SORTER_ID);
 			SorterDescriptor sorterDesc= getSorter(sorterId);
 			if (sorterDesc != null)
 				map.put(pageId, sorterDesc);
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchPage.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchPage.java
index 4673b3eaaf1..96d56136520 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchPage.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchPage.java
@@ -459,8 +459,8 @@ public class FileSearchPage extends AbstractTextSearchViewPage implements IAdapt
 	@Override
 	protected void evaluateChangedElements(Match[] matches, Set<Object> changedElements) {
 		if (showLineMatches()) {
-			for (int i = 0; i < matches.length; i++) {
-				changedElements.add(((FileMatch) matches[i]).getLineElement());
+			for (Match match : matches) {
+				changedElements.add(((FileMatch) match).getLineElement());
 			}
 		} else {
 			super.evaluateChangedElements(matches, changedElements);
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchQuery.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchQuery.java
index a42870ecdeb..01b194212b4 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchQuery.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileSearchQuery.java
@@ -259,8 +259,8 @@ public class FileSearchQuery implements ISearchQuery {
 		String[] fileNamePatterns= fScope.getFileNamePatterns();
 		if (fileNamePatterns == null)
 			return true;
-		for (int i= 0; i < fileNamePatterns.length; i++) {
-			if ("*".equals(fileNamePatterns[i])) { //$NON-NLS-1$
+		for (String fileNamePattern : fileNamePatterns) {
+			if ("*".equals(fileNamePattern)) { //$NON-NLS-1$
 				return true;
 			}
 		}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTableContentProvider.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTableContentProvider.java
index 2f8121cca99..06362f406f3 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTableContentProvider.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTableContentProvider.java
@@ -59,16 +59,16 @@ public class FileTableContentProvider implements IStructuredContentProvider, IFi
 		TableViewer viewer= getViewer();
 		int elementLimit= getElementLimit();
 		boolean tableLimited= elementLimit != -1;
-		for (int i= 0; i < updatedElements.length; i++) {
-			if (fResult.getMatchCount(updatedElements[i]) > 0) {
-				if (viewer.testFindItem(updatedElements[i]) != null)
-					viewer.update(updatedElements[i], null);
+		for (Object updatedElement : updatedElements) {
+			if (fResult.getMatchCount(updatedElement) > 0) {
+				if (viewer.testFindItem(updatedElement) != null)
+					viewer.update(updatedElement, null);
 				else {
 					if (!tableLimited || viewer.getTable().getItemCount() < elementLimit)
-						viewer.add(updatedElements[i]);
+						viewer.add(updatedElement);
 				}
 			} else
-				viewer.remove(updatedElements[i]);
+				viewer.remove(updatedElement);
 		}
 	}
 
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTreeContentProvider.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTreeContentProvider.java
index b8c6b27deec..f254c0b06eb 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTreeContentProvider.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/FileTreeContentProvider.java
@@ -78,14 +78,14 @@ public class FileTreeContentProvider implements ITreeContentProvider, IFileSearc
 
 		if (result != null) {
 			Object[] elements= result.getElements();
-			for (int i= 0; i < elements.length; i++) {
+			for (Object element : elements) {
 				if (showLineMatches) {
-					Match[] matches= result.getMatches(elements[i]);
-					for (int j= 0; j < matches.length; j++) {
-						insert(((FileMatch) matches[j]).getLineElement(), false);
+					Match[] matches= result.getMatches(element);
+					for (Match match : matches) {
+						insert(((FileMatch) match).getLineElement(), false);
 					}
 				} else {
-					insert(elements[i], false);
+					insert(element, false);
 				}
 			}
 		}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/LineElement.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/LineElement.java
index cb9c0d2ff16..60402ecf77a 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/LineElement.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/LineElement.java
@@ -64,8 +64,8 @@ public class LineElement {
 	public FileMatch[] getMatches(AbstractTextSearchResult result) {
 		ArrayList<FileMatch> res= new ArrayList<>();
 		Match[] matches= result.getMatches(fParent);
-		for (int i= 0; i < matches.length; i++) {
-			FileMatch curr= (FileMatch) matches[i];
+		for (Match match : matches) {
+			FileMatch curr= (FileMatch) match;
 			if (curr.getLineElement() == this) {
 				res.add(curr);
 			}
@@ -76,8 +76,8 @@ public class LineElement {
 	public int getNumberOfMatches(AbstractTextSearchResult result) {
 		int count= 0;
 		Match[] matches= result.getMatches(fParent);
-		for (int i= 0; i < matches.length; i++) {
-			FileMatch curr= (FileMatch) matches[i];
+		for (Match match : matches) {
+			FileMatch curr= (FileMatch) match;
 			if (curr.getLineElement() == this) {
 				count++;
 			}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/ReplaceRefactoring.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/ReplaceRefactoring.java
index 1e7697f10e9..6ae6d9da17a 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/ReplaceRefactoring.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/ReplaceRefactoring.java
@@ -17,7 +17,6 @@ import java.util.Collection;
 import java.util.Comparator;
 import java.util.HashMap;
 import java.util.HashSet;
-import java.util.Iterator;
 import java.util.Map;
 import java.util.Set;
 import java.util.regex.Matcher;
@@ -126,8 +125,7 @@ public class ReplaceRefactoring extends Refactoring {
 		private Match[] getMatches() {
 			if (fMatches == null) {
 				ArrayList<FileMatch> matches= new ArrayList<>();
-				for (int i= 0; i < fMatchGroups.length; i++) {
-					MatchGroup curr= fMatchGroups[i];
+				for (MatchGroup curr : fMatchGroups) {
 					if (curr.group.isEnabled()) {
 						FileMatch match= curr.match;
 						matches.add(match);
@@ -211,13 +209,13 @@ public class ReplaceRefactoring extends Refactoring {
 		fMatches.clear();
 
 		if (fSelection != null) {
-			for (int i= 0; i < fSelection.length; i++) {
-				collectMatches(fSelection[i]);
+			for (Object element : fSelection) {
+				collectMatches(element);
 			}
 		} else {
 			Object[] elements= fResult.getElements();
-			for (int i= 0; i < elements.length; i++) {
-				collectMatches(elements[i]);
+			for (Object element : elements) {
+				collectMatches(element);
 			}
 		}
 		if (!hasMatches()) {
@@ -230,8 +228,7 @@ public class ReplaceRefactoring extends Refactoring {
 		if (object instanceof LineElement) {
 			LineElement lineElement= (LineElement) object;
 			FileMatch[] matches= lineElement.getMatches(fResult);
-			for (int i= 0; i < matches.length; i++) {
-				FileMatch fileMatch= matches[i];
+			for (FileMatch fileMatch : matches) {
 				if (isMatchToBeIncluded(fileMatch)) {
 					getBucket(fileMatch.getFile()).add(fileMatch);
 				}
@@ -239,15 +236,15 @@ public class ReplaceRefactoring extends Refactoring {
 		} else if (object instanceof IContainer) {
 			IContainer container= (IContainer) object;
 			IResource[] members= container.members();
-			for (int i= 0; i < members.length; i++) {
-				collectMatches(members[i]);
+			for (IResource member : members) {
+				collectMatches(member);
 			}
 		} else if (object instanceof IFile) {
 			Match[] matches= fResult.getMatches(object);
 			if (matches.length > 0) {
 				Collection<FileMatch> bucket= null;
-				for (int i= 0; i < matches.length; i++) {
-					FileMatch fileMatch= (FileMatch) matches[i];
+				for (Match match : matches) {
+					FileMatch fileMatch= (FileMatch) match;
 					if (isMatchToBeIncluded(fileMatch)) {
 						if (bucket == null) {
 							bucket= getBucket((IFile)object);
@@ -265,8 +262,7 @@ public class ReplaceRefactoring extends Refactoring {
 
 	public int getNumberOfMatches() {
 		int count= 0;
-		for (Iterator<Set<FileMatch>> iterator= fMatches.values().iterator(); iterator.hasNext();) {
-			Set<FileMatch> bucket= iterator.next();
+		for (Set<FileMatch> bucket : fMatches.values()) {
 			count += bucket.size();
 		}
 		return count;
@@ -290,8 +286,8 @@ public class ReplaceRefactoring extends Refactoring {
 		if (uri == null)
 			return true;
 
-		for (Iterator<URI> iter= fAlreadyCollected.keySet().iterator(); iter.hasNext();) {
-			if (URIUtil.equals(iter.next(), uri)) {
+		for (URI uri2 : fAlreadyCollected.keySet()) {
+			if (URIUtil.equals(uri2, uri)) {
 				if (file.equals(fAlreadyCollected.get(uri)))
 					return true; // another FileMatch for an IFile which already had matches
 
@@ -358,8 +354,7 @@ public class ReplaceRefactoring extends Refactoring {
 		ArrayList<MatchGroup> matchGroups= new ArrayList<>();
 		boolean hasChanges= false;
 		try {
-			for (int i= 0; i < allFiles.length; i++) {
-				IFile file= allFiles[i];
+			for (IFile file : allFiles) {
 				Set<FileMatch> bucket= fMatches.get(file);
 				if (!bucket.isEmpty()) {
 					try {
@@ -390,8 +385,7 @@ public class ReplaceRefactoring extends Refactoring {
 
 	private void checkFilesToBeChanged(IFile[] filesToBeChanged, RefactoringStatus resultingStatus) throws CoreException {
 		ArrayList<IFile> readOnly= new ArrayList<>();
-		for (int i= 0; i < filesToBeChanged.length; i++) {
-			IFile file= filesToBeChanged[i];
+		for (IFile file : filesToBeChanged) {
 			if (file.isReadOnly())
 				readOnly.add(file);
 		}
@@ -425,8 +419,7 @@ public class ReplaceRefactoring extends Refactoring {
 			IDocument document= textFileBuffer.getDocument();
 			String lineDelimiter= TextUtilities.getDefaultLineDelimiter(document);
 
-			for (Iterator<FileMatch> iterator= matches.iterator(); iterator.hasNext();) {
-				FileMatch match= iterator.next();
+			for (FileMatch match : matches) {
 				int offset= match.getOffset();
 				int length= match.getLength();
 				Position currentPosition= tracker.getCurrentPosition(match);
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/TextSearchPage.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/TextSearchPage.java
index 4e7c493766a..786d34a9ec4 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/text/TextSearchPage.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/text/TextSearchPage.java
@@ -349,8 +349,8 @@ public class TextSearchPage extends DialogPage implements ISearchPage, IReplaceP
 						return FileTextSearchScope.newWorkspaceScope(getExtensions(), fSearchDerived);
 					}
 					IAdaptable[] elements= workingSet.getElements();
-					for (int i= 0; i < elements.length; i++) {
-						IResource resource= elements[i].getAdapter(IResource.class);
+					for (IAdaptable element : elements) {
+						IResource resource= element.getAdapter(IResource.class);
 						if (resource != null && resource.isAccessible()) {
 							resources.add(resource);
 						}
@@ -390,8 +390,7 @@ public class TextSearchPage extends DialogPage implements ISearchPage, IReplaceP
 
 
 	private SearchPatternData findInPrevious(String pattern) {
-		for (Iterator<SearchPatternData> iter= fPreviousSearchPatterns.iterator(); iter.hasNext();) {
-			SearchPatternData element= iter.next();
+		for (SearchPatternData element : fPreviousSearchPatterns) {
 			if (pattern.equals(element.textPattern)) {
 				return element;
 			}
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/util/ExtendedDialogWindow.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/util/ExtendedDialogWindow.java
index 70819f64665..949193be252 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/util/ExtendedDialogWindow.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/util/ExtendedDialogWindow.java
@@ -14,7 +14,6 @@ package org.eclipse.search.internal.ui.util;
 import java.lang.reflect.InvocationTargetException;
 import java.util.ArrayList;
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.List;
 
 import org.eclipse.swt.SWT;
@@ -174,8 +173,7 @@ public abstract class ExtendedDialogWindow extends TrayDialog implements IRunnab
 	 * @param state The new state
 	 */
 	public void setPerformActionEnabled(boolean state) {
-		for (Iterator<Button> buttons = fActionButtons.iterator(); buttons.hasNext(); ) {
-			Button element = buttons.next();
+		for (Button element : fActionButtons) {
 			element.setEnabled(state);
 		}
 	}
@@ -271,16 +269,15 @@ public abstract class ExtendedDialogWindow extends TrayDialog implements IRunnab
 
 	private void setDisplayCursor(Display d, Cursor c) {
 		Shell[] shells= d.getShells();
-		for (int i= 0; i < shells.length; i++)
-			shells[i].setCursor(c);
+		for (Shell shell2 : shells)
+			shell2.setCursor(c);
 	}
 
 	//---- UI state save and restoring ---------------------------------------------
 
 	private void restoreUIState(HashMap<Object, Object> state) {
 		restoreEnableState(fCancelButton, state);
-		for (Iterator<Button> actionButtons = fActionButtons.iterator(); actionButtons.hasNext(); ) {
-			Button button = actionButtons.next();
+		for (Button button : fActionButtons) {
 			restoreEnableState(button, state);
 		}
 		ControlEnableState pageState= (ControlEnableState)state.get("tabForm"); //$NON-NLS-1$
@@ -301,8 +298,7 @@ public abstract class ExtendedDialogWindow extends TrayDialog implements IRunnab
 	private HashMap<Object, Object> saveUIState(boolean keepCancelEnabled) {
 		HashMap<Object, Object> savedState= new HashMap<>(10);
 		saveEnableStateAndSet(fCancelButton, savedState, keepCancelEnabled);
-		for (Iterator<Button> actionButtons = fActionButtons.iterator(); actionButtons.hasNext(); ) {
-			Button button = actionButtons.next();
+		for (Button button : fActionButtons) {
 			saveEnableStateAndSet(button, savedState, false);
 		}
 		savedState.put("tabForm", ControlEnableState.disable(fContents)); //$NON-NLS-1$
diff --git a/org.eclipse.search/search/org/eclipse/search/internal/ui/util/FileTypeEditor.java b/org.eclipse.search/search/org/eclipse/search/internal/ui/util/FileTypeEditor.java
index 6556c413b2c..e3fe7252591 100644
--- a/org.eclipse.search/search/org/eclipse/search/internal/ui/util/FileTypeEditor.java
+++ b/org.eclipse.search/search/org/eclipse/search/internal/ui/util/FileTypeEditor.java
@@ -95,8 +95,8 @@ public class FileTypeEditor extends SelectionAdapter implements DisposeListener
 			Object[] result= dialog.getResult();
 			HashSet<String> patterns= new HashSet<>();
 			boolean starIncluded= false;
-			for (int i= 0; i < result.length; i++) {
-				String curr= result[i].toString();
+			for (Object element : result) {
+				String curr= element.toString();
 				if (curr.equals("*")) { //$NON-NLS-1$
 					starIncluded= true;
 				} else {