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

        Mario Fetka created issue -
        Hide
        Patrick Corless added a comment -

        On the the Sun JDK this is normally just a warning messages. Are you getting an error when compiling with the Open JDK?

        Show
        Patrick Corless added a comment - On the the Sun JDK this is normally just a warning messages. Are you getting an error when compiling with the Open JDK?
        Hide
        Mario Fetka added a comment -

        Yes i am getting a compile error
        will append the build log

        Show
        Mario Fetka added a comment - Yes i am getting a compile error will append the build log
        Hide
        Mario Fetka added a comment -

        the build log

        if you want to have other ant task aktivated just let me know

        Show
        Mario Fetka added a comment - the build log if you want to have other ant task aktivated just let me know
        Mario Fetka made changes -
        Field Original Value New Value
        Attachment build.log [ 12664 ]
        Hide
        Patrick Corless added a comment -

        Scheduling for 4.2. We don't officially support Open JDK but given recent happening in the Java community is probably time to add support.

        If Open JDK doesn't have the equivalent API support we can always try loading the class reflectively.

        Show
        Patrick Corless added a comment - Scheduling for 4.2. We don't officially support Open JDK but given recent happening in the Java community is probably time to add support. If Open JDK doesn't have the equivalent API support we can always try loading the class reflectively.
        Patrick Corless made changes -
        Salesforce Case []
        Fix Version/s 4.2 [ 10243 ]
        Hide
        Mario Fetka added a comment -

        Great news
        thx
        Mario

        Show
        Mario Fetka added a comment - Great news thx Mario
        Patrick Corless made changes -
        Salesforce Case []
        Assignee Priority P1
        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.
        Patrick Corless made changes -
        Salesforce Case []
        Fix Version/s 4.2 [ 10243 ]
        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
        Mario Fetka made changes -
        Attachment build.log [ 13802 ]
        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.
        Patrick Corless made changes -
        Fix Version/s 5.1 [ 10675 ]
        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.
        Patrick Corless made changes -
        Fix Version/s 5.0.1 [ 10872 ]
        Fix Version/s 5.1 [ 10675 ]
        Patrick Corless made changes -
        Fix Version/s 5.1 [ 10675 ]
        Fix Version/s 5.0.1 [ 10872 ]
        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.
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Patrick Corless made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: