RFR: 8311591: Add SystemModulesPlugin test case that splits module descriptors with new local variables defined by DedupSetBuilder [v6]

Mandy Chung mchung at openjdk.org
Tue Aug 15 19:58:12 UTC 2023


On Tue, 15 Aug 2023 19:54:41 GMT, Christoph <duke at openjdk.org> wrote:

>> Add new test case with sample modules that contains some requires/exports/uses/provides.
>> 
>> We are just unsure if and how we should add some last step of verificaiton with the extracted and decompiled class.
>> 
>> Follow up task from https://github.com/openjdk/jdk/pull/14408
>
> Christoph has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix missing import, use size instead of count

JLinkDedupTestBatchSizeOne.java line 115-144 can be deleted; otherwise, looks good.

-------------

PR Review: https://git.openjdk.org/jdk/pull/15234#pullrequestreview-1579317868


More information about the core-libs-dev mailing list