Attachment 267792 Details for Bug 511793 – Patch with a single synchronized method (with explicit name) and single lock which is taken before the synchronization and without extra new lines

[patch] Patch with a single synchronized method (with explicit name) and single lock which is taken before the synchronization and without extra new lines

511793-v4.patch (text/plain), 2.16 KB, created by Aurelien Pupier

on 2017-04-13 08:52:33 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Aurelien Pupier

Created: 2017-04-13 08:52:33 EDT

Size: 2.16 KB

patch

obsolete

>diff --git a/plugins/org.eclipse.wst.common.modulecore/modulecore-src/org/eclipse/wst/common/componentcore/internal/util/ComponentImplManager.java b/plugins/org.eclipse.wst.common.modulecore/modulecore-src/org/eclipse/wst/common/componentcore/internal/util/ComponentImplManager.java
>index 422e468..283613a 100644
>--- a/plugins/org.eclipse.wst.common.modulecore/modulecore-src/org/eclipse/wst/common/componentcore/internal/util/ComponentImplManager.java
>+++ b/plugins/org.eclipse.wst.common.modulecore/modulecore-src/org/eclipse/wst/common/componentcore/internal/util/ComponentImplManager.java
>@@ -202,23 +202,24 @@
> 	}
> 
> 	public IVirtualComponent createComponent(IProject project) {
>-		IVirtualComponent retVal = null;
>-		// acquire the lock that StructureEdit will need already, to prevent others from locking that before calling createComponent() - see bug 508685
>+		return createComponent(project, true);
>+	}
>+
>+	public IVirtualComponent createComponent(IProject project, boolean checkSettings) {
> 		ILock lock = EMFWorkbenchEditContextFactory.getProjectLockObject(project);
> 		try{
> 			if(null != lock){
> 				lock.acquire();
> 			}
>-			retVal = createComponent(project, true);
>+			return createComponentSynchronously(project, checkSettings);
> 		} finally{
> 			if(null != lock){
> 				lock.release();
> 			}
> 		}
>-		return retVal;
> 	}
> 
>-	public synchronized IVirtualComponent createComponent(IProject project, boolean checkSettings) {
>+	private synchronized IVirtualComponent createComponentSynchronously(IProject project, boolean checkSettings) {
> 		try {
> 			IVirtualComponent component = ComponentCacheManager.instance().getComponent(project);
> 			if(component != null) {
>@@ -248,11 +249,9 @@
> 			}
> 		}
> 		IVirtualComponent component = new VirtualComponent(project, new Path("/")); //$NON-NLS-1$
>-		if(component != null) {
>-			ComponentCacheManager.instance().setComponentImplFactory(project, null);
>-			ComponentCacheManager.instance().setComponent(project, component);
>-			registerListener(project);
>-		}
>+		ComponentCacheManager.instance().setComponentImplFactory(project, null);
>+		ComponentCacheManager.instance().setComponent(project, component);
>+		registerListener(project);
> 
> 		return component;
> 	}

Flags:

ccc: iplog+

Actions: View | Diff

Attachments on bug 511793: 267682 | 267697 | 267761 | 267762 | 267776 | 267792