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

bug 371797: prevent possible deadlock for specs detectors

Browse source
This commit is contained in:
Andrew Gvozdev 2012-04-14 13:07:36 -04:00
parent d474c67844
commit 8fbf618249
1 changed files with 15 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
core/org.eclipse.cdt.core/model/org/eclipse/cdt/internal/core/language/settings/providers/LanguageSettingsProvidersSerializer.java
View file

@ -105,6 +105,7 @@ public class LanguageSettingsProvidersSerializer {
private static ListenerList fLanguageSettingsChangeListeners = new ListenerList(ListenerList.IDENTITY); private static ListenerList fLanguageSettingsChangeListeners = new ListenerList(ListenerList.IDENTITY);
private static ILock serializingLock = Job.getJobManager().newLock(); private static ILock serializingLock = Job.getJobManager().newLock();
private static ILock serializingLockWsp = Job.getJobManager().newLock();
/** /**
* Dummy class to represent ill-defined provider. * Dummy class to represent ill-defined provider.
@ -468,20 +469,20 @@ public class LanguageSettingsProvidersSerializer {
IProject[] projects = root.getProjects(); IProject[] projects = root.getProjects();
for (IProject project : projects) { for (IProject project : projects) {
if (project.isAccessible()) { try {
ICProjectDescription prjDescription = CCorePlugin.getDefault().getProjectDescription(project, false); if (project.isAccessible()) {
if (prjDescription != null) { ICProjectDescription prjDescription = CCorePlugin.getDefault().getProjectDescription(project, false);
try { if (prjDescription != null) {
LanguageSettingsChangeEvent event = createEvent(prjDescription, providerIds); LanguageSettingsChangeEvent event = createEvent(prjDescription, providerIds);
if (event != null) { if (event != null) {
events.add(event); events.add(event);
} }
} catch (Throwable e) {
// log and swallow any exception
CCorePlugin.log("Error creating event about changes in workspace language settings providers, " //$NON-NLS-1$
+ "project=" + project.getName(), e); //$NON-NLS-1$
} }
} }
} catch (Throwable e) {
// log and swallow any exception
CCorePlugin.log("Error creating event about changes in workspace language settings providers, " //$NON-NLS-1$
+ "project=" + project.getName(), e); //$NON-NLS-1$
} }
} }
@ -508,12 +509,12 @@ public class LanguageSettingsProvidersSerializer {
if (serializableWorkspaceProviders.isEmpty()) { if (serializableWorkspaceProviders.isEmpty()) {
java.io.File fileStoreWsp = new java.io.File(uriStoreWsp); java.io.File fileStoreWsp = new java.io.File(uriStoreWsp);
try { try {
serializingLock.acquire(); serializingLockWsp.acquire();
fileStoreWsp.delete(); fileStoreWsp.delete();
// manufacture events while inside the lock // manufacture events while inside the lock
events = createLanguageSettingsChangeEvents(serializableWorkspaceProviders); events = createLanguageSettingsChangeEvents(serializableWorkspaceProviders);
} finally { } finally {
serializingLock.release(); serializingLockWsp.release();
} }
} else { } else {
Document doc = XmlUtil.newDocument(); Document doc = XmlUtil.newDocument();
@ -526,12 +527,12 @@ public class LanguageSettingsProvidersSerializer {
} }
try { try {
serializingLock.acquire(); serializingLockWsp.acquire();
XmlUtil.serializeXml(doc, uriStoreWsp); XmlUtil.serializeXml(doc, uriStoreWsp);
// manufacture events while inside the lock // manufacture events while inside the lock
events = createLanguageSettingsChangeEvents(serializableWorkspaceProviders); events = createLanguageSettingsChangeEvents(serializableWorkspaceProviders);
} finally { } finally {
serializingLock.release(); serializingLockWsp.release();
} }
} }
// notify the listeners outside the lock // notify the listeners outside the lock
@ -596,12 +597,12 @@ public class LanguageSettingsProvidersSerializer {
Document doc = null; Document doc = null;
try { try {
serializingLock.acquire(); serializingLockWsp.acquire();
doc = XmlUtil.loadXml(uriStoreWsp); doc = XmlUtil.loadXml(uriStoreWsp);
} catch (Exception e) { } catch (Exception e) {
CCorePlugin.log("Can't load preferences from file " + uriStoreWsp, e); //$NON-NLS-1$ CCorePlugin.log("Can't load preferences from file " + uriStoreWsp, e); //$NON-NLS-1$
} finally { } finally {
serializingLock.release(); serializingLockWsp.release();
} }
if (doc != null) { if (doc != null) {