https://issues.apache.org/bugzilla/show_bug.cgi?id=50657

--- Comment #2 from Joshua Marquart <[email protected]> 2011-01-26 
11:55:23 EST ---
Created an attachment (id=26552)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=26552)
Replaces earlier attachment, use this instead.

I made a couple mistakes in my initial patch.

Differences in this patch are

ImageWriterParams 
1) now uses constant from TIFFImageDecoder for cm instead of arbitrary
hard-coded value 3.

2) replaced unused proposed rowsPerStrip value and new boolean
rowsPerStripOverridden with more appropriately named oneRowPerStrip boolean.

3) revised hasResolution method to check both X and Y, which are both set in
setResolution and their own individual methods.

4) added javadoc comments to many methods that were not commented; my bad.

ImageIOTIFFImageWriter
1) replaced call to isRowsPerStripOverridden with call to NOT isOneRowPerStrip

2) Consolidated identical boolean checks.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to