Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCarsten Hammer2019-09-25 20:00:38 +0000
committerAlexander Kurtakov2019-10-16 05:10:16 +0000
commite06819792b23c8e9305c7d5662cc11c39c781577 (patch)
treec150a186221e2c02ae942de1d90683007c58a093 /bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox
parentc3e6490d91f762660b540236b6e45a963243df23 (diff)
downloadrt.equinox.p2-e06819792b23c8e9305c7d5662cc11c39c781577.tar.gz
rt.equinox.p2-e06819792b23c8e9305c7d5662cc11c39c781577.tar.xz
rt.equinox.p2-e06819792b23c8e9305c7d5662cc11c39c781577.zip
Use jdk 5 for-each loop
Replace simple uses of Iterator with a corresponding for-loop. Also add missing braces on loops as necessary. Change-Id: Id71f030335f035816de194384d1a3c042a350886 Signed-off-by: Carsten Hammer <carsten.hammer@t-online.de>
Diffstat (limited to 'bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox')
-rw-r--r--bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/CompositeWriter.java5
-rw-r--r--bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/XMLWriter.java4
-rw-r--r--bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/repository/CacheManager.java6
3 files changed, 8 insertions, 7 deletions
diff --git a/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/CompositeWriter.java b/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/CompositeWriter.java
index 68b0f41bc..e06da2ba8 100644
--- a/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/CompositeWriter.java
+++ b/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/CompositeWriter.java
@@ -39,8 +39,9 @@ public class CompositeWriter extends XMLWriter implements XMLConstants {
return;
start(CHILDREN_ELEMENT);
attribute(COLLECTION_SIZE_ATTRIBUTE, children.length);
- for (int i = 0; i < children.length; i++)
- writeChild(children[i]);
+ for (URI child : children) {
+ writeChild(child);
+ }
end(CHILDREN_ELEMENT);
}
diff --git a/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/XMLWriter.java b/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/XMLWriter.java
index 79e96f5d2..011417df1 100644
--- a/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/XMLWriter.java
+++ b/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/persistence/XMLWriter.java
@@ -71,8 +71,8 @@ public class XMLWriter implements XMLConstants {
this.open = false;
this.indent = " "; //$NON-NLS-1$
if (piElements != null) {
- for (int i = 0; i < piElements.length; i++) {
- println(piElements[i].toString());
+ for (ProcessingInstruction piElement : piElements) {
+ println(piElement.toString());
}
}
}
diff --git a/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/repository/CacheManager.java b/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/repository/CacheManager.java
index 7cd469d4a..99b3e9f41 100644
--- a/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/repository/CacheManager.java
+++ b/bundles/org.eclipse.equinox.p2.repository/src/org/eclipse/equinox/internal/p2/repository/CacheManager.java
@@ -274,11 +274,11 @@ public class CacheManager {
void deleteCache(URI repositoryLocation) {
for (String prefix : knownPrefixes) {
File[] cacheFiles = getCacheFiles(repositoryLocation, prefix);
- for (int i = 0; i < cacheFiles.length; i++) {
+ for (File cacheFile : cacheFiles) {
// delete the cache file if it exists
- safeDelete(cacheFiles[i]);
+ safeDelete(cacheFile);
// delete a resumable download if it exists
- safeDelete(new File(new File(cacheFiles[i].getParentFile(), DOWNLOADING), cacheFiles[i].getName()));
+ safeDelete(new File(new File(cacheFile.getParentFile(), DOWNLOADING), cacheFile.getName()));
}
}
}

Back to the top