i need sponsor for this fix as well.
Vyom
On 3/3/2016 2:34 PM, Chris Hegarty wrote:
On 3 Mar 2016, at 05:33, Vyom Tewari <vyom.tew...@oracle.com> wrote:
incorporated the review comment, updated webrev in place.
http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html
<http://cr.openjdk.java.net/%7Enkumar/vyom/8150521/webrev0.1/index.html>
Looks fine.
-Chris.
Vyom
On 3/2/2016 1:45 PM, Alan Bateman wrote:
On 02/03/2016 07:44, vyom wrote:
Hi Chris,
Thanks for review please find the updated
webrev(http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html
<http://cr.openjdk.java.net/%7Enkumar/vyom/8150521/webrev0.1/index.html>).
This looks okay to me. Style wide, can you put a space in "if(" to keep it
consistent.
-Alan.
