On Fri, 15 Jul 2022 07:19:19 GMT, John Hendrikx <[email protected]> wrote:
>> I introduced a bug with the fluent bindings PR which affects all
>> ObjectBindings.
>>
>> This is the code that fails:
>>
>> SimpleObjectProperty<Boolean> condition = new
>> SimpleObjectProperty<>(true);
>> ObservableValue<String> binding = condition.map(Object::toString);
>>
>> binding.addListener(o -> { binding.getValue(); });
>>
>> condition.set(false);
>>
>> assertEquals(false, binding.getValue()); // returns null (!)
>>
>> This PR fixes this problem and adds a test case to cover it.
>
> John Hendrikx has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Change explanatory comment to block comment
Marked as reviewed by mstrauss (Author).
-------------
PR: https://git.openjdk.org/jfx/pull/829