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

Reply via email to