Use foreach loops Project: http://git-wip-us.apache.org/repos/asf/ant/repo Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/061d29b6 Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/061d29b6 Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/061d29b6
Branch: refs/heads/master Commit: 061d29b6fe242c365705a76b28112392de9cad80 Parents: 96ed9bd Author: Gintas Grigelionis <gin...@apache.org> Authored: Thu Apr 12 09:22:30 2018 +0200 Committer: Gintas Grigelionis <gin...@apache.org> Committed: Thu Apr 12 09:22:30 2018 +0200 ---------------------------------------------------------------------- src/main/org/apache/tools/ant/Project.java | 4 +--- src/main/org/apache/tools/ant/taskdefs/CVSPass.java | 4 ++-- src/main/org/apache/tools/ant/taskdefs/Concat.java | 7 ++----- src/main/org/apache/tools/ant/taskdefs/MacroInstance.java | 3 +-- src/main/org/apache/tools/ant/taskdefs/SQLExec.java | 4 +--- .../org/apache/tools/ant/taskdefs/condition/IsSigned.java | 4 +--- .../tools/ant/taskdefs/optional/jsp/Jasper41Mangler.java | 4 +--- .../tools/ant/taskdefs/optional/jsp/JspNameMangler.java | 3 +-- src/main/org/apache/tools/ant/util/StringUtils.java | 3 +-- src/main/org/apache/tools/tar/TarOutputStream.java | 7 +++---- 10 files changed, 14 insertions(+), 29 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/Project.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/Project.java b/src/main/org/apache/tools/ant/Project.java index 8d59ba0..1d1da17 100644 --- a/src/main/org/apache/tools/ant/Project.java +++ b/src/main/org/apache/tools/ant/Project.java @@ -941,9 +941,7 @@ public class Project implements ResourceFactory { */ public void setSystemProperties() { final Properties systemP = System.getProperties(); - final Enumeration<?> e = systemP.propertyNames(); - while (e.hasMoreElements()) { - final String propertyName = (String) e.nextElement(); + for (final String propertyName : systemP.stringPropertyNames()) { final String value = systemP.getProperty(propertyName); if (value != null) { this.setPropertyInternal(propertyName, value); http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/taskdefs/CVSPass.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/CVSPass.java b/src/main/org/apache/tools/ant/taskdefs/CVSPass.java index 19a6a41..2ea2e11 100644 --- a/src/main/org/apache/tools/ant/taskdefs/CVSPass.java +++ b/src/main/org/apache/tools/ant/taskdefs/CVSPass.java @@ -131,8 +131,8 @@ public class CVSPass extends Task { private final String mangle(String password) { StringBuilder buf = new StringBuilder(); - for (int i = 0; i < password.length(); i++) { - buf.append(shifts[password.charAt(i)]); + for (final char ch : password.toCharArray()) { + buf.append(shifts[ch]); } return buf.toString(); } http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/taskdefs/Concat.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/Concat.java b/src/main/org/apache/tools/ant/taskdefs/Concat.java index 59802f9..b58928d 100644 --- a/src/main/org/apache/tools/ant/taskdefs/Concat.java +++ b/src/main/org/apache/tools/ant/taskdefs/Concat.java @@ -185,12 +185,9 @@ public class Concat extends Task implements ResourceCollection { value = ""; } if (trimLeading) { - char[] current = value.toCharArray(); - StringBuilder b = new StringBuilder(current.length); + StringBuilder b = new StringBuilder(); boolean startOfLine = true; - int pos = 0; - while (pos < current.length) { - char ch = current[pos++]; + for (final char ch : value.toCharArray()) { if (startOfLine) { if (ch == ' ' || ch == '\t') { continue; http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/taskdefs/MacroInstance.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/MacroInstance.java b/src/main/org/apache/tools/ant/taskdefs/MacroInstance.java index 23ff8ab..7190e39 100644 --- a/src/main/org/apache/tools/ant/taskdefs/MacroInstance.java +++ b/src/main/org/apache/tools/ant/taskdefs/MacroInstance.java @@ -174,8 +174,7 @@ public class MacroInstance extends Task implements DynamicAttribute, TaskContain StringBuilder macroName = null; int state = STATE_NORMAL; - for (int i = 0; i < s.length(); ++i) { - char ch = s.charAt(i); + for (final char ch : s.toCharArray()) { switch (state) { case STATE_NORMAL: if (ch == '@') { http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/taskdefs/SQLExec.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/SQLExec.java b/src/main/org/apache/tools/ant/taskdefs/SQLExec.java index 55f80c6..73deb8a 100644 --- a/src/main/org/apache/tools/ant/taskdefs/SQLExec.java +++ b/src/main/org/apache/tools/ant/taskdefs/SQLExec.java @@ -911,10 +911,8 @@ public class SQLExec extends JDBCTask { return s; } StringBuilder sb = new StringBuilder(csvQuoteChar); - int len = s.length(); char q = csvQuoteChar.charAt(0); - for (int i = 0; i < len; i++) { - char c = s.charAt(i); + for (final char c : s.toCharArray()) { if (c == q) { sb.append(q); } http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/taskdefs/condition/IsSigned.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/condition/IsSigned.java b/src/main/org/apache/tools/ant/taskdefs/condition/IsSigned.java index 267ff19..851bf12 100644 --- a/src/main/org/apache/tools/ant/taskdefs/condition/IsSigned.java +++ b/src/main/org/apache/tools/ant/taskdefs/condition/IsSigned.java @@ -133,10 +133,8 @@ public class IsSigned extends DataType implements Condition { private static String replaceInvalidChars(final String name) { StringBuilder sb = new StringBuilder(); - final int len = name.length(); boolean changes = false; - for (int i = 0; i < len; i++) { - final char ch = name.charAt(i); + for (final char ch : name.toCharArray()) { if (ManifestTask.VALID_ATTRIBUTE_CHARS.indexOf(ch) < 0) { sb.append("_"); changes = true; http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/taskdefs/optional/jsp/Jasper41Mangler.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/jsp/Jasper41Mangler.java b/src/main/org/apache/tools/ant/taskdefs/optional/jsp/Jasper41Mangler.java index d863715..fe83a33 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/jsp/Jasper41Mangler.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/jsp/Jasper41Mangler.java @@ -37,7 +37,6 @@ public class Jasper41Mangler implements JspMangler { public String mapJspToJavaName(File jspFile) { String jspUri = jspFile.getAbsolutePath(); int start = jspUri.lastIndexOf(File.separatorChar) + 1; - int end = jspUri.length(); StringBuilder modifiedClassName = new StringBuilder(jspUri.length() - start); if (!Character.isJavaIdentifierStart(jspUri.charAt(start)) || jspUri.charAt(start) == '_') { @@ -45,8 +44,7 @@ public class Jasper41Mangler implements JspMangler { // prepend a '_'. modifiedClassName.append('_'); } - for (int i = start; i < end; i++) { - char ch = jspUri.charAt(i); + for (final char ch : jspUri.substring(start).toCharArray()) { if (Character.isJavaIdentifierPart(ch)) { modifiedClassName.append(ch); } else if (ch == '.') { http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java b/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java index 8fe43ab..5d7dd7b 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java @@ -92,8 +92,7 @@ public class JspNameMangler implements JspMangler { modifiedClassName.append(mangleChar(firstChar)); } // this is the rest - for (int i = 1; i < className.length(); i++) { - char subChar = className.charAt(i); + for (final char subChar : className.substring(1).toCharArray()) { if (Character.isJavaIdentifierPart(subChar)) { modifiedClassName.append(subChar); } else { http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/ant/util/StringUtils.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/util/StringUtils.java b/src/main/org/apache/tools/ant/util/StringUtils.java index c9f4f7a..86c1dab 100644 --- a/src/main/org/apache/tools/ant/util/StringUtils.java +++ b/src/main/org/apache/tools/ant/util/StringUtils.java @@ -155,8 +155,7 @@ public final class StringUtils { public static String resolveBackSlash(String input) { StringBuilder b = new StringBuilder(); boolean backSlashSeen = false; - for (int i = 0; i < input.length(); ++i) { - char c = input.charAt(i); + for (final char c : input.toCharArray()) { if (!backSlashSeen) { if (c == '\\') { backSlashSeen = true; http://git-wip-us.apache.org/repos/asf/ant/blob/061d29b6/src/main/org/apache/tools/tar/TarOutputStream.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/tar/TarOutputStream.java b/src/main/org/apache/tools/tar/TarOutputStream.java index 2802e95..0414e12 100644 --- a/src/main/org/apache/tools/tar/TarOutputStream.java +++ b/src/main/org/apache/tools/tar/TarOutputStream.java @@ -521,10 +521,9 @@ public class TarOutputStream extends FilterOutputStream { } private String stripTo7Bits(String name) { - final int length = name.length(); - StringBuilder result = new StringBuilder(length); - for (int i = 0; i < length; i++) { - char stripped = (char) (name.charAt(i) & 0x7F); + StringBuilder result = new StringBuilder(name.length()); + for (final char ch : name.toCharArray()) { + char stripped = (char) (ch & 0x7F); if (stripped != 0) { // would be read as Trailing null result.append(stripped); }