Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorspingel2010-01-10 21:26:07 -0500
committerspingel2010-01-10 21:26:07 -0500
commitd08e5656b504c0f5cd322f341767e570439ad477 (patch)
tree511b8b00952717c57fd178d276352f0d4f58b0c4 /org.eclipse.mylyn.tasks.core
parentd1e092244733a2f43b1b5908320a436fbaa7692f (diff)
downloadorg.eclipse.mylyn.tasks-d08e5656b504c0f5cd322f341767e570439ad477.tar.gz
org.eclipse.mylyn.tasks-d08e5656b504c0f5cd322f341767e570439ad477.tar.xz
org.eclipse.mylyn.tasks-d08e5656b504c0f5cd322f341767e570439ad477.zip
NEW - bug 280015: IllegalArgumentException dumped to console while running tests
https://bugs.eclipse.org/bugs/show_bug.cgi?id=280015
Diffstat (limited to 'org.eclipse.mylyn.tasks.core')
-rw-r--r--org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/ExternalizationManager.java15
1 files changed, 7 insertions, 8 deletions
diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/ExternalizationManager.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/ExternalizationManager.java
index 611db2573..0fbc14784 100644
--- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/ExternalizationManager.java
+++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/internal/tasks/core/externalization/ExternalizationManager.java
@@ -28,7 +28,6 @@ import org.eclipse.core.runtime.Status;
import org.eclipse.core.runtime.SubProgressMonitor;
import org.eclipse.core.runtime.jobs.ISchedulingRule;
import org.eclipse.core.runtime.jobs.Job;
-import org.eclipse.mylyn.commons.core.CoreUtil;
import org.eclipse.mylyn.commons.core.StatusHandler;
import org.eclipse.mylyn.commons.net.Policy;
import org.eclipse.mylyn.internal.tasks.core.ITasksCoreConstants;
@@ -132,11 +131,7 @@ public class ExternalizationManager {
public void requestSave() {
if (!saveDisabled) {
- if (!CoreUtil.TEST_MODE) {
- saveJob.schedule(SAVE_DELAY);
- } else {
- saveJob.run(new NullProgressMonitor());
- }
+ saveJob.schedule(SAVE_DELAY);
}
}
@@ -145,14 +140,18 @@ public class ExternalizationManager {
saveDisabled = true;
// run save job as early as possible
- saveJob.wakeUp();
- saveJob.join();
+ saveNow();
} catch (InterruptedException e) {
StatusHandler.log(new Status(IStatus.ERROR, ITasksCoreConstants.ID_PLUGIN,
"Task List save on shutdown canceled.", e)); //$NON-NLS-1$
}
}
+ public void saveNow() throws InterruptedException {
+ saveJob.wakeUp();
+ saveJob.join();
+ }
+
/**
* Clients invoking this method must hold all necessary scheduling rules.
*/

Back to the top