[jboss-cvs] JBossAS SVN: r94639 - in projects/jboss-osgi/projects/bundles/hotdeploy/trunk: src/main/java/org/jboss/osgi/hotdeploy/internal and 1 other directory.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Sat Oct 10 11:09:32 EDT 2009


Author: thomas.diesler at jboss.com
Date: 2009-10-10 11:09:32 -0400 (Sat, 10 Oct 2009)
New Revision: 94639

Modified:
   projects/jboss-osgi/projects/bundles/hotdeploy/trunk/pom.xml
   projects/jboss-osgi/projects/bundles/hotdeploy/trunk/src/main/java/org/jboss/osgi/hotdeploy/internal/DeploymentScannerImpl.java
Log:
Optimize call to deployer delegate for empty set

Modified: projects/jboss-osgi/projects/bundles/hotdeploy/trunk/pom.xml
===================================================================
--- projects/jboss-osgi/projects/bundles/hotdeploy/trunk/pom.xml	2009-10-10 14:15:46 UTC (rev 94638)
+++ projects/jboss-osgi/projects/bundles/hotdeploy/trunk/pom.xml	2009-10-10 15:09:32 UTC (rev 94639)
@@ -27,12 +27,12 @@
   <parent>
     <groupId>org.jboss.osgi</groupId>
     <artifactId>jboss-osgi-parent</artifactId>
-    <version>1.0.2</version>
+    <version>1.0.3-SNAPSHOT</version>
   </parent>
   
   <!-- Properties -->
   <properties>
-    <version.jboss.osgi.common>1.0.1</version.jboss.osgi.common>
+    <version.jboss.osgi.common>1.0.2-SNAPSHOT</version.jboss.osgi.common>
     <version.osgi>r4v42</version.osgi>
   </properties>
   

Modified: projects/jboss-osgi/projects/bundles/hotdeploy/trunk/src/main/java/org/jboss/osgi/hotdeploy/internal/DeploymentScannerImpl.java
===================================================================
--- projects/jboss-osgi/projects/bundles/hotdeploy/trunk/src/main/java/org/jboss/osgi/hotdeploy/internal/DeploymentScannerImpl.java	2009-10-10 14:15:46 UTC (rev 94638)
+++ projects/jboss-osgi/projects/bundles/hotdeploy/trunk/src/main/java/org/jboss/osgi/hotdeploy/internal/DeploymentScannerImpl.java	2009-10-10 15:09:32 UTC (rev 94639)
@@ -230,15 +230,18 @@
          logBundleDeployments("NEW diff", diff);
 
       // Deploy the bundles through the DeployerService
-      try
+      if (diff.size() > 0)
       {
-         BundleDeployment[] depArr = diff.toArray(new BundleDeployment[diff.size()]);
-         deployer.deploy(depArr);
+         try
+         {
+            BundleDeployment[] depArr = diff.toArray(new BundleDeployment[diff.size()]);
+            deployer.deploy(depArr);
+         }
+         catch (Exception ex)
+         {
+            log.log(LogService.LOG_ERROR, "Cannot deploy bundles", ex);
+         }
       }
-      catch (Exception ex)
-      {
-         log.log(LogService.LOG_ERROR, "Cannot deploy bundles", ex);
-      }
       
       return diff.size();
    }




More information about the jboss-cvs-commits mailing list