Repository: tajo Updated Branches: refs/heads/master 869dd2315 -> 0d434cdba
http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Constant.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Constant.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Constant.java index 11b9390..de05506 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Constant.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Constant.java @@ -225,8 +225,8 @@ class Constant { this.objVals = bsmArgs; int hashCode = 'y' + name.hashCode() * desc.hashCode() * bsm.hashCode(); - for (int i = 0; i < bsmArgs.length; i++) { - hashCode *= bsmArgs[i].hashCode(); + for (Object bsmArg : bsmArgs) { + hashCode *= bsmArg.hashCode(); } this.hashCode = 0x7FFFFFFF & hashCode; } http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/ConstantPool.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/ConstantPool.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/ConstantPool.java index e23487c..7bb3360 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/ConstantPool.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/ConstantPool.java @@ -218,8 +218,8 @@ public class ConstantPool extends HashMap<Constant, Constant> { newNameType(name, desc); newHandle(bsm.getTag(), bsm.getOwner(), bsm.getName(), bsm.getDesc()); - for (int i = 0; i < bsmArgs.length; i++) { - newConst(bsmArgs[i]); + for (Object bsmArg : bsmArgs) { + newConst(bsmArg); } result = new Constant(key5); put(result); http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/JarOptimizer.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/JarOptimizer.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/JarOptimizer.java index 066273d..a1fa82f 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/JarOptimizer.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/JarOptimizer.java @@ -99,8 +99,8 @@ public class JarOptimizer { if (f.isDirectory()) { File[] files = f.listFiles(); - for (int i = 0; i < files.length; ++i) { - optimize(files[i]); + for (File file : files) { + optimize(file); } } else if (f.getName().endsWith(".jar")) { File g = new File(f.getParentFile(), f.getName() + ".new"); http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Shrinker.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Shrinker.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Shrinker.java index cc2f131..61c284e 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Shrinker.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/optimizer/Shrinker.java @@ -98,8 +98,8 @@ public class Shrinker { throws IOException { if (f.isDirectory()) { File[] files = f.listFiles(); - for (int i = 0; i < files.length; ++i) { - optimize(files[i], d, remapper); + for (File file : files) { + optimize(file, d, remapper); } } else if (f.getName().endsWith(".class")) { ConstantPool cp = new ConstantPool(); http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/AnnotationNode.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/AnnotationNode.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/AnnotationNode.java index 4df5978..16a8a56 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/AnnotationNode.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/AnnotationNode.java @@ -213,8 +213,8 @@ public class AnnotationNode extends AnnotationVisitor { } else if (value instanceof List) { AnnotationVisitor v = av.visitArray(name); List<?> array = (List<?>) value; - for (int j = 0; j < array.size(); ++j) { - accept(v, null, array.get(j)); + for (Object anArray : array) { + accept(v, null, anArray); } v.visitEnd(); } else { http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/LookupSwitchInsnNode.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/LookupSwitchInsnNode.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/LookupSwitchInsnNode.java index 3e4c97a..9f3bb3f 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/LookupSwitchInsnNode.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/LookupSwitchInsnNode.java @@ -80,8 +80,8 @@ public class LookupSwitchInsnNode extends AbstractInsnNode { this.labels = new ArrayList<>(labels == null ? 0 : labels.length); if (keys != null) { - for (int i = 0; i < keys.length; ++i) { - this.keys.add(new Integer(keys[i])); + for (int key : keys) { + this.keys.add(new Integer(key)); } } if (labels != null) { http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/Analyzer.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/Analyzer.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/Analyzer.java index 1284ab6..6d2555f 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/Analyzer.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/Analyzer.java @@ -164,9 +164,9 @@ public class Analyzer<V extends Value> implements Opcodes { Type ctype = Type.getObjectType(owner); current.setLocal(local++, interpreter.newValue(ctype)); } - for (int i = 0; i < args.length; ++i) { - current.setLocal(local++, interpreter.newValue(args[i])); - if (args[i].getSize() == 2) { + for (Type arg : args) { + current.setLocal(local++, interpreter.newValue(arg)); + if (arg.getSize() == 2) { current.setLocal(local++, interpreter.newValue(null)); } } @@ -272,8 +272,7 @@ public class Analyzer<V extends Value> implements Opcodes { List<TryCatchBlockNode> insnHandlers = handlers[insn]; if (insnHandlers != null) { - for (int i = 0; i < insnHandlers.size(); ++i) { - TryCatchBlockNode tcb = insnHandlers.get(i); + for (TryCatchBlockNode tcb : insnHandlers) { Type type; if (tcb.type == null) { type = Type.getObjectType("java/lang/Throwable"); @@ -342,8 +341,7 @@ public class Analyzer<V extends Value> implements Opcodes { // calls findSubroutine recursively on exception handler successors List<TryCatchBlockNode> insnHandlers = handlers[insn]; if (insnHandlers != null) { - for (int i = 0; i < insnHandlers.size(); ++i) { - TryCatchBlockNode tcb = insnHandlers.get(i); + for (TryCatchBlockNode tcb : insnHandlers) { findSubroutine(insns.indexOf(tcb.handler), sub, calls); } } http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/BasicVerifier.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/BasicVerifier.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/BasicVerifier.java index 9ad66c3..d5e0bce 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/BasicVerifier.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/BasicVerifier.java @@ -376,10 +376,10 @@ public class BasicVerifier extends BasicInterpreter { final List<? extends BasicValue> values) throws AnalyzerException { int opcode = insn.getOpcode(); if (opcode == MULTIANEWARRAY) { - for (int i = 0; i < values.size(); ++i) { - if (!BasicValue.INT_VALUE.equals(values.get(i))) { + for (BasicValue value : values) { + if (!BasicValue.INT_VALUE.equals(value)) { throw new AnalyzerException(insn, null, - BasicValue.INT_VALUE, values.get(i)); + BasicValue.INT_VALUE, value); } } } else { http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/SimpleVerifier.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/SimpleVerifier.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/SimpleVerifier.java index ac4971c..7b51a17 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/SimpleVerifier.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/tree/analysis/SimpleVerifier.java @@ -290,8 +290,7 @@ public class SimpleVerifier extends BasicVerifier { return true; } if (currentClassInterfaces != null) { - for (int i = 0; i < currentClassInterfaces.size(); ++i) { - Type v = currentClassInterfaces.get(i); + for (Type v : currentClassInterfaces) { if (isAssignableFrom(t, v)) { return true; } http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/ASMifier.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/ASMifier.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/ASMifier.java index 42ce546..b844e7c 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/ASMifier.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/ASMifier.java @@ -660,8 +660,8 @@ public class ASMifier extends Printer { public void visitTableSwitchInsn(final int min, final int max, final Label dflt, final Label... labels) { buf.setLength(0); - for (int i = 0; i < labels.length; ++i) { - declareLabel(labels[i]); + for (Label label : labels) { + declareLabel(label); } declareLabel(dflt); @@ -681,8 +681,8 @@ public class ASMifier extends Printer { public void visitLookupSwitchInsn(final Label dflt, final int[] keys, final Label[] labels) { buf.setLength(0); - for (int i = 0; i < labels.length; ++i) { - declareLabel(labels[i]); + for (Label label : labels) { + declareLabel(label); } declareLabel(dflt); http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckClassAdapter.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckClassAdapter.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckClassAdapter.java index d030099..9735418 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckClassAdapter.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckClassAdapter.java @@ -214,12 +214,11 @@ public class CheckClassAdapter extends ClassVisitor { List<MethodNode> methods = cn.methods; List<Type> interfaces = new ArrayList<>(); - for (Iterator<String> i = cn.interfaces.iterator(); i.hasNext();) { - interfaces.add(Type.getObjectType(i.next())); + for (String anInterface : cn.interfaces) { + interfaces.add(Type.getObjectType(anInterface)); } - for (int i = 0; i < methods.size(); ++i) { - MethodNode method = methods.get(i); + for (MethodNode method : methods) { SimpleVerifier verifier = new SimpleVerifier( Type.getObjectType(cn.name), syperType, interfaces, (cn.access & Opcodes.ACC_INTERFACE) != 0); http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckMethodAdapter.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckMethodAdapter.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckMethodAdapter.java index 920ac94..d72dfdd 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckMethodAdapter.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/CheckMethodAdapter.java @@ -674,8 +674,8 @@ public class CheckMethodAdapter extends MethodVisitor { throw new IllegalArgumentException("invalid handle tag " + bsm.getTag()); } - for (int i = 0; i < bsmArgs.length; i++) { - checkLDCConstant(bsmArgs[i]); + for (Object bsmArg : bsmArgs) { + checkLDCConstant(bsmArg); } super.visitInvokeDynamicInsn(name, desc, bsm, bsmArgs); ++insnCount; @@ -744,8 +744,8 @@ public class CheckMethodAdapter extends MethodVisitor { checkNonDebugLabel(labels[i]); } super.visitTableSwitchInsn(min, max, dflt, labels); - for (int i = 0; i < labels.length; ++i) { - usedLabels.add(labels[i]); + for (Label label : labels) { + usedLabels.add(label); } ++insnCount; } @@ -767,8 +767,8 @@ public class CheckMethodAdapter extends MethodVisitor { } super.visitLookupSwitchInsn(dflt, keys, labels); usedLabels.add(dflt); - for (int i = 0; i < labels.length; ++i) { - usedLabels.add(labels[i]); + for (Label label : labels) { + usedLabels.add(label); } ++insnCount; } http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Printer.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Printer.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Printer.java index 1fb0e53..cd744c0 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Printer.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Printer.java @@ -488,8 +488,7 @@ public abstract class Printer { * string lists, and so on recursively. */ static void printList(final PrintWriter pw, final List<?> l) { - for (int i = 0; i < l.size(); ++i) { - Object o = l.get(i); + for (Object o : l) { if (o instanceof List) { printList(pw, (List<?>) o); } else { http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Textifier.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Textifier.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Textifier.java index 1b733fe..779868c 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Textifier.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/util/Textifier.java @@ -245,8 +245,8 @@ public class Textifier extends Printer { } if (interfaces != null && interfaces.length > 0) { buf.append(" implements "); - for (int i = 0; i < interfaces.length; ++i) { - appendDescriptor(INTERNAL_NAME, interfaces[i]); + for (String anInterface : interfaces) { + appendDescriptor(INTERNAL_NAME, anInterface); buf.append(' '); } } @@ -408,8 +408,8 @@ public class Textifier extends Printer { appendDescriptor(METHOD_DESCRIPTOR, desc); if (exceptions != null && exceptions.length > 0) { buf.append(" throws "); - for (int i = 0; i < exceptions.length; ++i) { - appendDescriptor(INTERNAL_NAME, exceptions[i]); + for (String exception : exceptions) { + appendDescriptor(INTERNAL_NAME, exception); buf.append(' '); } } @@ -787,8 +787,7 @@ public class Textifier extends Printer { buf.append(" none"); } else { buf.append('\n').append(tab3); - for (int i = 0; i < bsmArgs.length; i++) { - Object cst = bsmArgs[i]; + for (Object cst : bsmArgs) { if (cst instanceof String) { Printer.appendString(buf, (String) cst); } else if (cst instanceof Type) { http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/ASMContentHandler.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/ASMContentHandler.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/ASMContentHandler.java index 9a15fd8..68dfc14 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/ASMContentHandler.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/ASMContentHandler.java @@ -466,15 +466,13 @@ public class ASMContentHandler extends DefaultHandler implements Opcodes { } int n = path.lastIndexOf('/'); - for (Iterator<String> it = lpatterns.iterator(); it.hasNext();) { - String pattern = it.next(); + for (String pattern : lpatterns) { if (path.substring(n).endsWith(pattern)) { return rules.get(pattern); } } - for (Iterator<String> it = rpatterns.iterator(); it.hasNext();) { - String pattern = it.next(); + for (String pattern : rpatterns) { if (path.startsWith(pattern)) { return rules.get(pattern); } http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXAnnotationAdapter.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXAnnotationAdapter.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXAnnotationAdapter.java index 056f47e..75ab624 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXAnnotationAdapter.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXAnnotationAdapter.java @@ -88,50 +88,50 @@ public final class SAXAnnotationAdapter extends AnnotationVisitor { AnnotationVisitor av = visitArray(name); if (value instanceof byte[]) { byte[] b = (byte[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, new Byte(b[i])); + for (byte aB : b) { + av.visit(null, new Byte(aB)); } } else if (value instanceof char[]) { char[] b = (char[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, new Character(b[i])); + for (char aB : b) { + av.visit(null, new Character(aB)); } } else if (value instanceof short[]) { short[] b = (short[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, new Short(b[i])); + for (short aB : b) { + av.visit(null, new Short(aB)); } } else if (value instanceof boolean[]) { boolean[] b = (boolean[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, Boolean.valueOf(b[i])); + for (boolean aB : b) { + av.visit(null, Boolean.valueOf(aB)); } } else if (value instanceof int[]) { int[] b = (int[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, new Integer(b[i])); + for (int aB : b) { + av.visit(null, new Integer(aB)); } } else if (value instanceof long[]) { long[] b = (long[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, new Long(b[i])); + for (long aB : b) { + av.visit(null, new Long(aB)); } } else if (value instanceof float[]) { float[] b = (float[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, new Float(b[i])); + for (float aB : b) { + av.visit(null, new Float(aB)); } } else if (value instanceof double[]) { double[] b = (double[]) value; - for (int i = 0; i < b.length; i++) { - av.visit(null, new Double(b[i])); + for (double aB : b) { + av.visit(null, new Double(aB)); } } http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXClassAdapter.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXClassAdapter.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXClassAdapter.java index b26451b..231f147 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXClassAdapter.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXClassAdapter.java @@ -150,9 +150,9 @@ public final class SAXClassAdapter extends ClassVisitor { sa.addStart("interfaces", new AttributesImpl()); if (interfaces != null && interfaces.length > 0) { - for (int i = 0; i < interfaces.length; i++) { + for (String anInterface : interfaces) { AttributesImpl att2 = new AttributesImpl(); - att2.addAttribute("", "name", "name", "", interfaces[i]); + att2.addAttribute("", "name", "name", "", anInterface); sa.addElement("interface", att2); } } @@ -197,9 +197,9 @@ public final class SAXClassAdapter extends ClassVisitor { sa.addStart("exceptions", new AttributesImpl()); if (exceptions != null && exceptions.length > 0) { - for (int i = 0; i < exceptions.length; i++) { + for (String exception : exceptions) { AttributesImpl att2 = new AttributesImpl(); - att2.addAttribute("", "name", "name", "", exceptions[i]); + att2.addAttribute("", "name", "name", "", exception); sa.addElement("exception", att2); } } http://git-wip-us.apache.org/repos/asf/tajo/blob/0d434cdb/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXCodeAdapter.java ---------------------------------------------------------------------- diff --git a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXCodeAdapter.java b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXCodeAdapter.java index 9cfe9cd..f3ac562 100644 --- a/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXCodeAdapter.java +++ b/tajo-thirdparty/asm/src/main/java/org/apache/tajo/org/objectweb/asm/xml/SAXCodeAdapter.java @@ -193,8 +193,8 @@ public final class SAXCodeAdapter extends MethodVisitor { attrs.addAttribute("", "bsm", "bsm", "", SAXClassAdapter.encode(bsm.toString())); sa.addStart("INVOKEDYNAMIC", attrs); - for (int i = 0; i < bsmArgs.length; i++) { - sa.addElement("bsmArg", getConstantAttribute(bsmArgs[i])); + for (Object bsmArg : bsmArgs) { + sa.addElement("bsmArg", getConstantAttribute(bsmArg)); } sa.addEnd("INVOKEDYNAMIC"); } @@ -244,9 +244,9 @@ public final class SAXCodeAdapter extends MethodVisitor { attrs.addAttribute("", "dflt", "dflt", "", getLabel(dflt)); String o = Printer.OPCODES[Opcodes.TABLESWITCH]; sa.addStart(o, attrs); - for (int i = 0; i < labels.length; i++) { + for (Label label : labels) { AttributesImpl att2 = new AttributesImpl(); - att2.addAttribute("", "name", "name", "", getLabel(labels[i])); + att2.addAttribute("", "name", "name", "", getLabel(label)); sa.addElement("label", att2); } sa.addEnd(o);
