You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2019/02/17 15:06:00 UTC

[jira] [Commented] (MSHARED-805) Code Improvements

    [ https://issues.apache.org/jira/browse/MSHARED-805?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16770435#comment-16770435 ] 

Hudson commented on MSHARED-805:
--------------------------------

Build succeeded in Jenkins: Maven TLP » maven-artifact-transfer » master #63

See https://builds.apache.org/job/maven-box/job/maven-artifact-transfer/job/master/63/

> Code Improvements
> -----------------
>
>                 Key: MSHARED-805
>                 URL: https://issues.apache.org/jira/browse/MSHARED-805
>             Project: Maven Shared Components
>          Issue Type: Improvement
>          Components: maven-artifact-transfer
>    Affects Versions: maven-artifact-transfer-0.11.0
>            Reporter: Karl Heinz Marbaise
>            Assignee: Karl Heinz Marbaise
>            Priority: Minor
>             Fix For: maven-artifact-transfer-0.11.0
>
>
> * No need for {{protected}} neither for {{static}} for {{isMaven31()}} and {{canFindCoreClass(..)}} in {{DefaultArtifactInstaller.java}}, {{DefaultArtifactResolver.java}}, {{DefaultDependencyCollector.java}}, {{DefaultDependencyResolver.java}}, {{DefaultRepositoryManager.java}}
> * Cleaned up code which means removed commented out code.
> * Cleaned up javadoc's.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)