+1 Hannes
> Am 22.09.2017 um 08:41 schrieb Sundararajan Athijegannathan > <sundararajan.athijegannat...@oracle.com>: > > +1 > > -Sundar > > On 22/09/17, 11:58 AM, Priya Lakshmi Muthuswamy wrote: >> Moved the changes to jdk10/master repo. >> >> webrev : http://cr.openjdk.java.net/~pmuthuswamy/8186180/webrev.02/ >> >> Thanks, >> Priya >> >> On 9/7/2017 2:07 PM, Sundararajan Athijegannathan wrote: >>> +1 >>> >>> On 07/09/17, 2:02 PM, Priya Lakshmi Muthuswamy wrote: >>>> Updated the test. >>>> >>>> Revised patch : http://cr.openjdk.java.net/~pmuthuswamy/8186180/webrev.01/ >>>> >>>> Thanks, >>>> Priya >>>> On 9/7/2017 1:49 PM, Sundararajan Athijegannathan wrote: >>>>> The formatting of test can be improved - whitespace around "{", new line >>>>> chars between lines etc. >>>>> >>>>> +1 otherwise. >>>>> >>>>> -Sundar >>>>> >>>>> On 07/09/17, 9:09 AM, Priya Lakshmi Muthuswamy wrote: >>>>>> Hi, >>>>>> >>>>>> Please review JDK-8186180 : Symbol.__proto__ results in error with >>>>>> format string placeholder >>>>>> >>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8186180 >>>>>> webrev: http://cr.openjdk.java.net/~pmuthuswamy/8186180/webrev.00/ >>>>>> >>>>>> Thanks, >>>>>> Priya >>>>>> >>>>>> >>>> >>
