| Summary: | regression causes correctly rendered PDF to print incorrectly with spirals instead of some spaces | ||
|---|---|---|---|
| Product: | poppler | Reporter: | wxl <wxl> |
| Component: | general | Assignee: | poppler-bugs <poppler-bugs> |
| Status: | RESOLVED MOVED | QA Contact: | |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | unspecified | ||
| Hardware: | x86 (IA32) | ||
| OS: | Linux (All) | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: |
pdf that prints with weird spirals instead of some spaces
ensure type 0 fonts contain all glyphs bug135281.pdf patch regresses in page 3 of it |
||
|
Description
wxl
2013-10-14 21:38:12 UTC
not sure why i couldn't edit the version, but it's poppler 0.20.5. the functional version was 0.18.4. Please list the poppler versions you use in the works and doesn't work machines. Comments crossed on the interwebs :D So the regression is caused by commit cd0764921064bfd455e9df52dc9bda6fbd2c2db2 Author: Carlos Garcia Campos <carlosgc@gnome.org> Date: Wed Aug 31 17:23:28 2011 +0200 xpdf303: Always define at least 256 glyphs for Type 0 fonts Now need to find out if we really need it for anything and needs some tweaking or if we can just revert it. Created attachment 90062 [details] [review] ensure type 0 fonts contain all glyphs This patch seems to fix the issue. I suggest regtesting it to ensure it does not introduce any regressions. (In reply to comment #5) > Created attachment 90062 [details] [review] [review] > ensure type 0 fonts contain all glyphs > > This patch seems to fix the issue. I suggest regtesting it to ensure it does > not introduce any regressions. 5993 tests passed (96.72%) 202 tests failed (3.26%) Most of the failures are in postscript and text backends, although I think the text differences are actually caused by the previous commit. Seems that it regresses on page3 of bug135281.pdf that i will attach Created attachment 90676 [details]
bug135281.pdf patch regresses in page 3 of it
Adrian did you have time to look at the regression i mention when applying your patch? I did look at it. I could not see any easy solution and have not had time to investigate it any further. -- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/poppler/poppler/issues/572. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.