About: Fix ghostpdf errors
"SaGS" <[email protected]> Sat, 19 Feb 2005 11:02:13 +0200
| Newsgroups | gmane.comp.printing.ghostscript.patches |
|---|---|
| Message-ID | <[email protected]> |
Hello,
Since you're at fixing the PPD, here are some more errors to fix
and improvements I suggest:
- The *PPD-Adobe line is duplicated, remove the 2nd occurence.
- The *PCFileName does not match the file name (wrong extension);
also fix the comment before it.
- For all *PageSize and *PageRegion, remove the *End; the latter
is to be used only when an <InvocationValue> or <QuotedValue>
spans across multiple lines.
- For <MediaOption>, it is preferable to use registered
<MediaOption>s from the Appendix B of the PPD 4.3 Ref Manual.
This means to replace 11x17->Tabloid, JISB0..JISB6->B0..B6,
C0..C6->EnvC0..EnvC6, HalfLetter->Statement,
FLSE/FLSA->FanFoldGermanLegal (see [*] below). There's
no need to change the translation strings ("B0/JIS B0").
(There's no registered keyword for 595x792pt, so PA4 is OK.)
- *PageRegion has an *OrderDependency of 30; shouldn't this be
20, as all other page device/ paper settings?
- *Default<MainKeyword>s expect a value of type <StringValue>,
that is without quotes. Currently some (but not all) of these
are specified with quotes (as if they were <InvocationValue>s
or <QuotedValue>s).
[*] I notice the sizes for FLSA ans FLSE are identical (612x936pt).
Is this what is intended? In the PPD Ref Manual I found
FanFoldGermanLegal 612x936pt, FanFoldUS 1071x792pt, and also
FanFoldGerman 612x864pt.
(This is NOT a code review, but an extract from some old notes of mine.)
Sincerily yours,
SaGS
----- Original Message -----
From: "Russell Lang" <[email protected]>
To: <[email protected]>
Sent: Friday, 18 February 2005 11:59
Subject: [gs-code-review] Fix ghostpdf errors
> ...