On Fri, 15 May 2026 08:32:36 GMT, Alan Bateman <[email protected]> wrote:

>> Manuel Hässig has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 58 commits:
>> 
>>  - Copyright years
>>  - Merge branch 'master' into remove-jvmci
>>  - serviceablity: remove CompilerTread.java
>>  - Remove unused boxed primitive classes
>>  - More dead code
>>    
>>    Co-Authored-By: Vladimir Ivanov <[email protected]>
>>  - Fix some asserts
>>    
>>    Co-Authored-By: Vladimir Kozlov <[email protected]>
>>  - Simplify ClassNameJrtStreamBuilder::systemModules
>>    
>>    Co-Authored-By: Alan Bateman <[email protected]>
>>  - Add back JVMCI flags to description of -XX:+NeverActAsServerClassMachine
>>  - Merge branch 'master' into remove-jvmci
>>  - Merge branch 'master' into remove-jvmci
>>  - ... and 48 more: https://git.openjdk.org/jdk/compare/e9bc05c7...2cf80162
>
> test/jdk/java/lang/Class/getDeclaredField/FieldSetAccessibleTest.java line 
> 280:
> 
>> 278:                 moduleToDeps.put(md.name(), deps);
>> 279:                 inverseDeps.put(md.name(), new HashSet<>());
>> 280:             }
> 
> Since this no longer filters the jdk.graal.* modules then it should be 
> possible to remove moduleToDeps and inverseDeps too, and simplify it down to
> 
>         return ModuleFinder.ofSystem()
>                 .findAll()
>                 .stream()
>                 .map(mref -> mref.descriptor().name())
>                 .collect(Collectors.toSet());

Thank you for the suggestions. Simplified in a96f3361f55.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30834#discussion_r3257555578

Reply via email to