Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSilenio Quarti2009-11-05 16:38:40 -0500
committerSilenio Quarti2009-11-05 16:38:40 -0500
commit6474e7939600422ffd1bd5925b79f3dd2407f440 (patch)
tree833479732659897428dac91f30e7b18542dbceb1
parent0c0ed69d97480cbb2ec2c3f4549bc02276c0d474 (diff)
downloadeclipse.platform.swt-R3_3_1_maintenance.tar.gz
eclipse.platform.swt-R3_3_1_maintenance.tar.xz
eclipse.platform.swt-R3_3_1_maintenance.zip
Bug 142593 - Scrolling a table eventually locks up the entire application (back poort to 3.3.1.1)v3347aR3_3_1_maintenance
-rwxr-xr-xbundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Table.java24
1 files changed, 12 insertions, 12 deletions
diff --git a/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Table.java b/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Table.java
index c162d185d1..fa18d2d43e 100755
--- a/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Table.java
+++ b/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Table.java
@@ -5843,6 +5843,17 @@ LRESULT wmNotifyChild (NMHDR hdr, int wParam, int lParam) {
// if (drawCount != 0 || !OS.IsWindowVisible (handle)) break;
NMLVDISPINFO plvfi = new NMLVDISPINFO ();
OS.MoveMemory (plvfi, lParam, NMLVDISPINFO.sizeof);
+
+ /*
+ * Feature in Windows. When a new table item is inserted
+ * using LVM_INSERTITEM in a table that is transparent
+ * (ie. LVM_SETBKCOLOR has been called with CLR_NONE),
+ * TVM_INSERTITEM calls LVN_GETDISPINFO before the item
+ * has been added to the array. The fix is to check for
+ * null.
+ */
+ TableItem item = _getItem (plvfi.iItem);
+ if (item == null) break;
/*
* When an item is being deleted from a virtual table, do not
@@ -5853,23 +5864,12 @@ LRESULT wmNotifyChild (NMHDR hdr, int wParam, int lParam) {
* in an inconsistent state. Rather than answering the data
* right away, queue a redraw for later.
*/
- if ((style & SWT.VIRTUAL) != 0) {
+ if ((style & SWT.VIRTUAL) != 0 && !item.cached) {
if (ignoreShrink) {
OS.SendMessage (handle, OS.LVM_REDRAWITEMS, plvfi.iItem, plvfi.iItem);
break;
}
}
-
- /*
- * Feature in Windows. When a new table item is inserted
- * using LVM_INSERTITEM in a table that is transparent
- * (ie. LVM_SETBKCOLOR has been called with CLR_NONE),
- * TVM_INSERTITEM calls LVN_GETDISPINFO before the item
- * has been added to the array. The fix is to check for
- * null.
- */
- TableItem item = _getItem (plvfi.iItem);
- if (item == null) break;
/*
* The cached flag is used by both virtual and non-virtual

Back to the top