ICEpdf
  1. ICEpdf
  2. PDF-762

DeviceN colorspace ordering issue.

          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 PDF contains a deviceN colour space,

            [/DeviceN [/Cyan /Magenta /Yellow /Black] /DeviceCMYK 25 0 R]

            And the black is specified with /CS0 cs 0 0 0 1 scn. Where 0,0,0,1 represents the CMYK values and more importantly black or K.

            I'm pretty sure the we are reversing the colour information which is why the cyan color is shown instead of black.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Fix Version/s 5.0.7 [ 11470 ]
              Patrick Corless made changes -
              Fix Version/s 5.1 [ 10675 ]
              Fix Version/s 5.0.7 [ 11470 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42060 Fri Aug 01 09:15:21 MDT 2014 patrick.corless PDF-762 removed CMYK reveral to preserve black ordering.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceN.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42062 Fri Aug 01 12:24:01 MDT 2014 patrick.corless PDF-762 reverted to previous version, adding back in reverse code.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceN.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Spent quite a bit more time on this one trying to make sense of some regressions that came about after the change. There seems to be an issue with colour component ordering and the number of names that define the colour. Here is a list of the conflicing files.

              D:/pdf-qa/top 1000/support_3768.pdf
              D:/pdf-qa/top 1000/PG_2011_AnnualReport.pdf
              D:/pdf-qa/top 1000/pdf-portfolio.pdf
              D:/pdf-qa/top 1000/PDF-615-sample.pdf
              D:/pdf-qa/top 1000/m1-113_m1-213_hub.pdf
              D:/pdf-qa/top 1000/ficha-3-para+impresion.pdf
              D:/pdf-qa/top 1000/complete_pdf_fall_win_09_10_en.pdf
              D:/pdf-qa/top 1000/BPVC_IIAVI_EDTN_2010.pdf
              D:/pdf-qa/top 1000/aaaTest.pdf

              Show
              Patrick Corless added a comment - Spent quite a bit more time on this one trying to make sense of some regressions that came about after the change. There seems to be an issue with colour component ordering and the number of names that define the colour. Here is a list of the conflicing files. D:/pdf-qa/top 1000/support_3768.pdf D:/pdf-qa/top 1000/PG_2011_AnnualReport.pdf D:/pdf-qa/top 1000/pdf-portfolio.pdf D:/pdf-qa/top 1000/ PDF-615 -sample.pdf D:/pdf-qa/top 1000/m1-113_m1-213_hub.pdf D:/pdf-qa/top 1000/ficha- 3 -para+impresion.pdf D:/pdf-qa/top 1000/complete_pdf_fall_win_09_10_en.pdf D:/pdf-qa/top 1000/BPVC_IIAVI_EDTN_2010.pdf D:/pdf-qa/top 1000/aaaTest.pdf
              Arran Mccullough made changes -
              Support Case References Support Case #13037 - https://icesoft.my.salesforce.com/5007000000nOvNF
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42584 Thu Sep 18 12:01:19 MDT 2014 patrick.corless PDF-762 additional DeviceN colour interpolation, still needs work.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceN.java
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/ShadingType2Pattern.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42587 Thu Sep 18 16:56:57 MDT 2014 patrick.corless PDF-762 additional DeviceN colour interpolation, still needs work.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/ShadingType2Pattern.java
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/Indexed.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42589 Fri Sep 19 08:40:46 MDT 2014 patrick.corless PDF-762 adjustment for fillAndStroke default colour modes.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Hide
              Permalink
              Patrick Corless added a comment -

              We've done quite a bit of work to reorganize this but there is still some holes in the logic we need to apply. The PDF renders fine as well as our other test files.

              Show
              Patrick Corless added a comment - We've done quite a bit of work to reorganize this but there is still some holes in the logic we need to apply. The PDF renders fine as well as our other test files.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42591 Fri Sep 19 10:07:32 MDT 2014 patrick.corless PDF-762 adjustment for fillAndStroke default colour modes.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.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: