continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From br...@apache.org
Subject svn commit: r232245 - /maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/scm/DefaultContinuumScm.java
Date Fri, 12 Aug 2005 09:18:18 GMT
Author: brett
Date: Fri Aug 12 02:18:13 2005
New Revision: 232245

URL: http://svn.apache.org/viewcvs?rev=232245&view=rev
Log:
don't create a change set without contents

Modified:
    maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/scm/DefaultContinuumScm.java

Modified: maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/scm/DefaultContinuumScm.java
URL: http://svn.apache.org/viewcvs/maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/scm/DefaultContinuumScm.java?rev=232245&r1=232244&r2=232245&view=diff
==============================================================================
--- maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/scm/DefaultContinuumScm.java
(original)
+++ maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/scm/DefaultContinuumScm.java
Fri Aug 12 02:18:13 2005
@@ -228,10 +228,12 @@
 
     private static ChangeSet convertScmFileSetToChangeSet( List files )
     {
-        ChangeSet changeSet = new ChangeSet();
+        ChangeSet changeSet = null;
 
         if ( files != null && !files.isEmpty() )
         {
+            changeSet = new ChangeSet();
+
             // TODO: author, etc.
             for ( Iterator it = files.iterator(); it.hasNext(); )
             {
@@ -261,7 +263,10 @@
 
         // TODO: is this valid?
         ChangeSet changeSet = convertScmFileSetToChangeSet( scmResult.getUpdatedFiles() );
-        result.addChange( changeSet );
+        if ( changeSet != null )
+        {
+            result.addChange( changeSet );
+        }
 
         if ( scmResult.getChanges() != null )
         {



Mime
View raw message