On Thu, 8 Jun 2023 06:47:44 GMT, Marius Hanl <mh...@openjdk.org> wrote:
>> modules/javafx.controls/src/main/java/javafx/scene/control/skin/VirtualFlow.java
>> line 1954:
>>
>>> 1952: * Gets the breadth of a specific cell
>>> 1953: */
>>> 1954: double getCellBreadth(T cell) {
>>
>> is there a possibility of introducing runtime errors?
>
> No, this is just cosmetic and more correct as every cell inside the
> VirtualFlow is bound to the type parameter T.
this method should have been private.
I see no other callers outside of VirtualFlow, so it's probably safe.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1150#discussion_r1223375875
