You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2018/03/04 15:57:39 UTC

svn commit: r1825820 - in /pdfbox/branches/2.0/fontbox/src: main/java/org/apache/fontbox/cmap/CMapParser.java test/java/org/apache/fontbox/cmap/TestCMapParser.java

Author: lehmi
Date: Sun Mar  4 15:57:39 2018
New Revision: 1825820

URL: http://svn.apache.org/viewvc?rev=1825820&view=rev
Log:
PDFBOX-4139: fixed swapped values, added test

Modified:
    pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java
    pdfbox/branches/2.0/fontbox/src/test/java/org/apache/fontbox/cmap/TestCMapParser.java

Modified: pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java?rev=1825820&r1=1825819&r2=1825820&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java Sun Mar  4 15:57:39 2018
@@ -305,7 +305,7 @@ public class CMapParser
                 // some CMaps are using CID ranges to map single values
                 if (end == start)
                 {
-                    result.addCIDMapping(start, mappedCode);
+                    result.addCIDMapping(mappedCode, start);
                 }
                 else
                 {

Modified: pdfbox/branches/2.0/fontbox/src/test/java/org/apache/fontbox/cmap/TestCMapParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/test/java/org/apache/fontbox/cmap/TestCMapParser.java?rev=1825820&r1=1825819&r2=1825820&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/test/java/org/apache/fontbox/cmap/TestCMapParser.java (original)
+++ pdfbox/branches/2.0/fontbox/src/test/java/org/apache/fontbox/cmap/TestCMapParser.java Sun Mar  4 15:57:39 2018
@@ -89,6 +89,17 @@ public class TestCMapParser extends Test
         assertEquals("Indentity-H CID 0xFFFF", 0xFFFF, cMap.toCID(0xFFFF));
     }
 
+    public void testUniJIS_UCS2_H() throws IOException
+    {
+        final String resourceDir = "src/main/resources/org/apache/fontbox/cmap";
+        File inDir = new File(resourceDir);
+
+        CMapParser parser = new CMapParser();
+        CMap cMap = parser.parse(new File(inDir, "UniJIS-UCS2-H"));
+
+        assertEquals("UniJIS-UCS2-H CID 65 -> 34", 34, cMap.toCID(65));
+    }
+
     /**
      * Test the parser against a valid, but poorly formatted CMap file.
      * @throws IOException If something went wrong