Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 580447

Summary: TextCellEditor creates Cursor without disposing it
Product: [Technology] NatTable Reporter: Andreas Lemmer <andreaslmmr>
Component: CoreAssignee: Dirk Fauth <dirk.fauth>
Status: CLOSED FIXED QA Contact:
Severity: major    
Priority: P3 CC: dirk.fauth
Version: 2.0.2   
Target Milestone: 2.0.3   
Hardware: PC   
OS: Windows 10   
See Also: https://git.eclipse.org/r/c/nattable/org.eclipse.nebula.widgets.nattable/+/194857
https://git.eclipse.org/c/nattable/org.eclipse.nebula.widgets.nattable.git/commit/?id=f79df117305b62b4b9fbf1c3ba7171b5c58aee72
Whiteboard:

Description Andreas Lemmer CLA 2022-07-25 09:30:14 EDT
In TextCellEditor#createEditorControl l. 355, the cursor of the in-cell text field is set like
textControl.setCursor(new Cursor(Display.getDefault(), SWT.CURSOR_IBEAM));

This cursor leaks a handle and cannot be disposed anymore.

It should be implemented like Display.getDefault().getSystemCursor(SWT.CURSOR_IBEAM).
Comment 1 Dirk Fauth CLA 2022-07-25 10:21:20 EDT
Thanks for reporting. The issue also seems to be present in ComboBoxCellEditor and FilterRowComboBoxCellEditor.
Comment 2 Eclipse Genie CLA 2022-07-25 10:38:32 EDT
New Gerrit change created: https://git.eclipse.org/r/c/nattable/org.eclipse.nebula.widgets.nattable/+/194857
Comment 4 Dirk Fauth CLA 2022-08-09 02:30:54 EDT
Released with 2.0.3