mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-08-16 04:35:45 +02:00
Code Issues Wiki Activity

Improve strategy to re-query in ElementSelectionDialog, related to bug 211251

Browse source
This commit is contained in:
Anton Leherbauer 2008-04-04 07:34:07 +00:00
parent b7e830fccc
commit dcf186c8c1
1 changed files with 24 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
core/org.eclipse.cdt.ui/browser/org/eclipse/cdt/internal/ui/browser/opentype/ElementSelectionDialog.java
View file

@ -1,5 +1,5 @@
/*******************************************************************************
* Copyright (c) 2007 Wind River Systems, Inc. and others.
* Copyright (c) 2007, 2008 Wind River Systems, Inc. and others.
* All rights reserved. This program and the accompanying materials
* are made available under the terms of the Eclipse Public License v1.0
* which accompanies this distribution, and is available at
@ -66,6 +66,12 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
*/
private class UpdateElementsJob extends Job {
/**
* The last used prefix to query the index. <code>null</code> means
* the query result should be empty.
*/
private volatile char[] fCurrentPrefix = null;
public UpdateElementsJob(String name) {
super(name);
setSystem(true);
@ -73,6 +79,15 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
setPriority(Job.LONG);
}
public char[] getCurrentPrefix() {
return fCurrentPrefix;
}
public void scheduleQuery(char[] prefix) {
fCurrentPrefix= prefix;
int delay = fCurrentPrefix == null ? 0 : (fCurrentPrefix.length < 5 ? 400 : 200);
schedule(delay);
}
@Override
public IStatus run(final IProgressMonitor monitor) {
monitor.beginTask(OpenTypeMessages.ElementSelectionDialog_UpdateElementsJob_inProgress, IProgressMonitor.UNKNOWN);
@ -148,12 +163,7 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
return rule == this;
}};
/**
* The last used prefix to query the index. <code>null</code> means the
* query result should be empty.
*/
private char[] fCurrentPrefix= null;
private Job fUpdateJob;
private UpdateElementsJob fUpdateJob;
private boolean fAllowEmptyPrefix= true;
private boolean fAllowEmptyString= true;
private ProgressMonitorPart fProgressMonitorPart;
@ -356,17 +366,18 @@ public class ElementSelectionDialog extends TypeSelectionDialog {
protected void scheduleUpdate(String filterText) {
char[] newPrefix= toPrefix(filterText);
boolean equivalentPrefix= isEquivalentPrefix(fCurrentPrefix, newPrefix);
final char[] currentPrefix= fUpdateJob.getCurrentPrefix();
final boolean equivalentPrefix= isEquivalentPrefix(currentPrefix, newPrefix);
boolean emptyQuery= newPrefix.length == 0 && !fAllowEmptyPrefix || filterText.length() == 0 && !fAllowEmptyString;
boolean needQuery= !equivalentPrefix;
final int jobState = fUpdateJob.getState();
boolean needQuery= !equivalentPrefix || (currentPrefix.length > newPrefix.length && currentPrefix.length < 5 && jobState == Job.RUNNING);
if (emptyQuery) {
newPrefix= null;
needQuery= needQuery || fCurrentPrefix != null;
needQuery= needQuery || currentPrefix != null;
}
if(needQuery || fUpdateJob.getState() == Job.WAITING || fUpdateJob.getState() == Job.SLEEPING) {
if(needQuery || jobState == Job.WAITING || jobState == Job.SLEEPING) {
fUpdateJob.cancel();
fCurrentPrefix= newPrefix;
fUpdateJob.schedule(200);
fUpdateJob.scheduleQuery(newPrefix);
}
}