You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Benedikt Ritter (JIRA)" <ji...@apache.org> on 2015/04/06 19:29:26 UTC
[jira] [Closed] (LANG-1073) Read wrong component type of array in
add in ArrayUtils
[ https://issues.apache.org/jira/browse/LANG-1073?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Benedikt Ritter closed LANG-1073.
---------------------------------
> 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: 3.4
>
>
> {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)