On 3 Mar 2016, at 09:40, Vyom Tewari <vyom.tew...@oracle.com> wrote:
> i need sponsor for this fix as well. Pushed: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/eb5798a009cc -Chris. > 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. >
