+1
-phil.
On 05/17/2016 03:38 AM, prasanta sadhukhan wrote:
Looks good.
Regards
Prasanta
On 5/17/2016 4:04 PM, Prem Balakrishnan wrote:
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; [email protected]
*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:*http://cr.openjdk.java.net/~pkbalakr/6587251/webrev.00/
<http://cr.openjdk.java.net/%7Epkbalakr/6587251/webrev.00/>
Removed unused Import declaration in sun.java2d.*
Regards,
Prem
