-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Feature/ascii85 off by one #333
Conversation
…whitespace before and after it, not just after it.
It will be best if this changes was against this projects master instead of your master, because will be easier to be merged even if the changes from your other PR are not accepted. |
I think this is related to #312 |
Why that? |
Does anybody happen to have a PDF file with |
The current state is not compatible with main. Would you mind to fix the merge conflicts? |
I think the main point of this PR was already adressed elsewhere. |
ASCII 85 can represent 2^32-1. This error was causing validations to break.
Note that this merge requires #332 to be merged first to be a nice clean merge.