On Tue, 12 Jul 2022 13:45:13 GMT, Marius Hanl <mh...@openjdk.org> wrote:

>> Johan Vos has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   reverse one more expected/actual set of args.
>
> modules/javafx.controls/src/main/java/javafx/scene/control/skin/VirtualFlow.java
>  line 1074:
> 
>> 1072:             lastHeight = -1;
>> 1073:             releaseCell(accumCell);
>> 1074: //            accumCell = null;
> 
> Is this intentional or did you may forgot to add back this two lines of code?

This is intentional -- it was already moved to comments, and there is no need 
to destroy the accumCell (releasing it is what we need)

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

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

Reply via email to