Re: RFR: 8314755: Resource leak: SwingWorker listener keeps strong reference to executor [v4]

2023-10-02 Thread Alexey Ivanov
On Tue, 12 Sep 2023 12:34:12 GMT, Christopher Sahnwaldt wrote: >> In >> https://github.com/openjdk/jdk/commit/b8af3d50192f8bc98d83f8102f0fd1989f302e32 >> the weak reference was accidentally changed from a field to a local >> variable, which means that the PropertyChangeListener keeps a strong

Re: RFR: 8314755: Resource leak: SwingWorker listener keeps strong reference to executor [v4]

2023-09-12 Thread Christopher Sahnwaldt
On Tue, 12 Sep 2023 12:34:12 GMT, Christopher Sahnwaldt wrote: >> In >> https://github.com/openjdk/jdk/commit/b8af3d50192f8bc98d83f8102f0fd1989f302e32 >> the weak reference was accidentally changed from a field to a local >> variable, which means that the PropertyChangeListener keeps a strong

Re: RFR: 8314755: Resource leak: SwingWorker listener keeps strong reference to executor [v4]

2023-09-12 Thread Christopher Sahnwaldt
On Tue, 12 Sep 2023 12:34:12 GMT, Christopher Sahnwaldt wrote: >> In >> https://github.com/openjdk/jdk/commit/b8af3d50192f8bc98d83f8102f0fd1989f302e32 >> the weak reference was accidentally changed from a field to a local >> variable, which means that the PropertyChangeListener keeps a strong

Re: RFR: 8314755: Resource leak: SwingWorker listener keeps strong reference to executor [v4]

2023-09-12 Thread Christopher Sahnwaldt
> In > https://github.com/openjdk/jdk/commit/b8af3d50192f8bc98d83f8102f0fd1989f302e32 > the weak reference was accidentally changed from a field to a local > variable, which means that the PropertyChangeListener keeps a strong > reference to executorService, which is a resource leak Christophe