Return-Path: 
 <commits-return-37871-apmail-commons-commits-archive=commons.apache.org@commons.apache.org>
X-Original-To: apmail-commons-commits-archive@minotaur.apache.org
Delivered-To: apmail-commons-commits-archive@minotaur.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id DF4CE1016C
	for <apmail-commons-commits-archive@minotaur.apache.org>;
 Mon, 25 Nov 2013 13:15:19 +0000 (UTC)
Received: (qmail 74071 invoked by uid 500); 25 Nov 2013 13:15:17 -0000
Delivered-To: apmail-commons-commits-archive@commons.apache.org
Received: (qmail 74053 invoked by uid 500); 25 Nov 2013 13:15:16 -0000
Mailing-List: contact commits-help@commons.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@commons.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@commons.apache.org>
List-Post: <mailto:commits@commons.apache.org>
List-Id: <commits.commons.apache.org>
Reply-To: dev@commons.apache.org
Delivered-To: mailing list commits@commons.apache.org
Received: (qmail 74046 invoked by uid 99); 25 Nov 2013 13:15:16 -0000
Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230)
    by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 25 Nov 2013 13:15:16 +0000
X-ASF-Spam-Status: No, hits=-2000.0 required=5.0
	tests=ALL_TRUSTED
X-Spam-Check-By: apache.org
Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4)
    by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 25 Nov 2013 13:15:13 +0000
Received: from eris.apache.org (localhost [127.0.0.1])
	by eris.apache.org (Postfix) with ESMTP id 9BDE2238889B;
	Mon, 25 Nov 2013 13:14:52 +0000 (UTC)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1545254 - in
 /commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats:
 png/PngImageParser.java tiff/TiffDirectory.java
Date: Mon, 25 Nov 2013 13:14:52 -0000
To: commits@commons.apache.org
From: ebourg@apache.org
X-Mailer: svnmailer-1.0.9
Message-Id: <20131125131452.9BDE2238889B@eris.apache.org>
X-Virus-Checked: Checked by ClamAV on apache.org

Author: ebourg
Date: Mon Nov 25 13:14:52 2013
New Revision: 1545254

URL: http://svn.apache.org/r1545254
Log:
foreach loops

Modified:
    commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/png/PngImageParser.java
    commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/tiff/TiffDirectory.java

Modified: commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/png/PngImageParser.java
URL: http://svn.apache.org/viewvc/commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/png/PngImageParser.java?rev=1545254&r1=1545253&r2=1545254&view=diff
==============================================================================
--- commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/png/PngImageParser.java (original)
+++ commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/png/PngImageParser.java Mon Nov 25 13:14:52 2013
@@ -300,10 +300,10 @@ public class PngImageParser extends Imag
 
         final ImageMetadata result = new ImageMetadata();
 
-        for (int i = 0; i < chunks.size(); i++) {
-            final PngTextChunk chunk = (PngTextChunk) chunks.get(i);
+        for (PngChunk chunk : chunks) {
+            final PngTextChunk textChunk = (PngTextChunk) chunk;
 
-            result.add(chunk.getKeyword(), chunk.getText());
+            result.add(textChunk.getKeyword(), textChunk.getText());
         }
 
         return result;
@@ -401,11 +401,10 @@ public class PngImageParser extends Imag
         throw new ImageReadException("PNG: unknown color type: " + colorType);
     }
 
-    private List<PngChunk> filterChunks(final List<PngChunk> v, final int type) {
+    private List<PngChunk> filterChunks(final List<PngChunk> chunks, final int type) {
         final List<PngChunk> result = new ArrayList<PngChunk>();
 
-        for (int i = 0; i < v.size(); i++) {
-            final PngChunk chunk = v.get(i);
+        for (PngChunk chunk : chunks) {
             if (chunk.chunkType == type) {
                 result.add(chunk);
             }
@@ -539,18 +538,18 @@ public class PngImageParser extends Imag
         final List<String> comments = new ArrayList<String>();
         final List<PngText> textChunks = new ArrayList<PngText>();
 
-        for (int i = 0; i < tEXts.size(); i++) {
-            final PngChunkText pngChunktEXt = (PngChunkText) tEXts.get(i);
+        for (PngChunk tEXt : tEXts) {
+            final PngChunkText pngChunktEXt = (PngChunkText) tEXt;
             comments.add(pngChunktEXt.keyword + ": " + pngChunktEXt.text);
             textChunks.add(pngChunktEXt.getContents());
         }
-        for (int i = 0; i < zTXts.size(); i++) {
-            final PngChunkZtxt pngChunkzTXt = (PngChunkZtxt) zTXts.get(i);
+        for (PngChunk zTXt : zTXts) {
+            final PngChunkZtxt pngChunkzTXt = (PngChunkZtxt) zTXt;
             comments.add(pngChunkzTXt.keyword + ": " + pngChunkzTXt.text);
             textChunks.add(pngChunkzTXt.getContents());
         }
-        for (int i = 0; i < iTXts.size(); i++) {
-            final PngChunkItxt pngChunkiTXt = (PngChunkItxt) iTXts.get(i);
+        for (PngChunk iTXt : iTXts) {
+            final PngChunkItxt pngChunkiTXt = (PngChunkItxt) iTXt;
             comments.add(pngChunkiTXt.keyword + ": " + pngChunkiTXt.text);
             textChunks.add(pngChunkiTXt.getContents());
         }
@@ -685,8 +684,8 @@ public class PngImageParser extends Imag
         }
 
         ByteArrayOutputStream baos = new ByteArrayOutputStream();
-        for (int i = 0; i < IDATs.size(); i++) {
-            final PngChunkIdat pngChunkIDAT = (PngChunkIdat) IDATs.get(i);
+        for (PngChunk IDAT : IDATs) {
+            final PngChunkIdat pngChunkIDAT = (PngChunkIdat) IDAT;
             final byte bytes[] = pngChunkIDAT.getBytes();
             // System.out.println(i + ": bytes: " + bytes.length);
             baos.write(bytes);
@@ -894,12 +893,12 @@ public class PngImageParser extends Imag
         }
 
         final List<PngChunkItxt> xmpChunks = new ArrayList<PngChunkItxt>();
-        for (int i = 0; i < chunks.size(); i++) {
-            final PngChunkItxt chunk = (PngChunkItxt) chunks.get(i);
-            if (!chunk.getKeyword().equals(PngConstants.XMP_KEYWORD)) {
+        for (PngChunk chunk : chunks) {
+            final PngChunkItxt itxtChunk = (PngChunkItxt) chunk;
+            if (!itxtChunk.getKeyword().equals(PngConstants.XMP_KEYWORD)) {
                 continue;
             }
-            xmpChunks.add(chunk);
+            xmpChunks.add(itxtChunk);
         }
 
         if (xmpChunks.isEmpty()) {

Modified: commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/tiff/TiffDirectory.java
URL: http://svn.apache.org/viewvc/commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/tiff/TiffDirectory.java?rev=1545254&r1=1545253&r2=1545254&view=diff
==============================================================================
--- commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/tiff/TiffDirectory.java (original)
+++ commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/tiff/TiffDirectory.java Mon Nov 25 13:14:52 2013
@@ -78,9 +78,7 @@ public class TiffDirectory extends TiffE
         long entryOffset = offset + TiffConstants.TIFF_DIRECTORY_HEADER_LENGTH;
 
         final StringBuilder result = new StringBuilder();
-        for (int i = 0; i < entries.size(); i++) {
-            final TiffField entry = entries.get(i);
-
+        for (TiffField entry : entries) {
             result.append(String.format("\t[%d]: %s (%d, 0x%x), %s, %d: %s%n",
                     entryOffset, entry.getTagInfo().name,
                     entry.getTag(), entry.getTag(),
@@ -171,8 +169,7 @@ public class TiffDirectory extends TiffE
             return null;
         }
 
-        for (int i = 0; i < entries.size(); i++) {
-            final TiffField field = entries.get(i);
+        for (TiffField field : entries) {
             if (field.getTag() == tag.tag) {
                 return field;
             }