On Mon, 28 Apr 2025 23:48:04 GMT, Sergey Bylokhov <[email protected]> wrote:
>> No need to call `List.indexOf(Object)` before `List.remove(int)`. Instead we
>> can call `List.remove(Object)` directly. It's faster and cleaner.
>> `invalidComponents` is an ArrayList.
>
> src/java.desktop/share/classes/javax/swing/RepaintManager.java line 394:
>
>> 392: }
>> 393: if (invalidComponents != null) {
>> 394: invalidComponents.remove(component);
>
> I guess the `addInvalidComponent` above can be simplified as well?
Simplified how?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24845#discussion_r2066621174
