> On 7 Dec 2016, at 14:48, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > Hi Felix, > > Looks good in general, but is > 74 return; > intended, or is that a test bug?
It is intended, as that is the code path that will be executed when the test invokes itself. -Chris. > best regards, > > -- daniel > > On 07/12/16 14:29, Felix Yang wrote: >> :-) >> >> -Felix >>> On 6 Dec 2016, at 9:28 AM, Felix Yang <felix.y...@oracle.com> wrote: >>> >>> Add core-libs. >>> >>> Thanks, >>> Felix >>> On 2016/12/5 22:14, Felix Yang wrote: >>>> Hi, >>>> >>>> updated webrev. May I have a reviewer to review this >>>> >>>> http://cr.openjdk.java.net/~xiaofeya/8043838/webrev.01/ >>>> >>>> -Felix >>>> On 2016/12/5 15:50, Felix Yang wrote: >>>>> >>>>> >>>>> On 2016/12/5 15:47, Langer, Christoph wrote: >>>>>> Hi Felix, >>>>>> >>>>>> looks ok to me. >>>>>> >>>>>> You probably should remove the reference to the old shell script in >>>>>> comment line 25, though: >>>>>> >>>>>> 25 * Test run from script, AcceptCauseFileDescriptorLeak.sh >>>>> Christoph, Good catch! >>>>> >>>>> Thanks, >>>>> Felix >>>>>> >>>>>> Best regards >>>>>> Christoph >>>>> >>>> >>> >> >