svn commit: r1696521 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java

[msahyoun]

Author: msahyoun
Date: Wed Aug 19 06:19:20 2015
New Revision: 1696521

URL: http://svn.apache.org/r1696521
Log:
PDFBOX-2852: use entrySet instead of keySet to retrieve Map values

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java

Modified: 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java
URL: 
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java?rev=1696521&r1=1696520&r2=1696521&view=diff
==============================================================================
--- 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java 
(original)
+++ 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java 
Wed Aug 19 06:19:20 2015
@@ -1616,17 +1616,17 @@ public class COSParser extends BaseParse
         if (bfSearchCOSObjectKeyOffsets != null)
         {
             xrefTrailerResolver.nextXrefObj( 0, XRefType.TABLE );
-            for (COSObjectKey objectKey : bfSearchCOSObjectKeyOffsets.keySet())
+            for (Entry<COSObjectKey, Long> entry : 
bfSearchCOSObjectKeyOffsets.entrySet())
             {
-                xrefTrailerResolver.setXRef(objectKey, 
bfSearchCOSObjectKeyOffsets.get(objectKey));
+                xrefTrailerResolver.setXRef(entry.getKey(), entry.getValue());
             }
             xrefTrailerResolver.setStartxref(0);
             trailer = xrefTrailerResolver.getTrailer();
             getDocument().setTrailer(trailer);
             // search for the different parts of the trailer dictionary 
-            for(COSObjectKey key : bfSearchCOSObjectKeyOffsets.keySet())
+            for(Entry<COSObjectKey, Long> entry : 
bfSearchCOSObjectKeyOffsets.entrySet())
             {
-                Long offset = bfSearchCOSObjectKeyOffsets.get(key);
+                Long offset = entry.getValue();
                 source.seek(offset);
                 readObjectNumber();
                 readGenerationNumber();
@@ -1639,7 +1639,7 @@ public class COSParser extends BaseParse
                         // document catalog
                         if 
(COSName.CATALOG.equals(dictionary.getCOSName(COSName.TYPE)))
                         {
-                            trailer.setItem(COSName.ROOT, 
document.getObjectFromPool(key));
+                            trailer.setItem(COSName.ROOT, 
document.getObjectFromPool(entry.getKey()));
                         }
                         // info dictionary
                         else if (dictionary.containsKey(COSName.TITLE)
@@ -1650,14 +1650,14 @@ public class COSParser extends BaseParse
                                 || dictionary.containsKey(COSName.PRODUCER)
                                 || 
dictionary.containsKey(COSName.CREATION_DATE))
                         {
-                            trailer.setItem(COSName.INFO, 
document.getObjectFromPool(key));
+                            trailer.setItem(COSName.INFO, 
document.getObjectFromPool(entry.getKey()));
                         }
                         // TODO encryption dictionary
                     }
                 }
                 catch(IOException exception)
                 {
-                    LOG.debug("Skipped object "+key+", either it's corrupt or 
not a dictionary");
+                    LOG.debug("Skipped object " + entry.getKey() + ", either 
it's corrupt or not a dictionary");
                 }
             }
         }