You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2014/06/18 19:50:10 UTC

svn commit: r1603551 - in /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox: afm/AFMParser.java util/autodetect/NativeFontDirFinder.java

Author: tilman
Date: Wed Jun 18 17:50:09 2014
New Revision: 1603551

URL: http://svn.apache.org/r1603551
Log:
PDFBOX-2147: use foreach loop

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/AFMParser.java
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/util/autodetect/NativeFontDirFinder.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/AFMParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/AFMParser.java?rev=1603551&r1=1603550&r2=1603551&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/AFMParser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/AFMParser.java Wed Jun 18 17:50:09 2014
@@ -16,6 +16,7 @@
  */
 package org.apache.fontbox.afm;
 
+import java.io.File;
 import java.io.InputStream;
 import java.io.IOException;
 
@@ -300,16 +301,16 @@ public class AFMParser
     {
         java.io.File afmDir = new java.io.File( "Resources/afm" );
         java.io.File[] files = afmDir.listFiles();
-        for( int i=0; i< files.length; i++ )
+        for (File file : files)
         {
-            if( files[i].getPath().toUpperCase().endsWith( ".AFM" ) )
+            if (file.getPath().toUpperCase().endsWith(".AFM"))
             {
                 long start = System.currentTimeMillis();
-                java.io.FileInputStream input = new java.io.FileInputStream( files[i] );
+                java.io.FileInputStream input = new java.io.FileInputStream(file);
                 AFMParser parser = new AFMParser( input );
                 parser.parse();
                 long stop = System.currentTimeMillis();
-                System.out.println( "Parsing:" + files[i].getPath() + " " + (stop-start) );
+                System.out.println("Parsing:" + file.getPath() + " " + (stop-start));
             }
         }
     }

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/util/autodetect/NativeFontDirFinder.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/util/autodetect/NativeFontDirFinder.java?rev=1603551&r1=1603550&r2=1603551&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/util/autodetect/NativeFontDirFinder.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/util/autodetect/NativeFontDirFinder.java Wed Jun 18 17:50:09 2014
@@ -38,9 +38,9 @@ public abstract class NativeFontDirFinde
         String[] searchableDirectories = getSearchableDirectories();
         if (searchableDirectories != null)
         {
-            for (int i = 0; i < searchableDirectories.length; i++)
+            for (String searchableDirectorie : searchableDirectories)
             {
-                File fontDir = new File(searchableDirectories[i]);
+                File fontDir = new File(searchableDirectorie);
                 if (fontDir.exists() && fontDir.canRead())
                 {
                     fontDirList.add(fontDir);