RFR: 8359996: Remove unnecessary List.indexOf key from Track.remove [v2]

Alexey Ivanov aivanov at openjdk.org
Fri Jun 20 19:20:29 UTC 2025


On Thu, 19 Jun 2025 20:28:42 GMT, Andrey Turbanov <aturbanov at openjdk.org> 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.
>> `eventsList` is an ArrayList.
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8359996: Remove unnecessary List.indexOf key from Track.remove
>   
>   apply suggestion

Looks good.

Maybe wait for another reviewer…

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

Marked as reviewed by aivanov (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/24818#pullrequestreview-2947095926


More information about the client-libs-dev mailing list