On Wed, 10 May 2023 06:57:04 GMT, John Hendrikx <[email protected]> wrote:

>> modules/javafx.controls/src/test/java/test/javafx/scene/control/skin/TreeTableRowSkinTest.java
>>  line 215:
>> 
>>> 213:         int right = 23;
>>> 214:         int bottom = 37;
>>> 215:         int left = 49;
>> 
>> After the fix and test change, this test now fails for me.
>
> I'm not quite seeing why these changes are needed.  How are they related to 
> the memory leak?

They shouldn't be needed. This points to a problem either with the fix, or a 
preexisting problem with this test.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1129#discussion_r1190136710

Reply via email to