From 5b31b196fa478a605ce7f8229c7766c534940f6f Mon Sep 17 00:00:00 2001
From: Anton Leherbauer <anton.leherbauer@windriver.com>
Date: Tue, 24 Apr 2007 09:43:44 +0000
Subject: [PATCH] Open Element: Faster queries and cancellation

---
 .../internal/core/pdom/dom/NamedNodeCollector.java   |  2 +-
 .../ui/browser/opentype/ElementSelectionDialog.java  | 12 ++++++------
 .../cdt/ui/browser/typeinfo/TypeSelectionDialog.java |  1 +
 3 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/pdom/dom/NamedNodeCollector.java b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/pdom/dom/NamedNodeCollector.java
index 66eae4e301e..91d3fb21f52 100644
--- a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/pdom/dom/NamedNodeCollector.java
+++ b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/pdom/dom/NamedNodeCollector.java
@@ -127,7 +127,7 @@ public class NamedNodeCollector implements IBTreeVisitor, IPDOMVisitor {
 	}
 
 	private void checkCancelled() {
-		if (++monitorCheckCounter % 0x100 == 0 && monitor.isCanceled()) {
+		if (++monitorCheckCounter % 0x1000 == 0 && monitor.isCanceled()) {
 			throw new OperationCanceledException();
 		}
 	}
diff --git a/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/internal/ui/browser/opentype/ElementSelectionDialog.java b/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/internal/ui/browser/opentype/ElementSelectionDialog.java
index 73df3545be1..9aae136034b 100644
--- a/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/internal/ui/browser/opentype/ElementSelectionDialog.java
+++ b/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/internal/ui/browser/opentype/ElementSelectionDialog.java
@@ -242,7 +242,7 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
 					index.acquireReadLock();
 					IBinding[] bindings= index.findBindingsForPrefix(prefix, false, IndexFilter.ALL_DECLARED, monitor);
 					for(int i=0; i<bindings.length; i++) {
-						if ((i % 100) == 0 && monitor.isCanceled()) {
+						if (i % 0x1000 == 0 && monitor.isCanceled()) {
 							return null;
 						}
 						IBinding binding = bindings[i];
@@ -254,11 +254,11 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
 								if(binding instanceof ICPPBinding) {
 									fqn= ((ICPPBinding)binding).getQualifiedName();
 								} else {
-									fqn = new String[] {binding.getName()};
+									fqn= new String[] {binding.getName()};
 								}
 								if (binding instanceof IFunction) {
-									final IFunction function = (IFunction)binding;
-									final String[] paramTypes = IndexModelUtil.extractParameterTypes(function);
+									final IFunction function= (IFunction)binding;
+									final String[] paramTypes= IndexModelUtil.extractParameterTypes(function);
 									final String returnType= IndexModelUtil.extractReturnType(function);
 									types.add(new IndexTypeInfo(fqn, elementType, paramTypes, returnType, index));
 								} else {
@@ -266,7 +266,7 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
 								}
 							}
 						} catch(DOMException de) {
-
+							CCorePlugin.log(de);
 						}
 					}
 				} finally {
@@ -309,7 +309,7 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
 
 	protected void scheduleUpdate(String filterText) {
 		char[] newPrefix= toPrefix(filterText);
-		if (!isEquivalentPrefix(fCurrentPrefix, newPrefix)) {
+		if (fUpdateJob.getState() == Job.RUNNING || !isEquivalentPrefix(fCurrentPrefix, newPrefix)) {
 			fUpdateJob.cancel();
 			fCurrentPrefix= newPrefix;
 			fUpdateJob.schedule(200);
diff --git a/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/ui/browser/typeinfo/TypeSelectionDialog.java b/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/ui/browser/typeinfo/TypeSelectionDialog.java
index 58c5096b56a..eb6ccbe1450 100644
--- a/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/ui/browser/typeinfo/TypeSelectionDialog.java
+++ b/core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/ui/browser/typeinfo/TypeSelectionDialog.java
@@ -458,6 +458,7 @@ public class TypeSelectionDialog extends TwoPaneElementSelector {
 	 */
 	void updateElements() {
 		fNewFilteredList.setFilter(fTextWidget.getText());
+		handleSelectionChanged();
 	}
 	
 	/**