On Wed, 5 Oct 2022 18:04:41 GMT, SWinxy <[email protected]> wrote:

>> Hashtable was used only from single thread. And only non-null key/values 
>> added to it. We can safely replace it with HashSet.
>
> src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java line 
> 1364:
> 
>> 1362:                            HashSet<SelectorMapping> alreadyChecked) {
>> 1363:         // Avoid descending the same mapping twice.
>> 1364:         if (!alreadyChecked.add(parentMapping)) {
> 
> Clever!

Seriously ? That is your explanation ? Do better.

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

PR: https://git.openjdk.org/jdk/pull/10522

Reply via email to