From 09758be2bc861ad55fb601f6a362762abb17bf98 Mon Sep 17 00:00:00 2001 From: David Dykstal Date: Mon, 25 Aug 2008 22:28:04 +0000 Subject: [PATCH] [243128] Problem during migration - NPE found in RSEPersistenceManager https://bugs.eclipse.org/bugs/show_bug.cgi?id=243128 --- .../rse/internal/persistence/RSEPersistenceManager.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/persistence/RSEPersistenceManager.java b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/persistence/RSEPersistenceManager.java index ff000ce4057..1662f1b8c7a 100644 --- a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/persistence/RSEPersistenceManager.java +++ b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/internal/persistence/RSEPersistenceManager.java @@ -22,6 +22,7 @@ * David Dykstal (IBM) - [226728] NPE during init with clean workspace * David Dykstal (IBM) - [197027] Can lose data if closing eclipse before profile is saved * Kevin Doyle (IBM) - [243821] Save occurring on Main Thread + * David Dykstal (IBM) - [243128] Problem during migration - NPE if provider does save without using a job. ********************************************************************************/ package org.eclipse.rse.internal.persistence; @@ -551,8 +552,8 @@ public class RSEPersistenceManager implements IRSEPersistenceManager { cleanTree(profile); if (dom.needsSave()) { Job job = provider.getSaveJob(dom); - job.addJobChangeListener(jobChangeListener); if (job != null && canScheduleSave) { + job.addJobChangeListener(jobChangeListener); job.schedule(2000); // two second delay } else { provider.saveRSEDOM(dom, new NullProgressMonitor());