RFR: 8290040: Provide simplified deterministic way to manage listeners [v3]

John Hendrikx jhendrikx at openjdk.org
Sun Sep 4 19:49:45 UTC 2022


On Thu, 1 Sep 2022 17:51:20 GMT, Nir Lisker <nlisker at openjdk.org> wrote:

>> John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update javadoc of "when" with better phrasing
>
> modules/javafx.base/src/main/java/javafx/beans/value/ObservableValue.java line 268:
> 
>> 266:      * for garbage collection while its source isn't. However, when using {@code when} this {@code ObservableValue}
>> 267:      * can still be eligible for garbage collection when {@code condition} holds {@code false} and {@code condition}
>> 268:      * itself is also eligible for garbage collection.
> 
> I think that these 2 paragraphs can be combined. There is some repetition in the last sentence ("However...") of the 1st paragraph. Maybe add to the 1st paragraph at the end:
> 
>> This is in contrast to the general behavior of bindings, where the binding is never eligible for GC as long as the source isn't.
> 
> If you think it's important to note that. Then remove the 2nd paragraph?

Yes, I've changed it.  I had to rephrase it to: "This is in contrast to the general behavior of bindings, where the binding is only eligible for garbage collection when not observed itself." -- with `when` there are two options for allowing GC, the one I just mentioned and when `condition` is `false` and not referenced itself.

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

PR: https://git.openjdk.org/jfx/pull/830


More information about the openjfx-dev mailing list