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