On Thu, 5 Jun 2025 20:58:19 GMT, Jean-Noël Rouvignac <[email protected]> wrote:
>> src/java.desktop/share/classes/javax/sound/midi/Track.java line 202:
>>
>>> 200: synchronized(eventsList) {
>>> 201: if (set.remove(event)) {
>>> 202: if (eventsList.remove(event)) {
>>
>> Can be rewritten as `if (set.remove(event) && eventsList.remove(event))`.
>
> `return set.remove(event) && eventsList.remove(event);` even...
>
> And remove the return statement coming after
This is *cleaner*.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24818#discussion_r2157582518
