Author: struberg
Date: Sun Feb 20 08:37:56 2011
New Revision: 1072529
URL: http://svn.apache.org/viewvc?rev=1072529&view=rev
Log:
fix PMD warnings
Modified:
maven/scm/trunk/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/src/main/java/org/apache/maven/scm/provider/git/gitexe/command/update/GitUpdateCommandConsumer.java
Modified:
maven/scm/trunk/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/src/main/java/org/apache/maven/scm/provider/git/gitexe/command/update/GitUpdateCommandConsumer.java
URL:
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/src/main/java/org/apache/maven/scm/provider/git/gitexe/command/update/GitUpdateCommandConsumer.java?rev=1072529&r1=1072528&r2=1072529&view=diff
==============================================================================
---
maven/scm/trunk/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/src/main/java/org/apache/maven/scm/provider/git/gitexe/command/update/GitUpdateCommandConsumer.java
(original)
+++
maven/scm/trunk/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/src/main/java/org/apache/maven/scm/provider/git/gitexe/command/update/GitUpdateCommandConsumer.java
Sun Feb 20 08:37:56 2011
@@ -61,14 +61,12 @@ public class GitUpdateCommandConsumer
{
return;
}
- if ( !updatingFound )
+ if ( !updatingFound && line.startsWith( "Updating" ) )
{
- if ( line.startsWith( "Updating" ) )
- {
- updatingFound = true;
- return;
- }
+ updatingFound = true;
+ return;
}
+
// skip summary line
//1 files changed, 1 insertions(+), 1 deletions(-)â[m
if ( line.indexOf( "files changed" ) >= 0 )
@@ -107,23 +105,20 @@ public class GitUpdateCommandConsumer
// delete mode 100644 README
// create mode 100644 test.txt
String[] changedFileLine = StringUtils.split( line, " " );
- if ( changedFileLine != null )
+ if ( changedFileLine != null && changedFileLine.length >= 4 )
{
- if ( changedFileLine.length >= 4 )
+ String status = changedFileLine[0];
+ String fileName = changedFileLine[3];
+ ScmFile scmFile = (ScmFile) scmFiles.get( fileName );
+ if ( scmFile != null )
{
- String status = changedFileLine[0];
- String fileName = changedFileLine[3];
- ScmFile scmFile = (ScmFile) scmFiles.get( fileName );
- if ( scmFile != null )
+ if ( StringUtils.equalsIgnoreCase( "delete", status ) )
+ {
+ scmFiles.put( fileName, new ScmFile( fileName,
ScmFileStatus.DELETED ) );
+ }
+ if ( StringUtils.equalsIgnoreCase( "create", status ) )
{
- if ( StringUtils.equalsIgnoreCase( "delete", status ) )
- {
- scmFiles.put( fileName, new ScmFile( fileName,
ScmFileStatus.DELETED ) );
- }
- if ( StringUtils.equalsIgnoreCase( "create", status ) )
- {
- scmFiles.put( fileName, new ScmFile( fileName,
ScmFileStatus.ADDED ) );
- }
+ scmFiles.put( fileName, new ScmFile( fileName,
ScmFileStatus.ADDED ) );
}
}
}