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 2025/10/05 12:44:27 UTC
svn commit: r1928961 - pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font
Author: tilman
Date: Sun Oct 5 12:44:27 2025
New Revision: 1928961
Log:
PDFBOX-5660: improve code, as suggested by Andreas Lehmkühler
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java Sun Oct 5 12:44:20 2025 (r1928960)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java Sun Oct 5 12:44:27 2025 (r1928961)
@@ -23,7 +23,7 @@ import java.io.ByteArrayOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.nio.charset.StandardCharsets;
-import java.util.ArrayList;
+import java.util.Arrays;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
@@ -55,22 +55,10 @@ abstract class TrueTypeEmbedder implemen
private static final String BASE25 = "BCDEFGHIJKLMNOPQRSTUVWXYZ";
// PDF spec required tables (if present), all others will be removed
- private static final List<String> TABLES = new ArrayList<>(10);
-
- static
- {
- TABLES.add("head");
- TABLES.add("hhea");
- TABLES.add("loca");
- TABLES.add("maxp");
- TABLES.add("cvt ");
- TABLES.add("prep");
- TABLES.add("glyf");
- TABLES.add("hmtx");
- TABLES.add("fpgm");
- // Windows ClearType
- TABLES.add("gasp");
- }
+ private static final List<String> TABLES =
+ Arrays.asList("head", "hhea", "loca", "maxp", "cvt ", "prep", "glyf", "hmtx", "fpgm",
+ // Windows ClearType
+ "gasp");
private final PDDocument document;
protected TrueTypeFont ttf;