ICEpdf
  1. ICEpdf
  2. PDF-126

OS Cmap parsing error.

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.0 - Beta
          • Fix Version/s: 4.0
          • Component/s: Core/Parsing
          • Labels:
            None
          • Environment:
            any

            Description

            The class org.icepdf.core.pobjects.fonts.ofont.CMap will only return a toUnicode cmap value if the cmap files has a cIdSystemInfo which is a bit strange. There is a simple fix for comment out the if statement on line 449 in the method toSelector(char charMap):char.

            I'll update the code with more information on the fix.

              Activity

              Ascending order - Click to sort in descending order
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #20182 Tue Jan 12 06:42:29 MST 2010 patrick.corless PDF-126 fixed a toUnicode error for embedded cmap files used for toUnicoe which prevent mappings from occuring if there was no cIdSystemInfo entry.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/fonts/ofont/CMap.java
              Patrick Corless created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Salesforce Case []
              Fix Version/s 4.0 [ 10222 ]
              Hide
              Permalink
              Patrick Corless added a comment -

              Removed check for a valid cidSystemInfo when processing a toSelector method call. I can't see any reason for this check to be made as all the important information for cmaps are in bfChars and bfRange definitions.

              Marking as fixed.

              Show
              Patrick Corless added a comment - Removed check for a valid cidSystemInfo when processing a toSelector method call. I can't see any reason for this check to be made as all the important information for cmaps are in bfChars and bfRange definitions. Marking as fixed.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Martin Wildam added a comment -

              Thank you very much! I can confirm that it works now (I tested it on Windows and Ubuntu).

              Show
              Martin Wildam added a comment - Thank you very much! I can confirm that it works now (I tested it on Windows and Ubuntu).
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Patrick Corless
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: