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|NEW                         |ASSIGNED




------- Additional Comments From [EMAIL PROTECTED]  2007-02-03 14:56 -------
Patch applied. Thanks, Dominic.
http://svn.apache.org/viewvc?view=rev&rev=503326

While reviewing I noticed one problem. PNGReader does not read the resolution
from the PNG file. So, if a PNG were used with a resolution other than 72dpi,
the intrinsic size of the image will be wrong when processed through the layout
engine. The problem is that PNG is a format consisting of chunks. The "pHYs"
chunk with the resolution info is optional any doesn't have to be near the
beginning of the file so it's not so easy to just extend PNGReader.

I'll have to think about how best to implement this, especially in the light of
the image library redesign where I'd like to avoid decoding the whole image just
to get the resolution. Therefore I'm keeping this issue open. One way or the
other, we need to fix this before the next release, because the bug fixes one
thing but essentially breaks another.

-- 
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