Closed
Bug 301631
Opened 21 years ago
Closed 21 years ago
Invalid PostScript comment header generated
Categories
(Core :: Printing: Output, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: caillon, Assigned: caillon)
Details
Attachments
(1 file)
|
1.63 KB,
patch
|
kherron+mozilla
:
review+
shaver
:
superreview+
shaver
:
approval1.8b4+
|
Details | Diff | Splinter Review |
%%BoundingBox takes only integer arguments, per the PostScript DSC spec:
http://partners.adobe.com/public/developer/en/ps/5001.DSC_Spec.pdf
(see section 5.1, page 39)
There was a DSC header Comment Extension specification put in place
http://partners.adobe.com/public/developer/en/ps/5644.Comment_Ext.pdf
which allows a new comment header, %%HiResBoundingBox takes floating point
numbers, however it recommends using both %%HiResBoundingBox and %%BoundingBox
as older viewers may not support %%HiResBoundingBox
Yes, there are some viewers which cannot view the output postscript. See
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=163684
| Assignee | ||
Comment 1•21 years ago
|
||
Fixes this, and very minor cleanup.
Attachment #190056 -
Flags: superreview?(shaver)
Attachment #190056 -
Flags: review?(cbiesinger)
Updated•21 years ago
|
Attachment #190056 -
Flags: review?(cbiesinger) → review?(kherron+mozilla)
Comment 2•21 years ago
|
||
Comment on attachment 190056 [details] [diff] [review]
Patch
Because HiResBoundingBox is an extension, it would be nice to add a comment
referencing the technical note. Looks fine, otherwise.
Attachment #190056 -
Flags: review?(kherron+mozilla) → review+
Comment 3•21 years ago
|
||
Comment on attachment 190056 [details] [diff] [review]
Patch
sr+a=shaver
Attachment #190056 -
Flags: superreview?(shaver)
Attachment #190056 -
Flags: superreview+
Attachment #190056 -
Flags: approval1.8b4+
| Assignee | ||
Comment 4•21 years ago
|
||
Commited to cvs HEAD 2005-07-26 08:54 PDT.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•