On Fri, 4 Oct 2024 18:13:11 GMT, Henry Jen <henry...@openjdk.org> wrote:

>> src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/SystemModulesPlugin.java
>>  line 1714:
>> 
>>> 1712:             }
>>> 1713: 
>>> 1714:             class SetReference<T extends Comparable<T>> implements 
>>> Comparable<SetReference<T>> {
>> 
>> The class name `SetReference` is not obvious what it does.   A comment would 
>> be helpful.
>
> Added, please let me know any suggestions or questions.

What about `RefCountedSet`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21022#discussion_r1894442861

Reply via email to