Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/8040279/webrev.01/
The createBufferedImage method is renamed to createImage in the CImage
class.
Thanks,
Alexandr.
On 4/21/2014 2:36 PM, Petr Pchelko wrote:
Hello, Alexander.
Just one minor comment: In CImage createBufferedImage method does not create a
BufferredImage any more, so it worths renaming.
With best regards. Petr.
On 15.04.2014, at 18:36, Alexander Scherbatiy <alexandr.scherba...@oracle.com>
wrote:
Hello,
Could you review the fix:
bug: https://bugs.openjdk.java.net/browse/JDK-8040279
webrev: http://cr.openjdk.java.net/~alexsch/8040279/webrev.00
MultiResolutionBufferedImage extends BufferedImage. In this case it needs to
use the base image to write it to the BufferedImage.
The fix renames MultiResolutionBufferedImage to MultiResolutionCahcedImage
and does not use the BufferedImage as the super class.
The base image is created and cached by demand.
CImage now always returns MultiResolutionCahcedImage. All requested
resolution variants are scaled by NSImage.
Thanks,
Alexandr.
