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

Should the PDF-hul log offsets wherever possible #947

Open
carlwilson opened this issue Aug 22, 2024 · 4 comments
Open

Should the PDF-hul log offsets wherever possible #947

carlwilson opened this issue Aug 22, 2024 · 4 comments
Assignees
Labels
feature New functionality to be developed P2 Medium priority issues to be scheduled in a future release

Comments

@carlwilson
Copy link
Member

One is for @asciim0's input, and I'd also be curious to know @samalloing's thoughts. The PDF-hul could be more consistent regarding whether it logs an error's offset. There are multiple places where offsets could be added to give better information. The downside is that adding this info might lead to multiple error reporting in some circumstances. The extra information may prevent some errors from getting collated into a single instance, as they are no longer considered identical.

@carlwilson carlwilson added the feature New functionality to be developed label Aug 22, 2024
@carlwilson carlwilson self-assigned this Aug 22, 2024
@carlwilson carlwilson added the P2 Medium priority issues to be scheduled in a future release label Aug 22, 2024
@samalloing
Copy link
Collaborator

Hi @carlwilson
I think it is fine as the offset is in an attribute. But I think these two errors (PDF-HUL-1 and 149) can be changed in the future to describe the errors better. But that's another pull request ;-)

Sam

@asciim0
Copy link
Contributor

asciim0 commented Aug 22, 2024

hey @carlwilson ! as mentioned on our call today i think for PDF it would be much more helpful if jhove could return the obj id instead of (or in addition to) the offset.

@carlwilson
Copy link
Member Author

@asciim0 I assume you mean #647

@asciim0
Copy link
Contributor

asciim0 commented Aug 23, 2024

yes! apparently i've requested it all over the board throughout the years :-D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality to be developed P2 Medium priority issues to be scheduled in a future release
Projects
None yet
Development

No branches or pull requests

3 participants