RFR: 8271484: Tree-/TableCell: NPE when accessing edit event from startEdit
Ajit Ghaisas
aghaisas at openjdk.java.net
Wed Aug 11 08:41:30 UTC 2021
On Thu, 29 Jul 2021 14:11:15 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:
> The NPE was an indirect effect:
>
> - the bug on part of the cell was an incorrect (== missing) edit location in cellEditEvent - that's fixed in this PR
> - a related bug is on part of the implementation of CellEditEvent (assuming that TablePosition != null, [JDK-8269871](https://bugs.openjdk.java.net/browse/JDK-8269871) which is not addressed in this PR
>
> Fix is to use the cell's state at startEdit for creating the editing location, added tests starting the edit on the cell that failed/passed before/after the fix, also added sanity tests with starting edit on the control that passed before/after.
Marked as reviewed by aghaisas (Reviewer).
The fix looks good to me.
-------------
PR: https://git.openjdk.java.net/jfx/pull/592
More information about the openjfx-dev
mailing list