[jboss-cvs] JBossAS SVN: r91720 - in projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation: virtual/api and 2 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Tue Jul 28 11:56:40 EDT 2009


Author: ALRubinger
Date: 2009-07-28 11:56:39 -0400 (Tue, 28 Jul 2009)
New Revision: 91720

Modified:
   projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/deployable/api/DeployableFactory.java
   projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/api/VirtualArchive.java
   projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/impl/vfs/VirtualVfsArchiveImpl.java
   projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/spi/vfs/VirtualVfsArchive.java
Log:
[EMB-32] Fix up some comments, correct the FQNs used in the factories to reflect recent refactoring

Modified: projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/deployable/api/DeployableFactory.java
===================================================================
--- projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/deployable/api/DeployableFactory.java	2009-07-28 15:49:59 UTC (rev 91719)
+++ projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/deployable/api/DeployableFactory.java	2009-07-28 15:56:39 UTC (rev 91720)
@@ -53,14 +53,9 @@
    /**
     * FQN of implementation Class used in creating new Deployables 
     */
-   private static final String CLASS_NAME_VFS_VDF_DEPLOYABLE = "org.jboss.embedded.core.incubation.deployable.impl.vdf.VfsVdfDeployable";
+   private static final String CLASS_NAME_VFS_VDF_DEPLOYABLE = "org.jboss.embedded.core.incubation.deployable.impl.vdf.VfsVdfDeployableImpl";
 
    /**
-    * FQN of type of parameter to implementation class constructor
-    */
-   private static final String CLASS_NAME_CTOR_PARAMETER = "org.jboss.embedded.core.incubation.virtual.spi.vfs.VirtualVfsArchive";
-
-   /**
     * FQN of the archive type actually supported by {@link DeployableFactory#createDeployable(VirtualArchive)}
     */
    private static final String CLASS_NAME_ARCHIVE_TYPE = "org.jboss.embedded.core.incubation.virtual.spi.vfs.VirtualVfsArchive";
@@ -173,7 +168,7 @@
          final Class<?> implClass = getClass(implClassName);
 
          // Load the ctor param class
-         final String paramClassName = CLASS_NAME_CTOR_PARAMETER;
+         final String paramClassName = CLASS_NAME_ARCHIVE_TYPE;
          final Class<?> paramClass = getClass(paramClassName);
 
          // Get and set the ctor

Modified: projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/api/VirtualArchive.java
===================================================================
--- projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/api/VirtualArchive.java	2009-07-28 15:49:59 UTC (rev 91719)
+++ projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/api/VirtualArchive.java	2009-07-28 15:56:39 UTC (rev 91720)
@@ -82,7 +82,9 @@
    VirtualArchive addResource(String name, ClassLoader cl) throws IllegalArgumentException;
 
    /**
-    * Adds the resource located at the specified URL to the archive
+    * Adds the resource located at the specified URL to the archive.  The
+    * location within the archive will be equal to the path portion of the 
+    * specified URL.
     * 
     * @param location
     * @return

Modified: projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/impl/vfs/VirtualVfsArchiveImpl.java
===================================================================
--- projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/impl/vfs/VirtualVfsArchiveImpl.java	2009-07-28 15:49:59 UTC (rev 91719)
+++ projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/impl/vfs/VirtualVfsArchiveImpl.java	2009-07-28 15:56:39 UTC (rev 91720)
@@ -36,7 +36,7 @@
 /**
  * VirtualVfsArchiveImpl
  * 
- * Concrete implementation of a VFS-backed virtual deployment
+ * Concrete implementation of a VFS-backed virtual archive
  *
  * @author <a href="mailto:andrew.rubinger at jboss.org">ALR</a>
  * @version $Revision: $

Modified: projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/spi/vfs/VirtualVfsArchive.java
===================================================================
--- projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/spi/vfs/VirtualVfsArchive.java	2009-07-28 15:49:59 UTC (rev 91719)
+++ projects/embedded/trunk/core/src/main/java/org/jboss/embedded/core/incubation/virtual/spi/vfs/VirtualVfsArchive.java	2009-07-28 15:56:39 UTC (rev 91720)
@@ -29,7 +29,9 @@
  * 
  * Represents a single in-memory collection
  * of resources which may
- * be constructed declaratively / programmatically.
+ * be constructed declaratively / programmatically.  Adds
+ * the constraint to providers that this is backed by the Virtual
+ * File System.
  * 
  * @author <a href="mailto:andrew.rubinger at jboss.org">ALR</a>
  * @version $Revision: $




More information about the jboss-cvs-commits mailing list