You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Maarten Coene (JIRA)" <ji...@apache.org> on 2010/01/09 23:33:54 UTC
[jira] Reopened: (LANG-470) Add containsAll methods to ArrayUtils
[ https://issues.apache.org/jira/browse/LANG-470?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Maarten Coene reopened LANG-470:
--------------------------------
Sorry but I don't agree on this one.
If the code you provided is the reason to not implement his, you should remove a lot of other utility methods as well from ArrayUtils and other *Utils classes.
In addition, your code will throw a NPE if one of the passed in arrays are null. The "one-line code" becomes a bit more complex to handle this situation properly.
Maarten
> Add containsAll methods to ArrayUtils
> -------------------------------------
>
> Key: LANG-470
> URL: https://issues.apache.org/jira/browse/LANG-470
> Project: Commons Lang
> Issue Type: New Feature
> Components: lang.*
> Affects Versions: 2.4
> Reporter: Maarten Coene
> Fix For: 3.0
>
> Attachments: ArrayUtilsContainsAll.pat
>
>
> The ArrayUtils class contains several "contains" methods.
> It could be usefull to add the corresponding containsAll(Object[] array1, Object[] array2) methods (and corresponding for primitive types).
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.