diff options
author | Christoph Stanik | 2017-07-17 18:53:45 +0000 |
---|---|---|
committer | Alexander Kurtakov | 2017-08-07 08:12:55 +0000 |
commit | 896e9624a30afb9e3f75daa648707b808d26ae0b (patch) | |
tree | fecf12e2a81a5065e23097c09860eb508f5792a1 /org.eclipse.search/search/org/eclipse/search | |
parent | 30c01e1e4b9f3f6feb22bf12a7248de589b24b9b (diff) | |
download | eclipse.platform.text-896e9624a30afb9e3f75daa648707b808d26ae0b.tar.gz eclipse.platform.text-896e9624a30afb9e3f75daa648707b808d26ae0b.tar.xz eclipse.platform.text-896e9624a30afb9e3f75daa648707b808d26ae0b.zip |
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>
Diffstat (limited to 'org.eclipse.search/search/org/eclipse/search')
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 { |