On Thu, 13 Aug 2026 22:04:31 GMT, Andy Goryachev <[email protected]> wrote:

>> Alexander Zuev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Added @since annotation.
>
> modules/javafx.graphics/src/main/java/javafx/scene/AccessibleAttribute.java 
> line 809:
> 
>> 807:      * <li>Used by: ListView and TableView </li>
>> 808:      * <li>Needs notify: no </li>
>> 809:      * <li>Return Type: {@code int[]} containing the first item index 
>> and the
> 
> please specify how it encodes a no-range case.
> 
> it looks like the code returns [0,0], but is this right?
> should it return an empty array or `null` instead?

The {0, 0} is the valid range, we should not return empty array or null in any 
case.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2217#discussion_r3781183481

Reply via email to