122494 - Xprint has problems parsing paper size enumaerations with tray names

Status: RESOLVED FIXED

(Core Graveyard :: Printing: Xprint, defect)

Description

[Roland Mainz]

Xprint has problems parsing paper size enumaerations with tray names.
Example (model-config):
-- snip --
*content-orientations-supported: portrait landscape reverse-portrait
reverse-landscape
*descriptor: Hewlett-Packard LaserJet 5N/5MN (PCL)
*document-formats-supported: {PCL 5}
*input-trays-supported: hp-tray-1 hp-tray-2 hp-tray-3 hp-envelope-feeder
*medium-source-sizes-supported: \
{ '' \
	{na-letter		FALSE	{6.35	209.55	6.35	273.05}} \
	{executive		FALSE	{6.35	177.8	6.35	260.35}} \
	{iso-a4			FALSE	{6.35	203.65	6.35	290.65}} \
	{na-legal		FALSE	{6.35	209.55	6.35	349.25}} \
} \
{ hp-tray-1 \
	{na-letter		FALSE	{6.35	209.55	6.35	273.05}} \
	{executive		FALSE	{6.35	177.8	6.35	260.35}} \
	{iso-a4			FALSE	{6.35	203.65	6.35	290.65}} \
	{iso-a5			FALSE	{6.35	141.65	6.35	203.65}} \
	{iso-c5			FALSE	{6.35	155.65	6.35	222.65}} \
	{iso-designated-long	FALSE	{6.35	103.65	6.35	213.65}} \
	{jis-b5			FALSE	{6.35	175.65	6.35	250.65}} \
	{monarch-envelope	FALSE	{6.35	91.948	6.35	184.15}} \
	{na-legal		FALSE	{6.35	209.55	6.35	349.25}} \
	{na-number-10-envelope	FALSE	{6.35	98.425	6.35	234.95}} \
} \
{ hp-envelope-feeder \
	{iso-c5			FALSE	{6.35	155.65	6.35	222.65}} \
	{iso-designated-long	FALSE	{6.35	103.65	6.35	213.65}} \
	{jis-b5			FALSE	{6.35	175.65	6.35	250.65}} \
	{monarch-envelope	FALSE	{6.35	91.948	6.35	184.15}} \
	{na-number-10-envelope	FALSE	{6.35	98.425	6.35	234.95}} \
}
*plexes-supported: simplex duplex tumble
*printer-model: Hewlett-Packard LaserJet 5N/5MN (PCL)
*printer-resolutions-supported: 600 300 200 150 100 75
*xp-ddx-identifier: XP-PCL-MONO
*xp-listfonts-modes-supported: xp-list-internal-printer-fonts
xp-list-glyph-fonts
*xp-embedded-formats-supported: {HPGL 2} {PCL 5}
*xp-raw-formats-supported: {HPGL 2} {PCL 5}
*xp-setup-proviso: xp-setup-optional
-- snip --

XpuEnumerateXpAttributeValue() fails to enumerate the paper sizes in the
"hp-tray-1" tray (fortunately the XpuParseMediumSourceSize() is very strict and
just skips the incoming rubbish...)...

Comment 1

[Roland Mainz]

A lex/yacc parser for the Xp attribute stuff would be nice - but that would make
the current xprintutil code non-threadsafe due the use of global vars. Fun...
;-(

Comment 2

[Roland Mainz]

Switching QA<-->Owner

Comment 3

[Roland Mainz]

Added fix to patch in bug 120916 ("PostScript/Xprint module revamp") ...

Comment 4

[Roland Mainz]

Fixed by patch from bug 120916, marking bug as FIXED.