Author: tilman Date: Sun Sep 14 05:27:10 2025 New Revision: 1928454 Log: PDFBOX-5660: optimize, as suggested by Valery Bokov; closes #253
Modified: pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDTrueTypeFontEmbedder.java Modified: pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDTrueTypeFontEmbedder.java ============================================================================== --- pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDTrueTypeFontEmbedder.java Sun Sep 14 05:27:06 2025 (r1928453) +++ pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDTrueTypeFontEmbedder.java Sun Sep 14 05:27:10 2025 (r1928454) @@ -23,6 +23,8 @@ import java.util.ArrayList; import java.util.Collections; import java.util.List; import java.util.Map; +import java.util.Set; + import org.apache.fontbox.ttf.HorizontalMetricsTable; import org.apache.fontbox.ttf.TrueTypeFont; import org.apache.pdfbox.cos.COSArray; @@ -79,9 +81,9 @@ final class PDTrueTypeFontEmbedder exten HorizontalMetricsTable hmtx = ttf.getHorizontalMetrics(); Map<Integer, String> codeToName = getFontEncoding().getCodeToNameMap(); - - int firstChar = Collections.min(codeToName.keySet()); - int lastChar = Collections.max(codeToName.keySet()); + Set<Integer> codes = codeToName.keySet(); + int firstChar = Collections.min(codes); + int lastChar = Collections.max(codes); List<Integer> widths = new ArrayList<>(lastChar - firstChar + 1); for (int i = 0; i < lastChar - firstChar + 1; i++)