On Thu, 10 Dec 2020 23:57:46 GMT, Mandy Chung <mch...@openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   removed lookup for inherited premain/agentmain methods
>
> Please update @bug in the tests to include this bug ID.
> 
> All InheritAgentXXX tests are updated to have the InheritAgentXXX class 
> public.  However, I think you need to go through them individually for this 
> behavioral change.   The existing comments may not be applicable and please 
> update them where appropriate.    For example InheritAgent0100 previously 
> verifies the invocation of the premain in its superclass.  Does the modified 
> test ensure that this fails to load?  Per the comment in the new 
> InheritAgent0100Super class, it expects the superclass' premain should be 
> called.

Mandy, thank you for comments.
You are right, the comments at 429-443 tests need some updates. I'm working on 
it now.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1694

Reply via email to