Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.1.1
    • Fix Version/s: 5.1
    • Component/s: Core/Parsing
    • Labels:
      None
    • Environment:
       * Searching for icedtea ...
      [I-O] [ ] dev-java/icedtea-6.1.9.1:6
    • Assignee Priority:
      P1

      Description

      The ICEpdf lib uses a intenal class from sun's internl function
      ...
           [javac] JPEGImageDecoder imageDecoder = JPEGCodec.createJPEGDecoder(bufferedInput);
          [javac] ^
          [javac] /var/tmp/portage/dev-java/ICEpdf-4.1.1/work/ICEpdf-4.1.1-src/icepdf/core/src/org/icepdf/core/pobjects/Stream.java:600: warning: com.sun.image.codec.jpeg.JPEGCodec is internal proprietary API and may be removed in a future release
          [javac] JPEGImageDecoder imageDecoder = JPEGCodec.createJPEGDecoder(bufferedInput);
          [javac] ^
          [javac] /var/tmp/portage/dev-java/ICEpdf-4.1.1/work/ICEpdf-4.1.1-src/icepdf/core/src/org/icepdf/core/pobjects/Stream.java:659: cannot find symbol
          [javac] symbol : method getEncodedColorID()
          [javac] location: interface com.sun.image.codec.jpeg.JPEGDecodeParam
          [javac] if (imageDecoder.getJPEGDecodeParam().getEncodedColorID() ==
          [javac] ^
          [javac] /var/tmp/portage/dev-java/ICEpdf-4.1.1/work/ICEpdf-4.1.1-src/icepdf/core/src/org/icepdf/core/pobjects/Stream.java:660: warning: com.sun.image.codec.jpeg.JPEGDecodeParam is internal proprietary API and may be removed in a future release
          [javac] com.sun.image.codec.jpeg.JPEGDecodeParam.COLOR_ID_YCbCrA) {
          [javac] ^
          [javac] /var/tmp/portage/dev-java/ICEpdf-4.1.1/work/ICEpdf-4.1.1-src/icepdf/core/src/org/icepdf/core/util/ImageCache.java:213: warning: com.sun.image.codec.jpeg.JPEGImageEncoder is internal proprietary API and may be removed in a future release
      ....
      1. build.log
        12 kB
        Mario Fetka
      2. build.log
        9 kB
        Mario Fetka

        Activity

        Hide
        Patrick Corless added a comment -

        Unfortunately I won't be able to fulfill this request for 4.2. The JPEGCodec library is heavily used by our Stream class for decoding images. When looking at the feasibility of this request I cam across the http://commons.apache.org/sanselan project which might be viable option for replacing the old JPEGCodec library.

        This is a big change to the core and will likely result in a numerous regressions which will take significant time to address. I would still like to do do this but unfortunately don't have the time allowance for this release.

        Show
        Patrick Corless added a comment - Unfortunately I won't be able to fulfill this request for 4.2. The JPEGCodec library is heavily used by our Stream class for decoding images. When looking at the feasibility of this request I cam across the http://commons.apache.org/sanselan project which might be viable option for replacing the old JPEGCodec library. This is a big change to the core and will likely result in a numerous regressions which will take significant time to address. I would still like to do do this but unfortunately don't have the time allowance for this release.
        Hide
        Mario Fetka added a comment -

        this is the buildlog with oracle-jdk-1.7

        the sun.* jpeg class has been removed

        Show
        Mario Fetka added a comment - this is the buildlog with oracle-jdk-1.7 the sun.* jpeg class has been removed
        Hide
        Patrick Corless added a comment -

        I put some notes in PDF-332 which explain how I got around this on the command line. The jpeg class is still there but isn't getting picked up on the classpath as it was in early JDK's.

        Show
        Patrick Corless added a comment - I put some notes in PDF-332 which explain how I got around this on the command line. The jpeg class is still there but isn't getting picked up on the classpath as it was in early JDK's.
        Hide
        Patrick Corless added a comment -

        This runtime issue is starting to show up more often in our support queue. Targeting 5.1 for further investigation as to our options for loading jpegs with out com.sun.image.

        Show
        Patrick Corless added a comment - This runtime issue is starting to show up more often in our support queue. Targeting 5.1 for further investigation as to our options for loading jpegs with out com.sun.image.
        Hide
        Patrick Corless added a comment -

        The use of com.sun.image.codec.jpeg.JPEGCodec has been removed from the main imaging code stream. It should no be possible to compile ICEpdf with icedtea and run ICEpdf with improved image rendering.

        Show
        Patrick Corless added a comment - The use of com.sun.image.codec.jpeg.JPEGCodec has been removed from the main imaging code stream. It should no be possible to compile ICEpdf with icedtea and run ICEpdf with improved image rendering.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: