RFR: 8236259: MemoryLeak in ProgressIndicator

[Jeanette Winzenburg]

On Thu, 19 Dec 2019 13:35:51 GMT, Florian Kirmaier <fkirmaier at openjdk.org> wrote:

>> modules/javafx.controls/src/main/java/javafx/scene/control/skin/ProgressIndicatorSkin.java line 555:
>> 
>>> 554:         }
>>> 555: 
>>> 556:         private void setFillOverride(Paint fillOverride) {
>> 
>> just a side-note: this will remove the whole chain of listeners to that property. Not a problem here, because the indicator seems to be the only participant listening. A bit strange using the skin's LambdaMultiplePropertyChangeListenerHandler ...
> 
> Yes, do you know an API for registerChangeListener which allows me to only remove a specific listener?

no way, afaik - there had been debates when the api was added somewhere in jbs, don't recall exactly where

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

PR: https://git.openjdk.java.net/jfx/pull/71