> `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 with a new target base due to a 
merge or a rebase. The incremental webrev excludes the unrelated changes 
brought in by the merge/rebase. The pull request contains two additional 
commits since the last revision:

 - Merge
 - TextComponent's getBeforeIndex boundary value check fix

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25941/files
  - new: https://git.openjdk.org/jdk/pull/25941/files/6e5eef63..b7852974

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25941&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25941&range=00-01

  Stats: 52056 lines in 1209 files changed: 35231 ins; 10992 del; 5833 mod
  Patch: https://git.openjdk.org/jdk/pull/25941.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25941/head:pull/25941

PR: https://git.openjdk.org/jdk/pull/25941

Reply via email to