mirror of https://github.com/apache/maven.git
fixed errors reported by Checkstyle
git-svn-id: https://svn.apache.org/repos/asf/maven/maven-3/trunk@813628 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
05b59c8935
commit
d64e8c5d70
|
@ -43,7 +43,7 @@ public class MojoDescriptor
|
|||
implements Cloneable
|
||||
{
|
||||
/** The Plexus component type */
|
||||
public static String MAVEN_PLUGIN = "maven-plugin";
|
||||
public static final String MAVEN_PLUGIN = "maven-plugin";
|
||||
|
||||
/** "once-per-session" execution strategy */
|
||||
public static final String SINGLE_PASS_EXEC_STRATEGY = "once-per-session";
|
||||
|
@ -57,7 +57,7 @@ public class MojoDescriptor
|
|||
|
||||
private List<Parameter> parameters;
|
||||
|
||||
private Map<String,Parameter> parameterMap;
|
||||
private Map<String, Parameter> parameterMap;
|
||||
|
||||
/** By default, the execution strategy is "once-per-session" */
|
||||
private String executionStrategy = SINGLE_PASS_EXEC_STRATEGY;
|
||||
|
@ -192,9 +192,9 @@ public class MojoDescriptor
|
|||
{
|
||||
if ( parameters != null && parameters.contains( parameter ) )
|
||||
{
|
||||
throw new DuplicateParameterException( parameter.getName() +
|
||||
" has been declared multiple times in mojo with goal: " + getGoal() + " (implementation: " +
|
||||
getImplementation() + ")" );
|
||||
throw new DuplicateParameterException( parameter.getName()
|
||||
+ " has been declared multiple times in mojo with goal: " + getGoal() + " (implementation: "
|
||||
+ getImplementation() + ")" );
|
||||
}
|
||||
|
||||
if ( parameters == null )
|
||||
|
@ -208,11 +208,11 @@ public class MojoDescriptor
|
|||
/**
|
||||
* @return the list parameters as a Map
|
||||
*/
|
||||
public Map<String,Parameter> getParameterMap()
|
||||
public Map<String, Parameter> getParameterMap()
|
||||
{
|
||||
if ( parameterMap == null )
|
||||
{
|
||||
parameterMap = new HashMap<String,Parameter>();
|
||||
parameterMap = new HashMap<String, Parameter>();
|
||||
|
||||
if ( parameters != null )
|
||||
{
|
||||
|
@ -262,7 +262,7 @@ public class MojoDescriptor
|
|||
* Gets the scope of (transitive) dependencies that should be collected. Dependency collection refers to the process
|
||||
* of calculating the complete dependency tree in terms of artifact coordinates. In contrast to dependency
|
||||
* resolution, this does not include the download of the files for the dependency artifacts.
|
||||
*
|
||||
*
|
||||
* @return The scope of (transitive) dependencies that should be collected or {@code null} if none.
|
||||
*/
|
||||
public String getDependencyCollectionRequired()
|
||||
|
|
|
@ -232,7 +232,7 @@ public class PluginDescriptor
|
|||
/**
|
||||
* Gets the artifacts that make up the plugin's class realm, excluding artifacts shadowed by the Maven core realm
|
||||
* like {@code maven-project}.
|
||||
*
|
||||
*
|
||||
* @return The plugin artifacts, never {@code null}.
|
||||
*/
|
||||
public List<Artifact> getArtifacts()
|
||||
|
@ -284,11 +284,11 @@ public class PluginDescriptor
|
|||
// changes above not being propogated to the map
|
||||
|
||||
MojoDescriptor mojoDescriptor = null;
|
||||
|
||||
|
||||
for ( Iterator<?> i = getMojos().iterator(); i.hasNext() && mojoDescriptor == null; )
|
||||
{
|
||||
MojoDescriptor desc = (MojoDescriptor) i.next();
|
||||
|
||||
|
||||
if ( goal.equals( desc.getGoal() ) )
|
||||
{
|
||||
mojoDescriptor = desc;
|
||||
|
@ -342,7 +342,7 @@ public class PluginDescriptor
|
|||
{
|
||||
this.plugin = plugin;
|
||||
}
|
||||
|
||||
|
||||
public Plugin getPlugin()
|
||||
{
|
||||
return plugin;
|
||||
|
|
Loading…
Reference in New Issue