| Summary: | Add -subst option to pdffonts to show the substituted fonts | ||
|---|---|---|---|
| Product: | poppler | Reporter: | Adrian Johnson <ajohnson> |
| Component: | utils | Assignee: | poppler-bugs <poppler-bugs> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | medium | ||
| Version: | unspecified | ||
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | add -subst option to pdffonts | ||
|
Description
Adrian Johnson
2012-01-03 03:27:17 UTC
The spacing in the GlobalParams::getDisplayFont looks a bit broken,
+ if (res == FcResultMatch && s2) {
+ substName = new GooString((char*)s2);
+ } else {
the if has spaces but the else has tabs
Also i'd just add the subst info all the time, why a switch?
(In reply to comment #1) > Also i'd just add the subst info all the time, why a switch? I thought it was better not to clutter the default output with too much information. There is also the question of how to format the output. Do you want to add the extra info (substitute fontname and filename) to the end of each line which is going to make the each line very long. Or can the info can be put on the next line or listed separately at the end of the current output? Yeah, length is a concern... ok, let's do it your way then, update the manpage and commit it to master. I've fixed the spacing, updated the man page and pushed to master. |
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.