Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct default value of IV entry in Table 322 from false to true. #50

Closed
crlf0710 opened this issue Feb 23, 2021 · 3 comments
Closed
Labels
ISO approved Resolved issue approved by ISO

Comments

@crlf0710
Copy link

In table 322 the default value of IV entry is false. However in existing commercial products like acrobat, the value of this entry has always been interpreted as true if missing. (And it is indeed that true as default value is more natural)

I'm not sure whether it's better to just change the specification in this case, or just claim that every existing implementation has been wrong. Personally i feel the former approach is more practical and helpful.

@crlf0710 crlf0710 added the bug Something isn't working label Feb 23, 2021
@lrosenthol
Copy link
Collaborator

I can confirm that the Adobe implementation sets the default value to true prior to reading this dictionary, thus leading to that becoming the "default".

@petervwyatt
Copy link
Member

Proposed solution as above

@petervwyatt petervwyatt added the proposed solution Proposed solution is ready for review label Feb 23, 2021
@petervwyatt
Copy link
Member

PDF TWG agreed

@petervwyatt petervwyatt added ISO submitted Resolved issue submitted to ISO for approval and removed bug Something isn't working proposed solution Proposed solution is ready for review labels Dec 6, 2021
@petervwyatt petervwyatt added this to the ISO Submission Feb 2022 milestone Dec 6, 2021
@petervwyatt petervwyatt added ISO approved Resolved issue approved by ISO and removed ISO submitted Resolved issue submitted to ISO for approval labels Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ISO approved Resolved issue approved by ISO
Projects
None yet
Development

No branches or pull requests

3 participants