You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2007/11/12 00:32:51 UTC
svn commit: r593992 - in
/maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help:
ActiveProfilesMojo.java DescribeMojo.java
Author: dennisl
Date: Sun Nov 11 15:32:46 2007
New Revision: 593992
URL: http://svn.apache.org/viewvc?rev=593992&view=rev
Log:
o Fix errors reported by Checkstyle.
Modified:
maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/ActiveProfilesMojo.java
maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/DescribeMojo.java
Modified: maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/ActiveProfilesMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/ActiveProfilesMojo.java?rev=593992&r1=593991&r2=593992&view=diff
==============================================================================
--- maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/ActiveProfilesMojo.java (original)
+++ maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/ActiveProfilesMojo.java Sun Nov 11 15:32:46 2007
@@ -101,7 +101,7 @@
{
File dir = output.getParentFile();
- if( !dir.exists() )
+ if ( !dir.exists() )
{
dir.mkdirs();
}
@@ -149,7 +149,7 @@
message.append( "Active Profiles for Project \'" + project.getId() + "\': \n\n" );
- if( profiles == null || profiles.isEmpty() )
+ if ( profiles == null || profiles.isEmpty() )
{
message.append( "There are no active profiles." );
}
@@ -163,7 +163,7 @@
message.append( "\n - " )
.append( profile.getId() )
- .append(" (source: " )
+ .append( " (source: " )
.append( profile.getSource() ).append( ")" );
}
Modified: maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/DescribeMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/DescribeMojo.java?rev=593992&r1=593991&r2=593992&view=diff
==============================================================================
--- maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/DescribeMojo.java (original)
+++ maven/plugins/trunk/maven-help-plugin/src/main/java/org/apache/maven/plugins/help/DescribeMojo.java Sun Nov 11 15:32:46 2007
@@ -323,18 +323,18 @@
}
catch ( ArtifactResolutionException e )
{
- throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId +
- "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
+ throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId
+ + "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
}
catch ( PluginManagerException e )
{
- throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId +
- "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
+ throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId
+ + "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
}
catch ( PluginVersionResolutionException e )
{
- throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId +
- "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
+ throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId
+ + "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
}
catch ( ArtifactNotFoundException e )
{
@@ -342,13 +342,13 @@
}
catch ( InvalidVersionSpecificationException e )
{
- throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId +
- "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
+ throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId
+ + "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
}
catch ( InvalidPluginException e )
{
- throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId +
- "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
+ throw new MojoExecutionException( "Error retrieving plugin descriptor for:\n\ngroupId: \'" + groupId
+ + "\'\nartifactId: \'" + artifactId + "\'\nversion: \'" + version + "\'\n\n", e );
}
catch ( PluginNotFoundException e )
{
@@ -364,8 +364,8 @@
if ( descriptor == null )
{
- throw new MojoFailureException(
- "Plugin could not be found. If you believe it is correct, check your pluginGroups setting, and run with -U to update the remote configuration" );
+ throw new MojoFailureException( "Plugin could not be found. If you believe it is correct,"
+ + " check your pluginGroups setting, and run with -U to update the remote configuration" );
}
return descriptor;
@@ -389,28 +389,20 @@
switch ( pluginParts.length )
{
case ( 1 ):
- {
pi.prefix = pluginParts[0];
break;
- }
case ( 2 ):
- {
pi.groupId = pluginParts[0];
pi.artifactId = pluginParts[1];
break;
- }
case ( 3 ):
- {
pi.groupId = pluginParts[0];
pi.artifactId = pluginParts[1];
pi.version = pluginParts[2];
break;
- }
default:
- {
throw new MojoFailureException(
"plugin parameter must be a plugin prefix, or conform to: 'groupId:artifactId[:version]." );
- }
}
}
else
@@ -461,7 +453,7 @@
{
MojoDescriptor md = (MojoDescriptor) it.next();
- if( full )
+ if ( full )
{
buffer.append( line );
buffer.append( "\nGoal: \'" ).append( md.getGoal() ).append( '\'' );
@@ -554,7 +546,7 @@
{
buffer.append( "\nDescription:\n" );
prettyAppend( formatDescription( md.getDescription() ), buffer );
- if( fullDescription )
+ if ( fullDescription )
{
buffer.append( "\n" );
}