[OpenJDK 2D-Dev] Review Request: JDK-6587251 Import declaration not used in sun.java2d.*

Prem Balakrishnan prem.balakrishnan at oracle.com
Tue May 17 10:34:35 UTC 2016


Hi Prasanta,

 

Thank you for the Review.

Updated patch as per review comments.

 

http://cr.openjdk.java.net/~pkbalakr/6587251/webrev.01/ 

 

Regards,
Prem

 

From: prasanta sadhukhan 
Sent: Tuesday, May 17, 2016 3:33 PM
To: Prem Balakrishnan; Philip Race; 2d-dev at openjdk.java.net
Subject: Re: Review Request: JDK-6587251 Import declaration not used in sun.java2d.*

 

Hi Prem,

Looks good to me.
Some files you have changed the copyright year, some files you have not like
sun/java2d/pipe/OutlineTextRenderer.java, sun/java2d/pipe/LoopPipe.java 
although I do not think it is necessary to change manually, but please be consistent.

Also, please put "noreg-cleanup" in the bug label.

Regards
Prasanta

On 5/17/2016 3:00 PM, Prem Balakrishnan wrote:

 

Hi,

Please review fix for JDK9,

Bug: https://bugs.openjdk.java.net/browse/JDK-6587251  

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Epkbalakr/6587251/webrev.00/"http://cr.openjdk.java.net/~pkbalakr/6587251/webrev.00/ 

 

Removed unused Import declaration in sun.java2d.*

 

Regards,
Prem

 

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160517/07b6dae3/attachment.html>


More information about the 2d-dev mailing list