On Thu, 10 Aug 2023 09:41:29 GMT, Yi Yang <yy...@openjdk.org> wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last revision: > > fix TestHeapDumpForInvokDynamic
Need to update copyright year in the files test/hotspot/jtreg/serviceability/sa/TestHeapDumpForInvokeDynamic.java line 40: > 38: import jdk.test.lib.hprof.parser.HprofReader; > 39: import jdk.test.lib.hprof.parser.PositionDataInputStream; > 40: import jdk.test.lib.hprof.HprofParser; PositionDataInputStream and HprofParser imports are not used anymore ------------- PR Comment: https://git.openjdk.org/jdk/pull/15202#issuecomment-1673795176 PR Review Comment: https://git.openjdk.org/jdk/pull/15202#discussion_r1290590834
