You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Paul Benedict (JIRA)" <ji...@codehaus.org> on 2014/07/02 15:45:18 UTC
[jira] (MNG-4597) FindBugs analysis effects
[ 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)