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

ispe required for every image item #74

Closed
dukesook opened this issue Nov 8, 2023 · 3 comments
Closed

ispe required for every image item #74

dukesook opened this issue Nov 8, 2023 · 3 comments

Comments

@dukesook
Copy link
Contributor

dukesook commented Nov 8, 2023

The sample image here fails the ComplianceWarden because items 2 & 3 don't have an ispe associated with them. However, these items are of type 'mime' and 'uri '. They're not images so they don't have a width or height.

This usually isn't an issue because most mime items don't have properties associated with them. However, the mime & uri items in the image above do in fact have at least 1 property associated with them. Looking in heif.cpp, the rule is only checked for items that have a property associated with them.

Is the file actually compliant? Are mime items considered to be "image items"?

@rbouqueau
Copy link
Member

The link seems dead. Can you check it?

@dukesook
Copy link
Contributor Author

dukesook commented Nov 9, 2023

Sorry about that, I updated the link to the correct url.

@rbouqueau
Copy link
Member

Thanks for reporting. Should be fixed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants