[jboss-cvs] JBossAS SVN: r59436 - in projects/osgi/trunk/deployment/src/main/org/jboss/osgi: deployers metadata/plugins

[jboss-cvs-commits at lists.jboss.org]

Author: alesj
Date: 2007-01-09 09:06:36 -0500 (Tue, 09 Jan 2007)
New Revision: 59436

Modified:
   projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleActivatorDeployer.java
   projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleClassLoaderDeployer.java
   projects/osgi/trunk/deployment/src/main/org/jboss/osgi/metadata/plugins/AbstractBundleActivatorMetadata.java
Log:
javadocs

Modified: projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleActivatorDeployer.java
===================================================================
--- projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleActivatorDeployer.java	2007-01-09 14:01:31 UTC (rev 59435)
+++ projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleActivatorDeployer.java	2007-01-09 14:06:36 UTC (rev 59436)
@@ -51,7 +51,7 @@
    private GUID guid;
 
    /**
-    * Create a new OSGiPluginDeployer.
+    * Create a new BundleActivatorDeployer
     *
     * @param kernel the kernel
     * @throws IllegalArgumentException for a null kernel
@@ -65,6 +65,12 @@
       setRelativeOrder(REAL_DEPLOYER - 2); // before osgi_plugin and beans, if activators add some additional OSGi info
    }
 
+   /**
+    * Deploy / install all activators as new BeanMD
+    *
+    * @param unit
+    * @throws DeploymentException
+    */
    public void deploy(DeploymentUnit unit) throws DeploymentException
    {
       BundleAdapter bundleAdapter = unit.getAttachment(BundleAdapter.class);
@@ -78,6 +84,13 @@
       }
    }
 
+   /**
+    * Let Controller take care of BundleActivator lifecycle.
+    *
+    * @param bamd
+    * @param bundleAdapter
+    * @throws DeploymentException
+    */
    private void registerBundleActivator(BundleActivatorMetaData bamd, BundleAdapter bundleAdapter) throws DeploymentException
    {
       try

Modified: projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleClassLoaderDeployer.java
===================================================================
--- projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleClassLoaderDeployer.java	2007-01-09 14:01:31 UTC (rev 59435)
+++ projects/osgi/trunk/deployment/src/main/org/jboss/osgi/deployers/BundleClassLoaderDeployer.java	2007-01-09 14:06:36 UTC (rev 59436)
@@ -38,7 +38,7 @@
       BundleAdapter bundleAdapter = context.getTransientAttachments().getAttachment(BundleAdapter.class);
       if (bundleAdapter != null)
       {
-         // TODO
+         // TODO - create CL
          return null;
       }
       return context.getClassLoader();

Modified: projects/osgi/trunk/deployment/src/main/org/jboss/osgi/metadata/plugins/AbstractBundleActivatorMetadata.java
===================================================================
--- projects/osgi/trunk/deployment/src/main/org/jboss/osgi/metadata/plugins/AbstractBundleActivatorMetadata.java	2007-01-09 14:01:31 UTC (rev 59435)
+++ projects/osgi/trunk/deployment/src/main/org/jboss/osgi/metadata/plugins/AbstractBundleActivatorMetadata.java	2007-01-09 14:06:36 UTC (rev 59436)
@@ -24,6 +24,8 @@
 import org.jboss.osgi.metadata.spi.BundleActivatorMetaData;
 
 /**
+ * Plain className holder.
+ * 
  * @author <a href="mailto:ales.justin at jboss.com">Ales Justin</a>
  */
 public class AbstractBundleActivatorMetaData implements BundleActivatorMetaData