[ 
https://jira.codehaus.org/browse/MNG-4597?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Paul Benedict updated MNG-4597:
-------------------------------

    Fix Version/s:     (was: Issues to be reviewed for 3.x)

> FindBugs analysis effects
> -------------------------
>
>                 Key: MNG-4597
>                 URL: https://jira.codehaus.org/browse/MNG-4597
>             Project: Maven
>          Issue Type: Bug
>    Affects Versions: 2.2.1
>         Environment: n/a
>            Reporter: M Pazurkiewicz
>            Priority: Minor
>
> I've run a FindBugs analysis for the 2.2.1 sources.
> I found a number of minor issues:
>  * lots of unread fields
>  * misleading class naming 
> (org.apache.maven.artifact.repository.metadata.RepositoryMetadataInstallationException
>  
>    and 
> org.apache.maven.artifact.repository.metadata.RepositoryMetadataDeploymentException
>  do not inherit from Exception  class)
>  * using Number constructors instead of static valueOf (the latter is 
> significantly more efficient)
> And as far as more serious issues:
> In src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequest.java 
> I found a curious self-assignment:
>     public void setSession( MavenSession session )
>     {
>         session = session;
>     }



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)

Reply via email to