RFR: 8273324: IllegalArgumentException: fromIndex(0) > toIndex(-1) after clear and select TableCell

Kevin Rushforth kcr at openjdk.java.net
Fri Sep 3 19:13:25 UTC 2021


On Fri, 3 Sep 2021 18:19:45 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

> This PR fixes the exception thrown by the sample code in [8273324](https://bugs.openjdk.java.net/browse/JDK-8273324), while retaining the incorrect behavior in the scenario described.
> 
> ---------
> ### Progress
> - [x] Change must not contain extraneous whitespace
> - [x] Commit message must refer to an issue
> - [ ] Change must be properly reviewed
> 
> ### Error
>  ⚠️ The pull request body must not be empty.
> 
> 
> 
> ### Reviewing
> <details><summary>Using <code>git</code></summary>
> 
> Checkout this PR locally: \
> `$ git fetch https://git.openjdk.java.net/jfx pull/617/head:pull/617` \
> `$ git checkout pull/617`
> 
> Update a local copy of the PR: \
> `$ git checkout pull/617` \
> `$ git pull https://git.openjdk.java.net/jfx pull/617/head`
> 
> </details>
> <details><summary>Using Skara CLI tools</summary>
> 
> Checkout this PR locally: \
> `$ git pr checkout 617`
> 
> View PR using the GUI difftool: \
> `$ git pr show -t 617`
> 
> </details>
> <details><summary>Using diff file</summary>
> 
> Download this PR as a diff file: \
> <a href="https://git.openjdk.java.net/jfx/pull/617.diff">https://git.openjdk.java.net/jfx/pull/617.diff</a>
> 
> </details>

The fix looks correct to me.

Can we get a unit test for this case?

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

PR: https://git.openjdk.java.net/jfx/pull/617


More information about the openjfx-dev mailing list