[jira] [Resolved] (LANG-1073) Read wrong component type of array in add in ArrayUtils

["Bruno P. Kinoshita (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/LANG-1073?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bruno P. Kinoshita resolved LANG-1073.
--------------------------------------
    Resolution: Fixed
      Assignee: Bruno P. Kinoshita

Fixed in r1645483.

> Read wrong component type of array in add in ArrayUtils
> -------------------------------------------------------
>
>                 Key: LANG-1073
>                 URL: https://issues.apache.org/jira/browse/LANG-1073
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.*
>    Affects Versions: 3.3.2
>            Reporter: haiyang li
>            Assignee: Bruno P. Kinoshita
>            Priority: Minor
>             Fix For: Discussion
>
>
> {code:title=org.apache.commons.lang3.ArrayUtils.java|borderStyle=solid}
>     public static <T> T[] add(final T[] array, final T element) {
>         Class<?> type;
>         if (array != null){
>             type = array.getClass();
>         } else if (element != null) {
>             type = element.getClass();
>         } else {
>             throw new IllegalArgumentException("Arguments cannot both be null");
>         }
>         .......
>     }
> {code}
> I think it should be:
> {code:title=org.apache.commons.lang3.ArrayUtils.java|borderStyle=solid}
>     public static <T> T[] add(final T[] array, final T element) {
>         Class<?> type;
>         if (array != null){
>             type = array.getClass().getComponentType();
>         } else if (element != null) {
>             type = element.getClass();
>         } else {
>             throw new IllegalArgumentException("Arguments cannot both be null");
>         }
>         .......
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)