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

Join Soft-mask image dictionary key limitations between Table 143 and Table 87 #13

Closed
petervwyatt opened this issue Jan 2, 2021 · 2 comments
Labels
ISO approved Resolved issue approved by ISO

Comments

@petervwyatt
Copy link
Member

Clause 11.6.5.2, "Table 143 — Restrictions on the entries in a soft-mask image dictionary" defines a number of image XObjects keys as being "Ignored" yet this information to ignore for soft-masks is not mentioned or cross-referenced from "Table 87 — Additional entries specific to an image dictionary".

Specifically for Table 87 keys: Intent, Alternates, Name, ID, StructParent.

Table 143 also says SMask key shall be absent, but Table 87 doesn't mention this.

Various simple, not overly wordy solutions and that don't duplicate technical requirements include:

  • add a reference to 11.6.5.2 Soft-mask images to the NOTE above Table 87

  • adding "Additional limitations also apply to this key when used in soft-mask image dictionaries - see clause 11.6.5.2 Soft-mask images." to each of the above keys in Table 87.

@petervwyatt petervwyatt added the bug Something isn't working label Jan 2, 2021
@lrosenthol
Copy link
Collaborator

I'd tend to favor the second suggestion since it's normative and makes it more clear which keys are impacted.

@petervwyatt petervwyatt added the proposed solution Proposed solution is ready for review label Jan 4, 2021
@petervwyatt
Copy link
Member Author

Proposed solution: add the sentence "Additional limitations also apply to this key when used in soft-mask image dictionaries - see clause 11.6.5.2 Soft-mask images." to the description for each of the Table 87 keys: Intent, Alternates, Name, ID, StructParent, and SMask.

@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 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

2 participants