You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Karl Heinz Marbaise (Jira)" <ji...@apache.org> on 2020/04/04 16:50:00 UTC
[jira] [Closed] (MSHARED-864) Refactor and simplify code
[ https://issues.apache.org/jira/browse/MSHARED-864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Karl Heinz Marbaise closed MSHARED-864.
---------------------------------------
Resolution: Done
> Refactor and simplify code
> --------------------------
>
> Key: MSHARED-864
> URL: https://issues.apache.org/jira/browse/MSHARED-864
> Project: Maven Shared Components
> Issue Type: Improvement
> Components: maven-artifact-transfer
> Affects Versions: maven-artifact-transfer-0.13.0
> Reporter: Karl Heinz Marbaise
> Assignee: Karl Heinz Marbaise
> Priority: Minor
> Fix For: maven-artifact-transfer-0.13.0
>
>
> * In the classes {{Maven30DependencyResolver.java}}, {{Maven31DependencyResolver.java}} code can be cleaned up.
> ** The code calls {{setDependencies( List<Dependency> dependencies )}} of {{CollectRequest}} which allows to give {{null}}
> * Refactoring in {{Invoker.java}}
> ** Remove not used methods.
> ** Inlining only locally called methods.
> ** Move the SuppressWarnings to the invoke methods only; Not in user code (outside of Invoker classes)
> ** Make code comment requirements to code requirements (number of elements in {{argClasses}} ands {{args}})
> ** Using generics removes the need of type casts in user code (outside Invoker classes)
> * Simplify catching exceptions (
> * {{MavenArtifactDeployer}} interface is not needed to be public.
> * {{MavenDependencyCollector}} interface is not needed to be public.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)