On Fri, 9 Jun 2023 22:11:10 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> modules/javafx.graphics/src/main/java/com/sun/javafx/css/BitSet.java line 44:
>>
>>> 42: * which makes matching faster.
>>> 43: */
>>> 44: abstract class BitSet<T> extends AbstractSet<T> implements
>>> ObservableSet<T> {
>>
>> Since this is an internal class, would it be possible / make sense to rename
>> it, to avoid confusion with the real Slim^H^H^H^H BitSet? StyleBitSetBase
>> or some such?
>
> BitSet name still bugs me.
Sure, but I think that would be out of scope for this PR.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1076#discussion_r1224906677
