[OpenJDK 2D-Dev] Review Request: JDK-6587251 Import declaration not used in sun.java2d.*
prasanta sadhukhan
prasanta.sadhukhan at oracle.com
Tue May 17 10:38:12 UTC 2016
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/
> <http://cr.openjdk.java.net/%7Epkbalakr/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:*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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160517/6be6eff2/attachment.html>
More information about the 2d-dev
mailing list