Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer [v3]

2024-04-24 Thread n-gabe
On Mon, 22 Apr 2024 15:05:07 GMT, n-gabe 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 vref : viewRefs) and removing items from >> the collection by

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer

2024-04-24 Thread Kevin Rushforth
On Sun, 21 Apr 2024 14:37:23 GMT, n-gabe 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 vref : viewRefs) and removing items from >> the collection by

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer [v3]

2024-04-22 Thread Alexander Matveev
On Mon, 22 Apr 2024 15:05:07 GMT, n-gabe 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 vref : viewRefs) and removing items from >> the collection by

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer [v2]

2024-04-22 Thread n-gabe
On Mon, 22 Apr 2024 13:52:39 GMT, Kevin Rushforth wrote: >> 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: *

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer [v3]

2024-04-22 Thread n-gabe
> 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 vref : viewRefs) and removing items from > the collection by viewRefs.remove(vref); within this loop. n-gabe has

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer [v2]

2024-04-22 Thread Kevin Rushforth
On Sun, 21 Apr 2024 14:39:46 GMT, n-gabe 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 vref : viewRefs) and removing items from >> the collection by

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer

2024-04-21 Thread n-gabe
On Thu, 22 Feb 2024 17:16:42 GMT, n-gabe 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 vref : viewRefs) and removing items from > the collection by

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer [v2]

2024-04-21 Thread n-gabe
> 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 vref : viewRefs) and removing items from > the collection by viewRefs.remove(vref); within this loop. n-gabe has

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer

2024-04-19 Thread Alexander Matveev
On Thu, 22 Feb 2024 17:16:42 GMT, n-gabe 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 vref : viewRefs) and removing items from > the collection by

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer

2024-04-19 Thread n-gabe
On Thu, 22 Feb 2024 17:16:42 GMT, n-gabe 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 vref : viewRefs) and removing items from > the collection by

Re: RFR: 8146918: ConcurrentModificationException in MediaPlayer

2024-04-18 Thread Alexander Matveev
On Thu, 22 Feb 2024 17:16:42 GMT, n-gabe 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 vref : viewRefs) and removing items from > the collection by

RFR: 8146918: ConcurrentModificationException in MediaPlayer

2024-04-18 Thread n-gabe
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 vref : viewRefs) and removing items from the collection by viewRefs.remove(vref); within this loop. - Commit