RFR: 8269871: CellEditEvent: must not throw NPE in accessors
mhanl at openjdk.java.net
Fri Sep 3 10:31:33 UTC 2021
On Thu, 26 Aug 2021 14:09:58 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:
> The issue is unguarded access to tablePosition though it might be null (since [JDK-8120610](https://bugs.openjdk.java.net/browse/JDK-8120610)
> The fix is to check against null in each accessor. Also required to fix the default onEditCommit handlers in Tree/TableColumn to really cope with null TablePostion on the event.
> Added tests that failed/pass before/after the fix.
> Note that there was an old test (in Tree/TableColumnTest each), that failed after the fix because it expected the NPE. Fixed by removing the expected parameter.
Looks good to me.
Good to see this fixed, I remember I had some NPEs in the past because of a null table position.
Marked as reviewed by mhanl (Author).
More information about the openjfx-dev