Looks fine, thanks.

> 
> webrev updated: http://cr.openjdk.java.net/~avstepan/8154314/webrev.03 
> <http://cr.openjdk.java.net/%7Eavstepan/8154314/webrev.03>
> 
> Removed unused module declaration
> 
> 
> On 20.12.2016 16:13, Sergey Bylokhov wrote:
>>> webrev updated: http://cr.openjdk.java.net/~avstepan/8154314/webrev.02 
>>> <http://cr.openjdk.java.net/%7Eavstepan/8154314/webrev.02>
>>> 
>>> getScaleFactor() code is now directly used GraphicsEnviroinment code to get 
>>> scale factor.
>> Since the "import sun.java2d.SunGraphics2D» was removed then probably 
>> "@modules java.desktop/sun.java2d» can be removed as well?
>> 
>>> 
>>> Thanks,
>>> Maksim
>>> 
>>> 
>>> On 19.12.2016 18:47, Maksim Khramov wrote:
>>>> Hello, 
>>>> 
>>>> please review this request... 
>>>> 
>>>> Webrev: http://cr.openjdk.java.net/~avstepan/8154314/webrev.01/ 
>>>> <http://cr.openjdk.java.net/%7Eavstepan/8154314/webrev.01/> 
>>>> <http://cr.openjdk.java.net/%7Eavstepan/8154314/webrev.01/> 
>>>> <http://cr.openjdk.java.net/%7Eavstepan/8154314/webrev.01/> 
>>>> Issue: https://bugs.openjdk.java.net/browse/JDK-8154314 
>>>> <https://bugs.openjdk.java.net/browse/JDK-8154314> 
>>>> 
>>>> Test bug. Compilation of 
>>>> java/awt/datatransfer/DragImage/MultiResolutionDragImageTest.java still 
>>>> fails due removed method 
>>>> 
>>>> Thanks, 
>>>> Maksim. 
>>> 
>> 
> 

Reply via email to