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

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

That makes some sense but it does need a comment in the code since it by no 
means obvious that this was the intent.

> Perhaps remove BaseRequest.unmodifiable
> ---------------------------------------
>
>                 Key: MNG-8484
>                 URL: https://issues.apache.org/jira/browse/MNG-8484
>             Project: Maven
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 4.0.0-rc-2
>            Reporter: Elliotte Rusty Harold
>            Priority: Blocker
>             Fix For: 4.0.0-rc-3
>
>
> Consider that it has the unexpected effect of making all empty lists 
> identical and equal, regardless of type, which might not be what we want. We 
> could simply use Collections.unmodifiableCollection instead.
> This method also converts all collections to lists. That might be 
> intentional, but if so it should be documented in a comment. 



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

Reply via email to