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

Misc waterfallplot and proofpdf fixes #638

Merged
merged 5 commits into from
Oct 5, 2018
Merged

Conversation

miguelsousa
Copy link
Member

No description provided.

The number in the page header referring to the total page count was not always correct.
Waterfalls whose text string was longer than 126 characters were displaying a repeated glyph after the 127th character.
Fixed by using 'latin-1' encoding instead of 'utf-8', when writing that section of the PDF.
Besides simplifying the opening commands, fixes a traceback that happened in py3 on Windows due to using 'os.getcwdu()'.
This info is not useful and creates problems when trying to diff results (because even if the element is skipped from the diff, the PDF's xref block will contain many changes)
python/afdko/fontpdf.py Show resolved Hide resolved
python/afdko/proofpdf.py Show resolved Hide resolved
Copy link
Contributor

@cjchapman cjchapman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of minor issues: please change tabs to spaces and remove unused imports (see comments above.) Thanks.

@miguelsousa
Copy link
Member Author

These files haven't been cleaned up. I'll do that separately, some other day.

@miguelsousa miguelsousa merged commit c5b7416 into develop Oct 5, 2018
@miguelsousa miguelsousa deleted the waterfallplot-fixes branch October 5, 2018 17:27
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

Successfully merging this pull request may close these issues.

2 participants