DO NOT REPLY [Bug 47151] New: ImageManager.getImageInfo returns incorrect image size due multiple unit conversions in ImageIOUtil.extractResolution

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

           Summary: ImageManager.getImageInfo returns incorrect image size
                    due multiple unit conversions in
                    ImageIOUtil.extractResolution
           Product: XMLGraphicsCommons
           Version: 1.3.1
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Utilities
        AssignedTo: general@...
        ReportedBy: dgreen99@...


PreloaderImageIO is used by ImageManager to get image information, including
size (height and width) and DPI. Unfortunately ImageIOUtil.extractResolution()
causes image sizes to be incorrect in some cases due to multiple floating point
operations.  This results in FOP rendering blurry images when creating PDFs.

--
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: general-unsubscribe@...
For additional commands, e-mail: general-help@...

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





--- Comment #1 from David Green <dgreen99@...>  2009-05-04 15:15:13 PST ---
Created an attachment (id=23609)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23609)
candidate fix

attached a candidate fix that eliminates a computation that causes loss of
precision

--
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: general-unsubscribe@...
For additional commands, e-mail: general-help@...

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


David Green <dgreen99@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |47157




--
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: general-unsubscribe@...
For additional commands, e-mail: general-help@...

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


Chris Bowditch <bowditch_chris@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|ImageManager.getImageInfo   |[PATCH]
                   |returns incorrect image     |ImageManager.getImageInfo
                   |size due multiple unit      |returns incorrect image
                   |conversions in              |size due multiple unit
                   |ImageIOUtil.extractResoluti |conversions in
                   |on                          |ImageIOUtil.extractResoluti
                   |                            |on




--- Comment #3 from Chris Bowditch <bowditch_chris@...>  2009-05-06 00:49:20 PST ---
Thanks for the patch. Would you mind also attaching a sample Image that
demonstrates the issue you reported. Thanks!

--
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: general-unsubscribe@...
For additional commands, e-mail: general-help@...

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





--- Comment #4 from David Green <dgreen99@...>  2009-05-06 09:00:26 PST ---
Created an attachment (id=23618)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23618)
updated patch with JUnit test

--
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: general-unsubscribe@...
For additional commands, e-mail: general-help@...

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





--- Comment #5 from David Green <dgreen99@...>  2009-05-06 09:00:31 PST ---
Created an attachment (id=23619)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23619)
mylyn/context/zip

--
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: general-unsubscribe@...
For additional commands, e-mail: general-help@...

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





--- Comment #6 from David Green <dgreen99@...>  2009-05-06 09:01:06 PST ---
Created an attachment (id=23620)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23620)
image used in JUnit

--
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: general-unsubscribe@...
For additional commands, e-mail: general-help@...