RFR (XS): 8151064: com/sun/jdi/RedefineAddPrivateMethod.sh fails intermittently
Daniel D. Daugherty
daniel.daugherty at oracle.com
Thu Mar 3 17:29:14 UTC 2016
> Webrev: http://cr.openjdk.java.net/~aeriksso/8151064/webrev/
test/com/sun/jdi/RedefineAddPrivateMethod.sh
L38: System.out.println("@1 breakpoint");
L39: System.out.println("@2 breakpoint");
With these tests, this is usually done like this:
System.out.println("stop here for breakpoint 1"); // @1 breakpoint
System.out.println("stop here for breakpoint 2"); // @2 breakpoint
I _think_ the scaffold stuff can handle the way you did
it on all platforms, but I'm not sure. Sorry, my memory
is a bit rusty on this stuff.
Thumbs up on the original version or you can change it to
the above. Your choice.
Dan
P.S.
It occurs to me that in the original code the main() method
is empty, i.e., just comment lines. I have a vague memory
about empty methods being treated differently in HotSpot,
but I don't remember the exact details...
On 3/3/16 10:05 AM, Andreas Eriksson wrote:
> Thanks Serguei.
> I'll go ahead and push this now, since I believe this change is small
> enough.
>
> - Andreas
>
> On 2016-03-03 17:54, serguei.spitsyn at oracle.com wrote:
>> Hi Andreas,
>>
>> Good++
>>
>> Thanks,
>> Serguei
>>
>>
>> On 3/3/16 06:35, Dmitry Samersoff wrote:
>>> Andreas,
>>>
>>> Looks good for me.
>>>
>>> -Dmitry
>>>
>>> On 2016-03-03 17:05, Andreas Eriksson wrote:
>>>> Hi,
>>>>
>>>> Can I please have a review of this fix for
>>>> 8151064: com/sun/jdi/RedefineAddPrivateMethod.sh fails intermittently
>>>> https://bugs.openjdk.java.net/browse/JDK-8151064
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~aeriksso/8151064/webrev/
>>>>
>>>> Still not sure why it only fails sometimes, but after this change the
>>>> test has not failed once after a couple of hours of testing. Before
>>>> the
>>>> change it would fail after ~5 minutes of running it in a loop.
>>>>
>>>> Thanks,
>>>> Andreas
>>>
>>
>
More information about the serviceability-dev
mailing list