RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v2]
Alex Menkov
amenkov at openjdk.org
Thu Aug 10 19:51:58 UTC 2023
On Thu, 10 Aug 2023 09:41:29 GMT, Yi Yang <yyang at 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
More information about the serviceability-dev
mailing list