[gs-bugs] [Bug 693399] New: PCL6 file that prints successfully gives error on Ghostscript

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

Summary: PCL6 file that prints successfully gives error on
Ghostscript
Product: GhostPCL
Version: 9.06
Platform: PC
OS/Version: Windows 7
Status: NEW
Severity: normal
Priority: P4
Component: PCL interpreter
AssignedTo: ***@artifex.com
ReportedBy: ***@gmail.com
QAContact: gs-***@ghostscript.com
Word Size: ---


Created an attachment (id=9013)
--> (http://bugs.ghostscript.com/attachment.cgi?id=9013)
spool file for ricoh mp c2050

The attached spool file prints without error on the Ricoh printer it was
created for and is rendered without error on the PCLWorks interpreter but gives
an error when processed with Ghostscript.

Are you able to loosen your error checking to mimic the printer?

Version: 9.06
Build date: Wed Aug 08 12:56:22 2012

pcl6-9.06-win32.exe" -dNOPAUSE -dBATCH
"C:\big.dev\print_testing\output\PCL6\RICOH Aficio MP C2050
PCL6\paper_size_name=12x18.prn"

Warning interpreter exited with error code -992
Flushing to end of job
PCL XL error
Subsystem: KERNEL
Error: IllegalAttributeValue
Operator: SetColorSpace
Position: 24
file position of error = 308779

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

Henry Stiles <***@artifex.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Keywords| |bountiable
CC| |***@ghostscript.com

--- Comment #1 from Henry Stiles <***@artifex.com> 2012-10-22 16:12:18 UTC ---
Indeed the color space error is not reported on the HP either. I'm making this
bountiable for Hin-Tak if he wants to work on it, if not someone else will get
to it.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

Shailesh Mistry <***@hotmail.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@hotmail.co.
| |uk

--- Comment #2 from Shailesh Mistry <***@hotmail.co.uk> 2012-10-23 19:32:03 UTC ---
Hi Hin-Tak, Are you looking into this? I have seen this before and can resolve
this if needed.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

Peter Williams <***@gmail.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@gmail.com

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

--- Comment #3 from Hin-Tak Leung <***@ghostscript.com> 2012-10-30 03:30:40 UTC ---
Created an attachment (id=9041)
--> (http://bugs.ghostscript.com/attachment.cgi?id=9041)
truncate too-large palette instead of err out

truncate too-large palette instead of err out.

The SetColorSpace operator was supplied with a palatte which is too large (768
instead of 256). Since some printer accepts it, I guess we'll just throw away
the extra instead of error'ing out.

With this patch processing of the input file continues to the 3rd page,

PCL XL error
Warning: IllegalMediaSize
Warning: IllegalMediaSize
file position of error = 308779

Will supply another patch to address that.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

--- Comment #4 from Hin-Tak Leung <***@ghostscript.com> 2012-10-30 04:36:47 UTC ---
Created an attachment (id=9042)
--> (http://bugs.ghostscript.com/attachment.cgi?id=9042)
add non-standard ricoh-only MediaSize enum

add non-standard ricoh-only MediaSize enum, 205 = paper size 12x18.

Adding mediasize enum 205 (12x18). Apparently some Ricoh drivers use 201-205
and 212-224 at least to means many paper sizes which aren't in published HP PXL
specs. I am not sure it is a good idea to add them all (since HP owns the spec
and interpretation/addition to those enum's), so this is just to add 205 which
is used here.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

--- Comment #5 from Hin-Tak Leung <***@ghostscript.com> 2012-10-30 04:42:08 UTC ---
With both patches, the file finishes processing to completion. I am somewhat
ambivalent about the 2nd patch - it is a Ricoh-only "enhancement" (or
off-spec-ness). OTOH, HP has not update the spec for a while and probably not
likely to.

There is nothing more to do about this bug.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

--- Comment #6 from Hin-Tak Leung <***@ghostscript.com> 2012-10-30 04:50:09 UTC ---
The C2050 also uses these:

201 = B6 (JIS)
202 = C5 Envelope
203 = 8Kai
204 = 16Kai

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

--- Comment #7 from Hin-Tak Leung <***@ghostscript.com> 2012-10-30 05:17:44 UTC ---
(In reply to comment #6)

Post by b***@ghostscript.com
201 = B6 (JIS)
202 = C6 Envelope
203 = 8Kai
204 = 16Kai

The dimensions are (in points):

B6 (JIS) = 363 x 516
C6 Env = 323 x 459
8Kai = 757 x 1106
16Kai = 553 x 757
12x18 = 864 x 1296

If it is needed, just divide by 72 and multiply by 300 as in the 2nd patch.

However, I'd say only add them if you need them. These seems to be Ricoh-only
"enhancements" so ghostpcl is probably correct to emit a warning about
IllegalMediaSize. I'd be interested to hear if Henry saw the IllegalMediaSize
on HP printers.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

--- Comment #8 from Henry Stiles <***@artifex.com> 2012-11-11 16:40:10 UTC ---
Yes, the original PXL file produces an error (Illegal Media Size) on the HP
Color Laserjet 4600. I'm going to add the new sizes anyway, thanks.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

Henry Stiles <***@artifex.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED

--- Comment #9 from Henry Stiles <***@artifex.com> 2012-11-11 16:51:08 UTC ---
Thanks Hin-Tak, can you make a complete patch with all the paper size changes
(comment #7)? Then we'll close this one down and clear the bounty payment.
Thanks.

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

Hin-Tak Leung <***@ghostscript.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #9042|0 |1
is obsolete| |

--- Comment #10 from Hin-Tak Leung <***@ghostscript.com> 2012-11-11 23:03:38 UTC ---
Created an attachment (id=9075)
--> (http://bugs.ghostscript.com/attachment.cgi?id=9075)
Ricoh specific paper-size enums

Adding Ricoh-specific paper-size enums, as well as modifying the pxl driver to
not output non-HP enums.

201-205 and 212-224 are found to be used by the C2050 and some Ricoh
multi-function printer/copiers. The paper size table is shared with the pxl
driver. Recognizing non-HP size code in the pxl interpreter is okay but the pxl
driver should output customsize with explicit dimension for non-HP sizes.

(Note also that Ricoh's 201 is the same size as HP's 18, and Ricoh's 203/204 is
the same as HP's 19/20, and some of Ricoh's other usage of 0-21
also seems to be different from HP's, e.g. in particular A5 and A6).

[b***@ghostscript.com]

http://bugs.ghostscript.com/show_bug.cgi?id=693399

Henry Stiles <***@artifex.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED

--- Comment #11 from Henry Stiles <***@artifex.com> 2012-11-13 03:11:28 UTC ---
Fixed in

commit 39e56129466aa8d0b84c8c1267dbaef9dcefd2bd
Author: Henry Stiles <***@artifex.com>
Date: Sun Nov 11 09:33:12 2012 -0700

Fixes 693399, PXL file errors out when color palette is too large.

The color palette should be truncated upon overflow, previously we
produced an error. Thanks to Hin-Tak Leung for the analysis and fix.


and

commit a971121e9e50c672908f1b3185f42d497adf0b02
Author: Henry Stiles <***@artifex.com>
Date: Mon Nov 12 20:05:31 2012 -0700

New PCL-XL paper sizes fixing part of bug 693399.

Thanks to Hin-Tak Leung for providing these new non standard paper
sizes to support Ricoh specific PCL-XL.