You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ct...@apache.org on 2010/04/13 04:48:46 UTC

svn commit: r933469 - /continuum/trunk/continuum-core/src/main/java/org/apache/continuum/builder/distributed/manager/DefaultDistributedBuildManager.java

Author: ctan
Date: Tue Apr 13 02:48:46 2010
New Revision: 933469

URL: http://svn.apache.org/viewvc?rev=933469&view=rev
Log:
prevent NPE when no build agent is configured

Modified:
    continuum/trunk/continuum-core/src/main/java/org/apache/continuum/builder/distributed/manager/DefaultDistributedBuildManager.java

Modified: continuum/trunk/continuum-core/src/main/java/org/apache/continuum/builder/distributed/manager/DefaultDistributedBuildManager.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-core/src/main/java/org/apache/continuum/builder/distributed/manager/DefaultDistributedBuildManager.java?rev=933469&r1=933468&r2=933469&view=diff
==============================================================================
--- continuum/trunk/continuum-core/src/main/java/org/apache/continuum/builder/distributed/manager/DefaultDistributedBuildManager.java (original)
+++ continuum/trunk/continuum-core/src/main/java/org/apache/continuum/builder/distributed/manager/DefaultDistributedBuildManager.java Tue Apr 13 02:48:46 2010
@@ -123,7 +123,7 @@ public class DefaultDistributedBuildMana
     {
         List<BuildAgentConfiguration> agents = configurationService.getBuildAgents();
 
-        if ( agents != null )
+        if ( !agents.isEmpty() )
         {
             synchronized( overallDistributedBuildQueues )
             {
@@ -173,7 +173,7 @@ public class DefaultDistributedBuildMana
     {
         List<BuildAgentConfiguration> agents = configurationService.getBuildAgents();
 
-        if ( agents != null )
+        if ( agents.isEmpty() )
         {
             return;
         }