DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=41488>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=41488


[EMAIL PROTECTED] changed:

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




------- Additional Comments From [EMAIL PROTECTED]  2007-02-07 04:55 -------
As I suspected, the resolution handling got broken that way because PNGReader
can't parse the resolution from the PNG. Adding that is not trivial since it's
not clear how many bytes have to be read until the pHYs chunk is found. If it's
more than 2048 bytes into the file (generally possible due to metadata and
because the order of the chunks is not fixed) the code will break because
BufferedInputStream.reset() will cause an exception.

After trying three different directions, I decided to revert the patch and
instead change RTFHandler so it consumes the decoded PNG image and reencodes it
as a new PNG for RTF output. The nice side-effect is that supporting GIFs is
then very easy. This is obviously inefficient and only a temporary measure until
the image package is redesigned and those problems will be solved.

http://svn.apache.org/viewvc?view=rev&rev=504511

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

Reply via email to