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