RFR 8203033: [Testbug] vmTestbase/nsk/jdi/TypeComponent/isSynthetic/issynthetic002/TestDescription.java fails with nestmates

[Daniil Titov]

Hi Serguei,

 

I updated the webrev to correct typo for debuggee instances as well.  However, I didn’t rename the shared class nsk.share.jdi.Debugee and nsk.share.jdi.Binder.bindToDebugee method  since I am not sure it should be done as a part of this fix. I would suggest filing a separate issue for that as that changes will affect multiple tests. 

 

Webrev: http://cr.openjdk.java.net/~dtitov/8203033/webrev.04/

Issue: https://bugs.openjdk.java.net/browse/JDK-8203033 

 

Thanks,

Daniil

 

From: "serguei.spitsyn at oracle.com" <serguei.spitsyn at oracle.com>
Date: Wednesday, June 6, 2018 at 12:18 PM
To: Daniil Titov <daniil.x.titov at oracle.com>, <serviceability-dev at openjdk.java.net>
Subject: Re: RFR 8203033: [Testbug] vmTestbase/nsk/jdi/TypeComponent/isSynthetic/issynthetic002/TestDescription.java fails with nestmates

 

It looks good.
There are also instances of "debugee" but they are
because of the supporting class named "Debugee".

Thanks,
Serguei


On 6/6/18 12:09, Daniil Titov wrote:

Thank you, Serguei and David, for reviewing this change.

 

Please review a new version  of the change that corrects the diagnostic message as Serguei suggested  and fixes typos.

 

Webrev: http://cr.openjdk.java.net/~dtitov/8203033/webrev.03

Issue: https://bugs.openjdk.java.net/browse/JDK-8203033 

 

Best regards,

Daniil

 

 

From: "serguei.spitsyn at oracle.com" <serguei.spitsyn at oracle.com>
Date: Wednesday, June 6, 2018 at 11:23 AM
To: Daniil Titov <daniil.x.titov at oracle.com>, <serviceability-dev at openjdk.java.net>
Subject: Re: RFR 8203033: [Testbug] vmTestbase/nsk/jdi/TypeComponent/isSynthetic/issynthetic002/TestDescription.java fails with nestmates

 

Hi Daniil,

It looks good.
One minor comment.

http://cr.openjdk.java.net/~dtitov/8203033/webrev.02/test/hotspot/jtreg/vmTestbase/nsk/jdi/TypeComponent/isSynthetic/issynthetic002.java.frames.html
 153                     log.complain("debuger FAILURE 3> Method's " + name + signature
 154                             + " synthetic is true, but expected "
 155                             + "is false");
The sub-string at lines 154 and 155 can be combined into one:
   + " synthetic is true, but expected is false");

But I'd suggest to simplify it to something like this:
   log.complain("debugger FAILURE 3> found unexpected synthetic method " + name + signature); 

Also, there are many instances of the typo: debuger => debugger.
Could you, please, fix it?

Thank you a lot for taking care about this!

Thanks,
Serguei


On 6/5/18 11:38, Daniil Titov wrote:
Please review the change that fixes vmTestbase/nsk/jdi/TypeComponent/isSynthetic/issynthetic002/TestDescription.java test. 
 
The fix uses the implementation of a parametrized interface in the test class to trigger the compiler to generate a synthetic bridge method. The test works fine in both nestmates and jdk/jdk repositories.
 
Issue: https://bugs.openjdk.java.net/browse/JDK-8203033 
Webrev: http://cr.openjdk.java.net/~dtitov/8203033/webrev.02
 
Thanks,
Daniil
 
 
 




-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180606/3023750d/attachment.html>