You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by br...@apache.org on 2005/08/12 11:18:18 UTC
svn commit: r232245 -
/maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/scm/DefaultContinuumScm.java
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 )
{