ICEpdf
  1. ICEpdf
  2. PDF-774

CCITTfax image not loading with JAI enabled.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.6_P01
    • Fix Version/s: 5.1
    • Component/s: Core/Parsing
    • Labels:
      None
    • Environment:
      any

      Description

      The file in question does not load if -Dorg.icepdf.core.ccittfax.jai=true. The image will be shown as solid black or white depending on the ICEpdf vesrion.

        Activity

        Patrick Corless created issue -
        Patrick Corless made changes -
        Field Original Value New Value
        Attachment Scan_Of_Font_List__AllBlac.pdf [ 17264 ]
        Arran Mccullough made changes -
        Support Case References Support Case #12907 - https://icesoft.my.salesforce.com/5007000000dugM6
        Patrick Corless made changes -
        Fix Version/s 5.1 [ 10675 ]
        Hide
        Patrick Corless added a comment -

        This appears to be fixed as the result of the image consolidation work done for PDF-322 around consolidation of imaging algorithms.

        Show
        Patrick Corless added a comment - This appears to be fixed as the result of the image consolidation work done for PDF-322 around consolidation of imaging algorithms.
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Patrick Corless added a comment -

        The imaging pipeline for ICEpdf was reworked to use a pure JAI approach for loading images. In previous versions we had a dependency on the older now deprecated JpegCodec class which was removed in JDK 1.8. During this work quite a bit of common code was consolidate and refactored creating a new package for Raster operations.

        During our tests on Linux and Windows the image question loaded correctly regardless of the system property "org.icepdf.core.ccittfax.jai".

        Show
        Patrick Corless added a comment - The imaging pipeline for ICEpdf was reworked to use a pure JAI approach for loading images. In previous versions we had a dependency on the older now deprecated JpegCodec class which was removed in JDK 1.8. During this work quite a bit of common code was consolidate and refactored creating a new package for Raster operations. During our tests on Linux and Windows the image question loaded correctly regardless of the system property "org.icepdf.core.ccittfax.jai".
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #42329 Fri Aug 29 10:29:27 MDT 2014 patrick.corless PDF-774 added logging for which ImageIO tiff image readers are present. Also insured exception buble up to the top so we can fall back on our on CCITTFax implementation if JAI fails.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/filters/CCITTFax.java
        Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/ImageStream.java
        Hide
        Patrick Corless added a comment -

        I have to get some exact memory numbers but by introducing an inline image cache I can reduce the total number of images on the first page 32,581 inline images down to a combined cache that has only 3,593 elements.

        Show
        Patrick Corless added a comment - I have to get some exact memory numbers but by introducing an inline image cache I can reduce the total number of images on the first page 32,581 inline images down to a combined cache that has only 3,593 elements.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #42537 Tue Sep 09 15:42:43 MDT 2014 patrick.corless PDF-774 updated how we create the buffered image when doing the image scale, this avoid any colour model incompatibilities.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/ScaledImageReference.java
        Hide
        Patrick Corless added a comment -

        Updated the ScaledImageReference to create a generally RGBA or RGB image instead of trying to reuse the color model. This avoid the incompatibility issue the the image is now property rendered during the scale copy.

        Show
        Patrick Corless added a comment - Updated the ScaledImageReference to create a generally RGBA or RGB image instead of trying to reuse the color model. This avoid the incompatibility issue the the image is now property rendered during the scale copy.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #42540 Wed Sep 10 08:17:01 MDT 2014 patrick.corless PDF-774 removed redundant null check.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/ScaledImageReference.java
        Patrick Corless made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: