[
https://issues.apache.org/jira/browse/LANG-567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12792517#action_12792517
]
Sebb commented on LANG-567:
---------------------------
I did consider checking the types before trying the array copy in which case
the issue would not have arisen.
However it should be cheaper to wait until a problem occurs, and also slightly
safer in case the check is too strict.
I'll update the code shortly.
> ArrayUtils.addAll(T[] array1, T... array2) does not handle mixed types very
> well
> --------------------------------------------------------------------------------
>
> Key: LANG-567
> URL: https://issues.apache.org/jira/browse/LANG-567
> Project: Commons Lang
> Issue Type: Bug
> Components: lang.*
> Reporter: Sebb
> Fix For: 3.0
>
>
> ArrayUtils.addAll(T[] array1, T... array2) does not handle mixed array types
> very well.
> The stack trace for
> Number[] st = ArrayUtils.addAll(new Integer[]{1}, new Long[]{2L});
> starts:
> java.lang.ArrayStoreException
> at java.lang.System.arraycopy(Native Method)
> at org.apache.commons.lang3.ArrayUtils.addAll(ArrayUtils.java:2962)
> which is not all that obvious.
> It would be a lot clearer if the method threw an IlegalArgumentException or
> similar.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
