On Sun, 21 Apr 2024 14:39:46 GMT, n-gabe <d...@openjdk.org> wrote:

>> There is a ConcurrentModificationException in MediaPlayer when removing a 
>> MediaView from it. The root cause is that you can't iterate over a HashSet 
>> with for (WeakReference<MediaView> vref : viewRefs) and removing items from 
>> the collection by viewRefs.remove(vref); within this loop.
>
> n-gabe has updated the pull request incrementally with one additional commit 
> since the last revision:
> 
>   Update Copyright

modules/javafx.media/src/main/java/javafx/scene/media/MediaPlayer.java line 2:

> 1: /*
> 2:  * Copyright (c) 2010, 2022, 2024, Oracle and/or its affiliates. All 
> rights reserved.

Copyright lines must have exactly one or two years. Please remove `, 2022`

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1377#discussion_r1574796815

Reply via email to