]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
FOP-2648: Code cleanup, use entryset
authorSimon Steiner <ssteiner@apache.org>
Fri, 16 Sep 2016 12:50:43 +0000 (12:50 +0000)
committerSimon Steiner <ssteiner@apache.org>
Fri, 16 Sep 2016 12:50:43 +0000 (12:50 +0000)
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@1761026 13f79535-47bb-0310-9956-ffa450edef68

25 files changed:
fop-core/src/main/java/org/apache/fop/afp/goca/AbstractGraphicsCoord.java
fop-core/src/main/java/org/apache/fop/area/inline/WordArea.java
fop-core/src/main/java/org/apache/fop/complexscripts/fonts/GlyphPositioningTable.java
fop-core/src/main/java/org/apache/fop/complexscripts/fonts/GlyphSubstitutionTable.java
fop-core/src/main/java/org/apache/fop/complexscripts/scripts/IndicScriptProcessor.java
fop-core/src/main/java/org/apache/fop/complexscripts/util/GlyphSequence.java
fop-core/src/main/java/org/apache/fop/complexscripts/util/NumberConverter.java
fop-core/src/main/java/org/apache/fop/fo/FOText.java
fop-core/src/main/java/org/apache/fop/fonts/CustomFont.java
fop-core/src/main/java/org/apache/fop/fonts/FontInfo.java
fop-core/src/main/java/org/apache/fop/fonts/truetype/OTFSubSetFile.java
fop-core/src/main/java/org/apache/fop/fonts/type1/Type1FontLoader.java
fop-core/src/main/java/org/apache/fop/layoutmgr/WhitespaceManagementPenalty.java
fop-core/src/main/java/org/apache/fop/layoutmgr/inline/LineLayoutManager.java
fop-core/src/main/java/org/apache/fop/pdf/PDFArray.java
fop-core/src/main/java/org/apache/fop/pdf/PDFFilterList.java
fop-core/src/main/java/org/apache/fop/pdf/PDFResources.java
fop-core/src/main/java/org/apache/fop/render/ImageHandlerRegistry.java
fop-core/src/main/java/org/apache/fop/render/intermediate/IFUtil.java
fop-core/src/main/java/org/apache/fop/render/pcl/fonts/truetype/PCLTTFFontReader.java
fop-core/src/main/java/org/apache/fop/util/ColorUtil.java
fop-core/src/test/java/org/apache/fop/apps/AFPRendererConfBuilder.java
fop-core/src/test/java/org/apache/fop/complexscripts/fonts/ttx/TTXFile.java
fop-core/src/test/java/org/apache/fop/fonts/CIDFullTestCase.java
fop-core/src/test/java/org/apache/fop/pdf/xref/CrossReferenceTableTestCase.java

index d9836f682b2ebefc1c93e9cd1e080c60a43410c8..92fd8658cc09b68876d187c9e46f3617a62d0d01 100644 (file)
@@ -140,7 +140,7 @@ public abstract class AbstractGraphicsCoord extends AbstractGraphicsDrawingOrder
             sb.append('=');
             sb.append(coords[i]);
         }
-        return getName() + "{" + sb.toString() + "}";
+        return getName() + "{" + sb + "}";
     }
 
     /**
index 24cc9478c48f50246ec5de9c4df498a44089342f..a00d7112b852435a474ca86de1f7eba3c2eaae71 100644 (file)
@@ -125,9 +125,7 @@ public class WordArea extends InlineArea {
         if (this.levels != null) {
             int n = end - start;
             int[] levels = new int [ n ];
-            for (int i = 0; i < n; i++) {
-                levels[i] = this.levels [ start + i ];
-            }
+            System.arraycopy(this.levels, start + 0, levels, 0, n);
             return levels;
         } else {
             return null;
index 4713f02cfab237138e3c344acbcfab2878155328..82a9f6afd54d16bef8196fe883caf7ae203fa78a 100644 (file)
@@ -21,6 +21,7 @@ package org.apache.fop.complexscripts.fonts;
 
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 
@@ -340,9 +341,7 @@ public class GlyphPositioningTable extends GlyphTable {
         public List getEntries() {
             if (values != null) {
                 List entries = new ArrayList(values.length);
-                for (Value value : values) {
-                    entries.add(value);
-                }
+                Collections.addAll(entries, values);
                 return entries;
             } else {
                 return null;
index 26e3a35c701f80d4c276f6d1bfd06142e547f1ff..9ca9ac328d5afb5cd033d19c832c97ac8c918475 100644 (file)
@@ -20,6 +20,7 @@
 package org.apache.fop.complexscripts.fonts;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 
@@ -506,9 +507,7 @@ public class GlyphSubstitutionTable extends GlyphTable {
         /** {@inheritDoc} */
         public List getEntries() {
             List entries = new ArrayList(gaa.length);
-            for (int[] aGaa : gaa) {
-                entries.add(aGaa);
-            }
+            Collections.addAll(entries, gaa);
             return entries;
         }
         /** {@inheritDoc} */
@@ -638,9 +637,7 @@ public class GlyphSubstitutionTable extends GlyphTable {
         /** {@inheritDoc} */
         public List getEntries() {
             List entries = new ArrayList(ligatureSets.length);
-            for (LigatureSet ligatureSet : ligatureSets) {
-                entries.add(ligatureSet);
-            }
+            Collections.addAll(entries, ligatureSets);
             return entries;
         }
         /** {@inheritDoc} */
index 9e7721121ae6024f487e25161994389ec8a673fe..93fd3e75ce1b87173b4c096713f9632016edc63c 100644 (file)
@@ -21,6 +21,7 @@ package org.apache.fop.complexscripts.scripts;
 
 import java.lang.reflect.Constructor;
 import java.lang.reflect.InvocationTargetException;
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Map;
@@ -241,9 +242,7 @@ public class IndicScriptProcessor extends DefaultScriptProcessor {
     };
     static {
         basicShapingFeatures = new HashSet<String>();
-        for (String s : BASIC_SHAPING_FEATURE_STRINGS) {
-            basicShapingFeatures.add(s);
-        }
+        Collections.addAll(basicShapingFeatures, BASIC_SHAPING_FEATURE_STRINGS);
     }
     private boolean isBasicShapingUse(GlyphTable.UseSpec us) {
         assert us != null;
@@ -265,9 +264,7 @@ public class IndicScriptProcessor extends DefaultScriptProcessor {
     };
     static {
         presentationFeatures = new HashSet<String>();
-        for (String s : PRESENTATION_FEATURE_STRINGS) {
-            presentationFeatures.add(s);
-        }
+        Collections.addAll(presentationFeatures, PRESENTATION_FEATURE_STRINGS);
     }
     private boolean isPresentationUse(GlyphTable.UseSpec us) {
         assert us != null;
index 72fc357f00242354a68dddff226418c405db919f..1c13a5efe9e0e049a4c604e5a6f3966055ce5a7d 100644 (file)
@@ -21,6 +21,7 @@ package org.apache.fop.complexscripts.util;
 
 import java.nio.IntBuffer;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 // CSOFF: LineLengthCheck
@@ -459,19 +460,13 @@ public class GlyphSequence implements Cloneable {
         if (na > 0) {
             List gl = new ArrayList(na);
             if (baa != null) {
-                for (CharAssociation aBaa : baa) {
-                    gl.add(aBaa);
-                }
+                Collections.addAll(gl, baa);
             }
             if (iaa != null) {
-                for (CharAssociation anIaa : iaa) {
-                    gl.add(anIaa);
-                }
+                Collections.addAll(gl, iaa);
             }
             if (laa != null) {
-                for (CharAssociation aLaa : laa) {
-                    gl.add(aLaa);
-                }
+                Collections.addAll(gl, laa);
             }
             return gl;
         } else {
index 6a21851fa0c9874fb2e651a254772297f5136238..48f9af9e36be28b2963f3ffc67043f6570abe04c 100644 (file)
@@ -20,6 +20,7 @@
 package org.apache.fop.complexscripts.util;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 // CSOFF: LineLengthCheck
@@ -478,9 +479,7 @@ public class NumberConverter {
     */
 
     private static void appendScalars(List<Integer> scalars, Integer[] sa) {
-        for (Integer s : sa) {
-            scalars.add(s);
-        }
+        Collections.addAll(scalars, sa);
     }
 
     private static String scalarsToString(List<Integer> scalars) {
index 9f286d888127a07bf33d04c9b27f7ae46501ce80..8b65e6891c7a4279b44909313f51c1e44081cf47 100644 (file)
@@ -801,9 +801,7 @@ public class FOText extends FONode implements CharSequence, TextFragment {
             assert start <= end;
             int n = end - start;
             int[] bidiLevels = new int [ n ];
-            for (int i = 0; i < n; i++) {
-                bidiLevels[i] = this.bidiLevels [ start + i ];
-            }
+            System.arraycopy(this.bidiLevels, start + 0, bidiLevels, 0, n);
             return bidiLevels;
         } else {
             return null;
index d19f2161f8a66adcefc05a36b69671d6d30d2de9..ed9f1f0397539193dcc120b5b096d8919032d3a9 100644 (file)
@@ -530,9 +530,7 @@ public abstract class CustomFont extends Typeface
      */
     public void setCMap(CMapSegment[] cmap) {
         this.cmap.clear();
-        for (CMapSegment c : cmap) {
-            this.cmap.add(c);
-        }
+        Collections.addAll(this.cmap, cmap);
     }
 
     /**
index e548448291bc6f71d14eeb662b8142d67022487b..7e44d56aea9d1a48c3a4cdc5da1e55c26a424bb5 100644 (file)
@@ -428,7 +428,7 @@ public class FontInfo {
             }
             throw new IllegalStateException(
                     "fontLookup must return an array with at least one "
-                            + "FontTriplet on the last call. Lookup: " + sb.toString());
+                            + "FontTriplet on the last call. Lookup: " + sb);
 
         }
         FontTriplet[] fontTriplets = new FontTriplet[matchedTriplets.size()];
index d8e363e9940c41d85f4a1ddc7b724d9a09987109..c28bca5fe8bf68fc33d8402a543a9df2c697a516 100644 (file)
@@ -334,13 +334,14 @@ public class OTFSubSetFile extends OTFFile {
 
         gidToSID = new LinkedHashMap<Integer, Integer>();
 
-        for (int gid : subsetGlyphs.keySet()) {
+        for (Entry<Integer, Integer> subsetGlyph : subsetGlyphs.entrySet()) {
+            int gid = subsetGlyph.getKey();
             int sid = cffReader.getSIDFromGID(charsetOffset, gid);
             //Check whether the SID falls into the standard string set
             if (sid < NUM_STANDARD_STRINGS) {
-                gidToSID.put(subsetGlyphs.get(gid), sid);
+                gidToSID.put(subsetGlyph.getValue(), sid);
                 if (mbFont != null) {
-                    mbFont.mapUsedGlyphName(subsetGlyphs.get(gid),
+                    mbFont.mapUsedGlyphName(subsetGlyph.getValue(),
                             CFFStandardString.getName(sid));
                 }
             } else {
@@ -348,16 +349,16 @@ public class OTFSubSetFile extends OTFFile {
                 //index is 0 based, should use < not <=
                 if (index < cffReader.getStringIndex().getNumObjects()) {
                     if (mbFont != null) {
-                        mbFont.mapUsedGlyphName(subsetGlyphs.get(gid),
+                        mbFont.mapUsedGlyphName(subsetGlyph.getValue(),
                                 new String(cffReader.getStringIndex().getValue(index)));
                     }
-                    gidToSID.put(subsetGlyphs.get(gid), stringIndexData.size() + 391);
+                    gidToSID.put(subsetGlyph.getValue(), stringIndexData.size() + 391);
                     stringIndexData.add(cffReader.getStringIndex().getValue(index));
                 } else {
                     if (mbFont != null) {
-                        mbFont.mapUsedGlyphName(subsetGlyphs.get(gid), ".notdef");
+                        mbFont.mapUsedGlyphName(subsetGlyph.getValue(), ".notdef");
                     }
-                    gidToSID.put(subsetGlyphs.get(gid), index);
+                    gidToSID.put(subsetGlyph.getValue(), index);
                 }
             }
         }
@@ -410,7 +411,8 @@ public class OTFSubSetFile extends OTFFile {
                 foundLocalUniques.add(new ArrayList<Integer>());
             }
             Map<Integer, Integer> gidHintMaskLengths = new HashMap<Integer, Integer>();
-            for (int gid : subsetGlyphs.keySet()) {
+            for (Entry<Integer, Integer> subsetGlyph : subsetGlyphs.entrySet()) {
+                int gid = subsetGlyph.getKey();
                 int group = subsetGroups.get(gid);
                 localIndexSubr = cffReader.getFDFonts().get(group).getLocalSubrData();
                 localUniques = foundLocalUniques.get(uniqueGroups.indexOf(subsetGroups.get(gid)));
@@ -418,7 +420,7 @@ public class OTFSubSetFile extends OTFFile {
 
                 FDIndexReference newFDReference = new FDIndexReference(
                         uniqueGroups.indexOf(subsetGroups.get(gid)), subsetGroups.get(gid));
-                subsetFDSelect.put(subsetGlyphs.get(gid), newFDReference);
+                subsetFDSelect.put(subsetGlyph.getValue(), newFDReference);
                 byte[] data = charStringsIndex.getValue(gid);
                 preScanForSubsetIndexSize(data);
                 gidHintMaskLengths.put(gid, type2Parser.getMaskLength());
@@ -439,13 +441,15 @@ public class OTFSubSetFile extends OTFFile {
             for (Integer uniqueGroup : uniqueGroups) {
                 foundLocalUniquesB.add(new ArrayList<Integer>());
             }
-            for (Integer gid : subsetGlyphs.keySet()) {
+            for (Entry<Integer, Integer> subsetGlyph : subsetGlyphs.entrySet()) {
+                int gid = subsetGlyph.getKey();
+                int value = subsetGlyph.getValue();
                 int group = subsetGroups.get(gid);
                 localIndexSubr = cffReader.getFDFonts().get(group).getLocalSubrData();
-                localUniques = foundLocalUniquesB.get(subsetFDSelect.get(subsetGlyphs.get(gid)).getNewFDIndex());
+                localUniques = foundLocalUniquesB.get(subsetFDSelect.get(value).getNewFDIndex());
                 byte[] data = charStringsIndex.getValue(gid);
-                subsetLocalIndexSubr = fdSubrs.get(subsetFDSelect.get(subsetGlyphs.get(gid)).getNewFDIndex());
-                subsetLocalSubrCount = foundLocalUniques.get(subsetFDSelect.get(subsetGlyphs.get(gid))
+                subsetLocalIndexSubr = fdSubrs.get(subsetFDSelect.get(value).getNewFDIndex());
+                subsetLocalSubrCount = foundLocalUniques.get(subsetFDSelect.get(value)
                         .getNewFDIndex()).size();
                 type2Parser = new Type2Parser();
                 type2Parser.setMaskLength(gidHintMaskLengths.get(gid));
@@ -457,15 +461,15 @@ public class OTFSubSetFile extends OTFFile {
 
     protected void writeFDSelect() {
         writeByte(0); //Format
-        for (Integer gid : subsetFDSelect.keySet()) {
-            writeByte(subsetFDSelect.get(gid).getNewFDIndex());
+        for (FDIndexReference e : subsetFDSelect.values()) {
+            writeByte(e.getNewFDIndex());
         }
     }
 
     protected List<Integer> getUsedFDFonts() {
         List<Integer> uniqueNewRefs = new ArrayList<Integer>();
-        for (int gid : subsetFDSelect.keySet()) {
-            int fdIndex = subsetFDSelect.get(gid).getOldFDIndex();
+        for (FDIndexReference e : subsetFDSelect.values()) {
+            int fdIndex = e.getOldFDIndex();
             if (!uniqueNewRefs.contains(fdIndex)) {
                 uniqueNewRefs.add(fdIndex);
             }
@@ -1023,11 +1027,11 @@ public class OTFSubSetFile extends OTFFile {
 
     private void writeCharsetTable(boolean cidFont) throws IOException {
         writeByte(0);
-        for (int gid : gidToSID.keySet()) {
-            if (cidFont && gid == 0) {
+        for (Entry<Integer, Integer> entry : gidToSID.entrySet()) {
+            if (cidFont && entry.getKey() == 0) {
                 continue;
             }
-            writeCard16((cidFont) ? gid : gidToSID.get(gid));
+            writeCard16((cidFont) ? entry.getKey() : entry.getValue());
         }
     }
 
index d364462cc595a58bd8e97159e0151076957d77b6..f11585cfbd8c1d8d87b91cfbacf291b492956654 100644 (file)
@@ -23,6 +23,7 @@ import java.awt.geom.RectangularShape;
 import java.io.IOException;
 import java.io.InputStream;
 import java.net.URISyntaxException;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
@@ -215,9 +216,7 @@ public class Type1FontLoader extends FontLoader {
 
     private Set<String> toGlyphSet(String[] glyphNames) {
         Set<String> glyphSet = new java.util.HashSet<String>();
-        for (String name : glyphNames) {
-            glyphSet.add(name);
-        }
+        Collections.addAll(glyphSet, glyphNames);
         return glyphSet;
     }
 
index e34d24b9af7b0d0b6fba77df648aca9f74cb13eb..f8725e7ceb0bd9e5c5b12aa760bc808eb35e5add 100644 (file)
@@ -84,7 +84,7 @@ public class WhitespaceManagementPenalty extends KnuthPenalty {
         String str = super.toString();
         StringBuffer buffer = new StringBuffer(64);
         buffer.append(" number of variants = " + variantList.size());
-        return str + buffer.toString();
+        return str + buffer;
     }
 
 }
index 40c7703ae2513a917a756b3ec9fc1d90aa883fff..8ee5d1e702250a0c6376aaee469a5636b0cea22f 100644 (file)
@@ -1326,7 +1326,7 @@ public class LineLayoutManager extends InlineStackingLayoutManager
                     }
                 }
                 if (log.isTraceEnabled()) {
-                    log.trace(" Word to hyphenate: " + sbChars.toString());
+                    log.trace(" Word to hyphenate: " + sbChars);
                 }
                 // find hyphenation points
                 HyphContext hc = getHyphenContext(sbChars);
index bfcc1c7e0da40555945d38d3f104f06596176f25..f0cb5197ce5dac72def69c6a83fceef042f9f68a 100644 (file)
@@ -22,6 +22,7 @@ package org.apache.fop.pdf;
 import java.io.IOException;
 import java.io.OutputStream;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import java.util.Set;
 
@@ -119,9 +120,7 @@ public class PDFArray extends PDFObject {
         /* generic creation of PDF object */
         super(parent);
 
-        for (Object value : values) {
-            this.values.add(value);
-        }
+        Collections.addAll(this.values, values);
     }
 
     /**
index fbb7d57a6db8590c19b68d49a35e9d250c68c080..d3f0f0ee38d2d9935ba2fedd88ec89c496930cd6 100644 (file)
@@ -278,9 +278,9 @@ public class PDFFilterList {
         }
         if (filterCount > 0) {
             if (filterCount > 1) {
-                return "/Filter [ " + sb.toString() + "]";
+                return "/Filter [ " + sb + "]";
             } else {
-                return "/Filter " + sb.toString();
+                return "/Filter " + sb;
             }
         } else {
             return "";
index 9ba9c7f8fb09464eab69279c35c78ef98659b00c..96d45a13a43a257d5db3c8957459e69de4fb8f81 100644 (file)
@@ -308,8 +308,8 @@ public class PDFResources extends PDFDictionary {
 
         if (!properties.isEmpty()) {
             PDFDictionary dict = new PDFDictionary(this);
-            for (String name : properties.keySet()) {
-                dict.put(name, properties.get(name));
+            for (Map.Entry<String, PDFReference> stringPDFReferenceEntry : properties.entrySet()) {
+                dict.put(stringPDFReferenceEntry.getKey(), stringPDFReferenceEntry.getValue());
             }
             put("Properties", dict);
         }
index 2ef22739f10521dbbb31b351e8754642bc2bd9a0..0a4b5e158f1a076dc9cc5b1f8aa5b1495c836572 100644 (file)
@@ -19,6 +19,7 @@
 
 package org.apache.fop.render;
 
+import java.util.Collections;
 import java.util.Comparator;
 import java.util.Iterator;
 import java.util.List;
@@ -142,9 +143,7 @@ public class ImageHandlerRegistry {
         for (ImageHandler handler : this.handlerList) {
             if (handler.isCompatible(context, null)) {
                 ImageFlavor[] f = handler.getSupportedImageFlavors();
-                for (ImageFlavor aF : f) {
-                    flavors.add(aF);
-                }
+                Collections.addAll(flavors, f);
             }
         }
         return flavors.toArray(new ImageFlavor[flavors.size()]);
index b877abad5dedd499302fcf5707106a70e62d049b..a1e85afb137f5e42acdd6460da2fa28f84f44637 100644 (file)
@@ -375,9 +375,7 @@ public final class IFUtil {
                 int[] paSrc = dp [ i + offset ];
                 if (paSrc != null) {
                     int[] paDst = new int [ 4 ];
-                    for (int k = 0; k < 4; k++) {
-                        paDst [ k ] = paSrc [ k ];
-                    }
+                    System.arraycopy(paSrc, 0, paDst, 0, 4);
                     dpNew [ i ] = paDst;
                 }
             }
index 6839bf94857c003d493809e937d56f130fd5b175..142262ade415aa6a721cf7dff940a9e4f70739d8 100644 (file)
@@ -605,8 +605,8 @@ public class PCLTTFFontReader extends PCLFontReader {
     private ByteArrayOutputStream updateOffsets(ByteArrayOutputStream baos, Map<Integer, byte[]> offsets)
             throws IOException {
         byte[] softFont = baos.toByteArray();
-        for (int offset : offsets.keySet()) {
-            PCLByteWriterUtil.updateDataAtLocation(softFont, offsets.get(offset), offset);
+        for (Entry<Integer, byte[]> integerEntry : offsets.entrySet()) {
+            PCLByteWriterUtil.updateDataAtLocation(softFont, integerEntry.getValue(), integerEntry.getKey());
         }
         baos = new ByteArrayOutputStream();
         baos.write(softFont);
index cf534c49f81a63582dc50a5764f20daf2238f8a9..1117b6e38d11bfb78601a6f7fbd8118bd81e3ecc 100644 (file)
@@ -798,7 +798,7 @@ public final class ColorUtil {
             }
         }
         sb.append(")");
-        return functionName + sb.toString();
+        return functionName + sb;
     }
 
     private static String toCIELabFunctionCall(ColorWithAlternatives color) {
index f108e4d44719bfc9826d871701cd9f53881ca309..580bb23d0509c7834ededa95a63108ff929c9627 100644 (file)
@@ -94,8 +94,8 @@ public final class AFPRendererConfBuilder extends RendererConfBuilder {
 
     public AFPRendererConfBuilder setDefaultResourceLevels(Map<String, String> levels) {
         Element e = createElement(DEFAULT_RESOURCE_LEVELS.getName());
-        for (String key : levels.keySet()) {
-            e.setAttribute(key, levels.get(key));
+        for (Map.Entry<String, String> stringStringEntry : levels.entrySet()) {
+            e.setAttribute(stringStringEntry.getKey(), stringStringEntry.getValue());
         }
         return this;
     }
index 41155613873d5d1d77ea2b677772111d4e3d6809..c94b0bcb0aa3b38122e98e1d6dd281d9ff532a2c 100644 (file)
@@ -2804,11 +2804,11 @@ public class TTXFile {
                 }
             }
             GlyphCoverageTable[] gca = new GlyphCoverageTable [ mi + 1 ];
-            for (String k : keys) {
-                if (k.startsWith(prefix)) {
-                    int i = Integer.parseInt(k.substring(prefixLength));
+            for (Map.Entry<String, GlyphCoverageTable> stringGlyphCoverageTableEntry : coverages.entrySet()) {
+                if (stringGlyphCoverageTableEntry.getKey().startsWith(prefix)) {
+                    int i = Integer.parseInt(stringGlyphCoverageTableEntry.getKey().substring(prefixLength));
                     if (i >= 0) {
-                        gca [ i ] = coverages.get(k);
+                        gca [ i ] = stringGlyphCoverageTableEntry.getValue();
                     }
                 }
             }
@@ -2915,11 +2915,11 @@ public class TTXFile {
         }
         private Map<GlyphTable.LookupSpec, List<String>> extractLookups() {
             Map<GlyphTable.LookupSpec, List<String>> lookups = new LinkedHashMap<GlyphTable.LookupSpec, List<String>>();
-            for (String st : scripts.keySet()) {
-                Map<String, List<String>> lm = scripts.get(st);
+            for (Map.Entry<String, Map<String, List<String>>> stringMapEntry : scripts.entrySet()) {
+                Map<String, List<String>> lm = stringMapEntry.getValue();
                 if (lm != null) {
-                    for (String lt : lm.keySet()) {
-                        List<String> fids = lm.get(lt);
+                    for (Map.Entry<String, List<String>> stringListEntry : lm.entrySet()) {
+                        List<String> fids = stringListEntry.getValue();
                         if (fids != null) {
                             for (String fid : fids) {
                                 if (fid != null) {
@@ -2929,7 +2929,7 @@ public class TTXFile {
                                         String ft = (String) fa[0];
                                         List<String> lids = (List<String>) fa[1];
                                         if ((lids != null) && (lids.size() > 0)) {
-                                            GlyphTable.LookupSpec ls = new GlyphTable.LookupSpec(st, lt, ft);
+                                            GlyphTable.LookupSpec ls = new GlyphTable.LookupSpec(stringMapEntry.getKey(), stringListEntry.getKey(), ft);
                                             lookups.put(ls, lids);
                                         }
                                     }
index 73ded6875f50b28d2928b20d3d057527ecd0a599..e73c29569ef0f75e7be08c53eb9d3d110f3978f6 100644 (file)
@@ -91,8 +91,8 @@ public class CIDFullTestCase {
     @Test
     public void testGetGlyphs() {
         Map<Integer, Integer> fontGlyphs = cidFull.getGlyphs();
-        for (Integer key : fontGlyphs.keySet()) {
-            assertEquals(fontGlyphs.get(key), glyphs.get(key));
+        for (Map.Entry<Integer, Integer> integerIntegerEntry : fontGlyphs.entrySet()) {
+            assertEquals(integerIntegerEntry.getValue(), glyphs.get(integerIntegerEntry.getKey()));
         }
         assertTrue(fontGlyphs.size() == glyphs.size());
     }
index 1c609e30f0ce92908a0e063871ba722865c62959..1f927587950f25b5a79e9ede3894d6f32caf9a8b 100644 (file)
@@ -64,7 +64,7 @@ public class CrossReferenceTableTestCase extends CrossReferenceObjectTest {
                 .append("\n0000000000 65535 f \n");
         for (Long objectReference : offsets) {
             final String padding = "0000000000";
-            String s = String.valueOf(objectReference).toString();
+            String s = String.valueOf(objectReference);
             String loc = padding.substring(s.length()) + s;
             expected.append(loc).append(" 00000 n \n");
         }