Bug 691718

Summary: minfeaturesize for 9.00
Product: Ghostscript Reporter: Marcos H. Woehrmann <marcos.woehrmann>
Component: GeneralAssignee: Robin Watts <robin.watts>
Status: NOTIFIED FIXED    
Severity: normal CC: robin.watts
Priority: P1    
Version: master   
Hardware: PC   
OS: All   
Customer: 531 Word Size: ---
Attachments: min_size.diff
min_size.zip

Description Marcos H. Woehrmann 2010-10-25 19:35:29 UTC
The customer requests a minfeaturesize feature for Ghostscript 9.00
Comment 1 Ray Johnston 2010-10-25 19:50:58 UTC
Created attachment 6830 [details]
min_size.diff

Attaching a diff of the (circa 9/2007) source files against the current HEAD.
Comment 2 Ray Johnston 2010-10-25 19:52:20 UTC
Created attachment 6831 [details]
min_size.zip

The .zip from which the modified source files were derived (in case they are
useful in determining more relevant diffs from the $Id: lines which document
the starting revision).
Comment 3 Robin Watts 2010-10-26 12:22:26 UTC
Rays cunning code hooks into gdev_fax_print_strip. Since revision 10316(ish) this hasn't been called from fax devices, hence it hasn't worked.

I'll look at moving the cunningness over to tiff_print_page.
Comment 4 Robin Watts 2010-10-26 12:23:12 UTC
> ...hasn't been called from fax devices...

tiff devices. It's tiff devices that haven't called it. Sorry.
Comment 5 Robin Watts 2010-10-26 18:14:01 UTC
Patch made to fit current code, and fitted to the TIFF devices as well as the fax ones. Committed as revision 11854.

This leaves the tiffscaled device as the only black and white TIFF device that does not support this. Leaving this bug open as a reminder to investigate that.
Comment 6 Ray Johnston 2010-10-26 19:43:25 UTC
Re: the comment "This leaves the tiffscaled device as the only ... device ..."

We _may_ want to also consider the tiffsep1 device, but since that device is
usually used for 'plate' applications at full resolution from PDF or PS,
it is less important.

In thinking back, I recall that this might be a useful parameter to be
buried in the 'get_bits' call and thus applicable to ALL 1-bit per component
devices such as 'bit' etc. Comments on this welcome.
Comment 7 Robin Watts 2010-10-26 23:32:50 UTC
I've got to introduce the MinFeatureSize parameter to vanilla tiff things tomorrow to make it work for TiffScaled. When I do so it should be trivial to make it work for the other tiff based devices (when in 1bpp mode), so I'll do that too.
Comment 8 Robin Watts 2010-10-27 19:51:32 UTC
Revision 11862 adds support for MinFeatureSize=2 to tiffscaled device.

MinFeatureSize and AdjustWidth now work for all black and white targetted tiff devices (but still not tiffsep1).