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

Elliotte Rusty Harold commented on MNG-8749:
--------------------------------------------

There's also ModelPathTranslator which has the same method with presumably a 
different algorithm, so if we're creating new classes for every algorithm, we 
really don't need an interface.

Honestly, this all seems vastly over complex for what amounts to static utility 
methods. ModelPathTranslator is used in exactly one place. It could easily and 
probably should be a private method in that one class, at least until we need 
it in a second place. 

> Hide PathTranslator from API
> ----------------------------
>
>                 Key: MNG-8749
>                 URL: https://issues.apache.org/jira/browse/MNG-8749
>             Project: Maven
>          Issue Type: Improvement
>            Reporter: Elliotte Rusty Harold
>            Priority: Blocker
>
> org.apache.maven.api.services.model.PathTranslator has exactly one 
> implementation that is only used in the impl package. Move it there, make it 
> non-public, and combine it with DefaultPathTranslator.
> It can be added to the org.apache.maven.api.services.model package later if a 
> need is uncovered, but right now no such need exists. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to