RFR: 8290310: ChangeListener events are incorrect or misleading when a nested change occurs [v12]

[Michael Strauß]

On Mon, 10 Mar 2025 16:14:36 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:

> Or perhaps you are asking if we can just ignore 4.iv and not do 4.v ? That's what the code did a few changes ago, but which would leave you in the dark that there is conflicting changes happening (whereas with `ExpressionHelper` this would lead to the `StackOverflowError` -- not perfect, but at least you're informed...)

What if we didn't call back the second listener (as you said, it would probably result in an infinite loop), and logged an error instead (that's not without precedence, bindings also log errors)? It seems like this would still allow other listeners to receive notifications, while the "defective" listener would not.

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

PR Comment: https://git.openjdk.org/jfx/pull/1081#issuecomment-2711312666