Bug 691747 - part of fill missing
Summary: part of fill missing
Status: CONFIRMED
Alias: None
Product: GhostPCL
Classification: Unclassified
Component: PCL interpreter (show other bugs)
Version: unspecified
Hardware: PC All
: P4 normal
Assignee: Robin Watts
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-03 23:42 UTC by Henry Stiles
Modified: 2023-05-23 15:34 UTC (History)
3 users (show)

See Also:
Customer:
Word Size: ---

Attachments
Cut down version of test file (13.70 KB, application/octet-stream)
2011-01-05 22:44 UTC, Shailesh Mistry 		Details
PDF output using cu down file (9.76 KB, application/pdf)
2011-01-05 22:46 UTC, Shailesh Mistry 		Details
jpeg output using cut down file (9.58 KB, image/jpeg)
2011-01-05 22:46 UTC, Shailesh Mistry 		Details
Smaller version of the test file (141 bytes, application/octet-stream)
2011-01-11 20:25 UTC, Shailesh Mistry 		Details
Debug output of small test file using -ZFIi (1.45 KB, text/plain)
2011-01-11 20:37 UTC, Shailesh Mistry 		Details
change code to touched painting (2.12 KB, patch)
2011-03-02 23:17 UTC, Henry Stiles 		Details | Diff
filladjust counterexample (16.61 KB, application/octet-stream)
2011-03-17 15:43 UTC, Henry Stiles 		Details
HP output for R_PTCPM_FG1 (139.40 KB, application/pdf)
2011-03-17 16:00 UTC, Henry Stiles 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Henry Stiles 2010-11-03 23:42:08 UTC 
Black fills are missing in table heading and the upper right graphic.
Comment 2 Shailesh Mistry 2011-01-05 22:44:05 UTC 
Created attachment 7090 [details]
Cut down version of test file

Attached is a cut down version of the test file showing just the section that used to be in the top right fill.

The fill is correct with pdfwrite but fails with jpeg.

Also attached are the output files for these two devices using the cut down file.
Comment 3 Shailesh Mistry 2011-01-05 22:46:07 UTC 
Created attachment 7091 [details]
PDF output using cu down file
Comment 4 Shailesh Mistry 2011-01-05 22:46:46 UTC 
Created attachment 7092 [details]
jpeg output using cut down file
Comment 5 Shailesh Mistry 2011-01-11 20:25:45 UTC 
Created attachment 7111 [details]
Smaller version of the test file
Comment 6 Shailesh Mistry 2011-01-11 20:37:16 UTC 
Created attachment 7112 [details]
Debug output of small test file using -ZFIi

The attached output file was created by running 

pcl6 -ZFIi -dNOPAUSE -sDEVICE=jpeg -sOutputFile=test0.jpg tracfone2_simpz.pcl

The original test file has a fill made from repeated units drawing a very thin horizontal line, closing the path off and filling it in.

Given that the closed path is of zero height, the jpeg driver does not fill in the shape and hence the overall fill is missing.
Comment 7 Henry Stiles 2011-03-02 23:17:37 UTC 
Created attachment 7308 [details]
change code to touched painting

We don't expect this to render at low resolution given the coordinates.  At 600 dpi and the attached patch the file will work.  There is fairly good evidence that HP is using touched rendering and not center of pixel.  Last I checked though switching to touched filling raised several problems in the regression tests which would need to be investigated.  I am not sure when we'll get to study this more at least we have a workaround patch now.
Comment 8 Henry Stiles 2011-03-17 15:43:27 UTC 
Created attachment 7384 [details]
filladjust counterexample

This file prints incorrectly with a fill adjust of 0.5 (attached patch), although it is partially improved.
Comment 9 Henry Stiles 2011-03-17 16:00:36 UTC 
Created attachment 7385 [details]
HP output for R_PTCPM_FG1