svn commit: r1936378 - pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub

[tilman]

Author: tilman
Date: Mon Jul 20 13:36:43 2026
New Revision: 1936378

Log:
PDFBOX-6220: cache splitter, as suggested by Olivier Bruchez

Modified:
   
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForDflt.java
   
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForLatin.java

Modified: 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForDflt.java
==============================================================================
--- 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForDflt.java
        Mon Jul 20 13:33:29 2026        (r1936377)
+++ 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForDflt.java
        Mon Jul 20 13:36:43 2026        (r1936378)
@@ -22,6 +22,7 @@ import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 import java.util.WeakHashMap;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
@@ -101,7 +102,8 @@ public class GsubWorkerForDflt implement
     private List<Integer> applyGsubFeature(ScriptFeature scriptFeature,
             List<Integer> originalGlyphs)
     {
-        if (scriptFeature.getAllGlyphIdsForSubstitution().isEmpty())
+        Set<List<Integer>> allGlyphIdsForSubstitution = 
scriptFeature.getAllGlyphIdsForSubstitution();
+        if (allGlyphIdsForSubstitution.isEmpty())
         {
             LOG.debug("getAllGlyphIdsForSubstitution() for " + 
scriptFeature.getName() + " is empty");
             return originalGlyphs;

Modified: 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForLatin.java
==============================================================================
--- 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForLatin.java
       Mon Jul 20 13:33:29 2026        (r1936377)
+++ 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForLatin.java
       Mon Jul 20 13:36:43 2026        (r1936378)
@@ -22,6 +22,7 @@ import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 import java.util.WeakHashMap;
 
 import org.apache.commons.logging.Log;
@@ -83,7 +84,8 @@ public class GsubWorkerForLatin implemen
     private List<Integer> applyGsubFeature(ScriptFeature scriptFeature,
             List<Integer> originalGlyphs)
     {
-        if (scriptFeature.getAllGlyphIdsForSubstitution().isEmpty())
+        Set<List<Integer>> allGlyphIdsForSubstitution = 
scriptFeature.getAllGlyphIdsForSubstitution();
+        if (allGlyphIdsForSubstitution.isEmpty())
         {
             LOG.debug("getAllGlyphIdsForSubstitution() for " + 
scriptFeature.getName() + " is empty");
             return originalGlyphs;