On Wed, 8 Jul 2026 17:51:54 GMT, Michael Strauß <[email protected]> wrote:

> Having said that, does #1081 fix this particular issue (no way to check 
> quickly, it seems to have a merge conflict).

I just checked that, the PR fixes this issue. 
cc @hjohn, as you wrote in 
https://github.com/openjdk/jfx/pull/1081#issuecomment-4509121523 - Here is a 
real bug that would have already been solved by your changes.
Additionally, the `listenerB` does not run as it is removed - as expected.

I understand that the PR is bigger than this localized fix. On the other hand, 
the PR has been around for three years now, and John is very knowledgeable 
about properties and bindings. And I tested with multiple big applications back 
then and found no regressions.

And on the other hand, most developers don't use bindings that 'complex' or 
learned to avoid them (like me), because removing a listener in another is not 
that common and can create problems very fast, as we can see in the reproducer.

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

PR Comment: https://git.openjdk.org/jfx/pull/2200#issuecomment-4918896960

Reply via email to