On Tue, 24 Jun 2025 11:44:13 GMT, Abhishek Kumar <abhis...@openjdk.org> wrote:

>> `AccessibleText.getBeforeIndex` method returns `null for last characte`r due 
>> to the **wrong boundary value condition check**.
>> This method returns `null` when the `passed index parameter` is equal to 
>> `text's length` which is incorrect.
>> `getBeforeIndex` method should return `null` only if the **passed index 
>> parameter is less than 0 and greater than the text's length**.
>> 
>> After modifying the condition check, expected character is returned. Test is 
>> added to verify the check,
>
> Abhishek Kumar has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Test extended for WORD and SENTENCE

Looks good.

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

Marked as reviewed by kizune (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25941#pullrequestreview-2962728423

Reply via email to