svn commit: r1917437 - in /pdfbox/branches/3.0/fontbox/src/test/java/org/apache/fontbox/ttf/gsub: GsubWorkerForAalt.java GsubWorkerForAaltTest.java

[ti...@apache.org]

Author: tilman
Date: Tue Apr 30 11:59:03 2024
New Revision: 1917437

URL: http://svn.apache.org/viewvc?rev=1917437&view=rev
Log:
PDFBOX-5808: add test for aalt

Added:
    pdfbox/branches/3.0/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java
      - copied, changed from r1917436, pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java
    pdfbox/branches/3.0/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAaltTest.java
      - copied unchanged from r1917436, pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAaltTest.java

Copied: pdfbox/branches/3.0/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java (from r1917436, pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java)
URL: http://svn.apache.org/viewvc/pdfbox/branches/3.0/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java?p2=pdfbox/branches/3.0/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java&p1=pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java&r1=1917436&r2=1917437&rev=1917437&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java (original)
+++ pdfbox/branches/3.0/fontbox/src/test/java/org/apache/fontbox/ttf/gsub/GsubWorkerForAalt.java Tue Apr 30 11:59:03 2024
@@ -22,12 +22,13 @@ import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 import org.apache.fontbox.ttf.CmapLookup;
 import org.apache.fontbox.ttf.model.GsubData;
 import org.apache.fontbox.ttf.model.ScriptFeature;
 
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
 
 /**
  * 
@@ -39,7 +40,7 @@ import org.apache.logging.log4j.Logger;
  */
 public class GsubWorkerForAalt implements GsubWorker
 {
-    private static final Logger LOG = LogManager.getLogger(GsubWorkerForAalt.class);
+    private static final Log LOG = LogFactory.getLog(GsubWorkerForAalt.class);
 
     private static final List<String> FEATURES_IN_ORDER = Arrays.asList("aalt");
 
@@ -61,11 +62,11 @@ public class GsubWorkerForAalt implement
         {
             if (!gsubData.isFeatureSupported(feature))
             {
-                LOG.debug("the feature {} was not found", feature);
+                LOG.debug("the feature " + feature + " was not found");
                 continue;
             }
 
-            LOG.debug("applying the feature {}", feature);
+            LOG.debug("applying the feature " + feature);
 
             ScriptFeature scriptFeature = gsubData.getFeature(feature);
 
@@ -81,8 +82,7 @@ public class GsubWorkerForAalt implement
     {
         if (scriptFeature.getAllGlyphIdsForSubstitution().isEmpty())
         {
-            LOG.debug("getAllGlyphIdsForSubstitution() for {} is empty",
-                        scriptFeature.getName());
+            LOG.debug("getAllGlyphIdsForSubstitution() for " + scriptFeature.getName() + " is empty");
             return originalGlyphs;
         }
         
@@ -106,7 +106,8 @@ public class GsubWorkerForAalt implement
             }
         }
 
-        LOG.debug("originalGlyphs: {}, gsubProcessedGlyphs: {}", originalGlyphs, gsubProcessedGlyphs);
+        LOG.debug("originalGlyphs: " + originalGlyphs + ", gsubProcessedGlyphs: "
+                + gsubProcessedGlyphs);
 
         return gsubProcessedGlyphs;
     }