On Fri, 15 Nov 2024 05:23:13 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Updated the test with some comments. > > test/hotspot/jtreg/serviceability/jvmti/RedefineClasses/RedefineVerifyError.java > line 71: > >> 69: fieldVisitor.visitEnd(); >> 70: } >> 71: { > > Please add a comment showing the Java code this asm is implementing. Actually why do you even need this constructor now? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22116#discussion_r1843204345