looks good. -- Igor
> On Jan 22, 2019, at 4:30 PM, Mandy Chung <mandy.ch...@oracle.com> wrote: > > +1 > > Mandy > > On 1/22/19 4:26 PM, Mikael Vidstedt wrote: >> Please review this change which back ports the problem listing of >> LFMultiThreadCachingTest.java to jdk12 (my original intent was actually to >> push it to jdk12, but that plan got lost somewhere on the way). >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8217349 >> Webrev: >> http://cr.openjdk.java.net/~mikael/webrevs/8217349/jdk12/webrev.00/open/webrev/ >> >> The change is the same as the one in jdk/jdk, but the copyright year was >> already updated in jdk12, so there was a trivially resolved merge conflict. >> >> Cheers, >> Mikael >> >>> On Jan 17, 2019, at 3:00 PM, Mikael Vidstedt <mikael.vidst...@oracle.com> >>> wrote: >>> >>> >>> Thanks Igor! Pushed. >>> >>> Cheers, >>> Mikael >>> >>>> On Jan 17, 2019, at 2:07 PM, Igor Ignatyev <igor.ignat...@oracle.com> >>>> wrote: >>>> >>>> Mikael, >>>> >>>> looks good and trivial to me. >>>> >>>> -- Igor >>>> >>>>> On Jan 17, 2019, at 2:02 PM, Mikael Vidstedt <mikael.vidst...@oracle.com> >>>>> wrote: >>>>> >>>>> >>>>> Please review this change which adds >>>>> java/lang/invoke/LFCaching/LFMultiThreadCachingTest.java to the problem >>>>> list until the main bug has been addressed. >>>>> >>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8217349 >>>>> <https://bugs.openjdk.java.net/browse/JDK-8217349> >>>>> Webrev: >>>>> http://cr.openjdk.java.net/~mikael/webrevs/8217349/webrev.00/open/webrev/ >>>>> <http://cr.openjdk.java.net/~mikael/webrevs/8217349/webrev.00/open/webrev/> >>>>> Main bug: https://bugs.openjdk.java.net/browse/JDK-8151492 >>>>> <https://bugs.openjdk.java.net/browse/JDK-8151492> >>>>> >>>>> Cheers, >>>>> Mikael >>>>> >