mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-06-07 17:56:01 +02:00
Code Issues Wiki Activity

Bug 315415: always clear a terminated job from pathToJobsMap, and store off fetchedExecutables as soon as possible, leaving only UI work to Display#asyncExec().

Browse source
This commit is contained in:
Ed Swartz 2010-06-04 18:23:39 +00:00
parent 8c5a9f0d91
commit 564e54347f
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/views/executables/SourceFilesContentProvider.java
View file

@ -124,15 +124,15 @@ public class SourceFilesContentProvider extends CElementContentProvider implemen
final QuickParseJob theJob = job; final QuickParseJob theJob = job;
job.addJobChangeListener(new JobChangeAdapter() { job.addJobChangeListener(new JobChangeAdapter() {
public void done(IJobChangeEvent event) { public void done(IJobChangeEvent event) {
synchronized (pathToJobMap) {
pathToJobMap.values().remove(theJob);
}
if (event.getResult().isOK()) { if (event.getResult().isOK()) {
synchronized (fetchedExecutables) {
fetchedExecutables.put(exePath, theJob.tus);
}
Display.getDefault().asyncExec(new Runnable() { Display.getDefault().asyncExec(new Runnable() {
public void run() { public void run() {
synchronized (pathToJobMap) {
pathToJobMap.values().remove(theJob);
}
synchronized (fetchedExecutables) {
fetchedExecutables.put(exePath, theJob.tus);
}
// update the viewer // update the viewer
if (!viewer.getControl().isDisposed()) { if (!viewer.getControl().isDisposed()) {
viewer.getTree().setLayoutDeferred(true); viewer.getTree().setLayoutDeferred(true);