RFR: 8332340: Add JavacBench as a test case for CDS [v3]
Ioi Lam
iklam at openjdk.org
Wed May 22 16:49:36 UTC 2024
On Tue, 21 May 2024 21:03:20 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:
>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>>
>> @calvinccheung comments
>
> test/lib/jdk/test/lib/cds/CDSAppTester.java line 147:
>
>> 145: }
>> 146:
>> 147: private OutputAnalyzer dumpStaticArchive() throws Exception {
>
> The code from 156 to 162 is repeated 3 times here, is it worth making another function for this?
Thanks for the suggestion. I've refactored the code a bit to make it easier to write new execution modes (which will be needed in Leyden).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19256#discussion_r1610325214
More information about the core-libs-dev
mailing list