continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From c...@apache.org
Subject svn commit: r933472 - /continuum/trunk/continuum-core/src/main/java/org/apache/continuum/builder/distributed/manager/DefaultDistributedBuildManager.java
Date Tue, 13 Apr 2010 03:08:26 GMT
Author: ctan
Date: Tue Apr 13 03:08:26 2010
New Revision: 933472

URL: http://svn.apache.org/viewvc?rev=933472&view=rev
Log:
revert previous changes. should check if null to 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=933472&r1=933471&r2=933472&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 03:08:26 2010
@@ -123,7 +123,7 @@ public class DefaultDistributedBuildMana
     {
         List<BuildAgentConfiguration> agents = configurationService.getBuildAgents();
 
-        if ( !agents.isEmpty() )
+        if ( agents != null )
         {
             synchronized( overallDistributedBuildQueues )
             {
@@ -173,7 +173,7 @@ public class DefaultDistributedBuildMana
     {
         List<BuildAgentConfiguration> agents = configurationService.getBuildAgents();
 
-        if ( agents.isEmpty() )
+        if ( agents == null )
         {
             return;
         }



Mime
View raw message