Author: tilman
Date: Sun Oct  5 12:44:16 2025
New Revision: 1928959

Log:
PDFBOX-5660: improve code, as suggested by Andreas Lehmkühler

Modified:
   
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java

Modified: 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
==============================================================================
--- 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
       Sun Oct  5 12:12:19 2025        (r1928958)
+++ 
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
       Sun Oct  5 12:44:16 2025        (r1928959)
@@ -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;

Reply via email to