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

fix: Fix opening a pdf file when the root node of the page tree is a simplified page dictionary #149

Closed
wants to merge 2 commits into from

Conversation

juniorschroder
Copy link
Contributor

There are PDF files whose page root structure contains only the /Kids and /Count entries, this means that the root node of the page tree is a simplified page dictionary. In this context, the /Kids entry contains a list of references to other nodes or pages and the /Count entry indicates the total number of pages in the document.

Copy link

what-the-diff bot commented Jul 31, 2024

PR Summary

  • Enhanced Method for Verifying Root Page in PDFs
    The PdfReader script now has an improved method to ascertain if a page is the root page in a PDF document. The HasRootPage method now includes an additional verification step IsSimplifiedPageDictionaryRootPage, to make these checks more reliable. This change will increase the accuracy of the software's root page identification functionality.

@VahidN
Copy link
Owner

VahidN commented Jul 31, 2024

Unfortunately I can't disable the pull section @github and I'm facing ppl who are not willing to add tests and they think I have to accept everything they post here. From now on, I will ban all of you. Yeah, just like that. DO NOT POST PR's at all.

@VahidN VahidN closed this Jul 31, 2024
@VahidN
Copy link
Owner

VahidN commented Jul 31, 2024

This user has been blocked from my repertoires. DO NOT POST PR's here.

Repository owner locked and limited conversation to collaborators Jul 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants