RFR: 8252936: Optimize removal of listeners from ExpressionHelper.Generic

dannygonzalez github.com+6702882+dannygonzalez at openjdk.java.net
Wed Sep 9 06:55:45 UTC 2020


On Tue, 8 Sep 2020 20:14:48 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>>> So I'd like to propose this third alternative, which would affect only VirtualFlow and the controls that use it, but
>>> wouldn't have any impact in the rest of controls as the other two options (as ExpressionHelper or Node listeners
>>> wouldn't be modified).
>> 
>> Given PR #185, which was mentioned above, (it isn't out for review yet, but I want to evaluate it), this would be a 4th
>> approach.
>> As long as this really doesn't introduce a leak, it seems promising.
>> 
>> I note that these are not mutually exclusive.
>> 
>> We should discuss this on the list and not just in one or more of of the 3 (soon to be 4) open pull requests.
>
> @dannygonzalez Per [this message](https://mail.openjdk.java.net/pipermail/openjfx-dev/2020-September/027534.html) on
> the openjfx-dev mailing list, I have filed a new JBS issue for this PR to use. Please change the title to:
> 8252936: Optimize removal of listeners from ExpressionHelper.Generic

Thanks @kevinrushforth, I've changed the title.

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

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


More information about the openjfx-dev mailing list