You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by me...@apache.org on 2007/03/04 23:53:56 UTC

svn commit: r514491 - /incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/java/JavaPhysicalComponentBuilder.java

Author: meerajk
Date: Sun Mar  4 14:53:55 2007
New Revision: 514491

URL: http://svn.apache.org/viewvc?view=rev&rev=514491
Log:
Extracted method.

Modified:
    incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/java/JavaPhysicalComponentBuilder.java

Modified: incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/java/JavaPhysicalComponentBuilder.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/java/JavaPhysicalComponentBuilder.java?view=diff&rev=514491&r1=514490&r2=514491
==============================================================================
--- incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/java/JavaPhysicalComponentBuilder.java (original)
+++ incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/implementation/java/JavaPhysicalComponentBuilder.java Sun Mar  4 14:53:55 2007
@@ -57,18 +57,9 @@
         
         JavaComponent component = new JavaComponent();
         
-        Scope scope = componentDefinition.getScope();
-        URI classLoaderId = componentDefinition.getClassLoaderId();
-        ScopeContainer scopeContainer = scopeRegistry.getScopeContainer(scope);
-        component.setScopeContainer(scopeContainer);
+        setScopeContainer(componentDefinition, component);
 
-        // TODO use MPCL to load IF class
-        byte[] instanceFactoryByteCode = componentDefinition.getInstanceFactoryByteCode();
-        ClassLoader appCl = classLoaderRegistry.getClassLoader(classLoaderId);
-        ClassLoader systemCl = getClass().getClassLoader();        
-        ClassLoader mpcl = null;
-        Class<InstanceFactory<?>> instanceFactoryClass = null;        
-        component.setInstanceFactoryClass(instanceFactoryClass);
+        setInstanceFactoryClass(componentDefinition, component);
         
         return component;
     }
@@ -89,6 +80,29 @@
     @Reference
     public void setScopeRegistry(ScopeRegistry scopeRegistry) {
         this.scopeRegistry = scopeRegistry;
+    }
+
+    /*
+     * Sets the instance factory class.
+     */
+    private void setInstanceFactoryClass(JavaPhysicalComponentDefinition componentDefinition, JavaComponent component) {
+        // TODO use MPCL to load IF class
+        URI classLoaderId = componentDefinition.getClassLoaderId();
+        byte[] instanceFactoryByteCode = componentDefinition.getInstanceFactoryByteCode();
+        ClassLoader appCl = classLoaderRegistry.getClassLoader(classLoaderId);
+        ClassLoader systemCl = getClass().getClassLoader();        
+        ClassLoader mpcl = null;
+        Class<InstanceFactory<?>> instanceFactoryClass = null;        
+        component.setInstanceFactoryClass(instanceFactoryClass);
+    }
+
+    /*
+     * Set the scope container.
+     */
+    private void setScopeContainer(JavaPhysicalComponentDefinition componentDefinition, JavaComponent component) {
+        Scope scope = componentDefinition.getScope();
+        ScopeContainer scopeContainer = scopeRegistry.getScopeContainer(scope);
+        component.setScopeContainer(scopeContainer);
     }
 
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org