[jira] [Closed] (LANG-660) Add methods to insert arrays into arrays at an index

["Duncan Jones (JIRA)" <ji...@apache.org>]

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

Duncan Jones closed LANG-660.
-----------------------------
       Resolution: Implemented
    Fix Version/s:     (was: Review Patch)
                   3.6

Thanks to everyone for their input into this. In the end, I wrote fresh implementations of these methods as I felt happier using {{arrayCopy}} rather than some of the object-based methods in the supplied patches.

I've also deprecated the {{add(array, index, element)}} methods, which are now superseded by the new methods.

In order to keep the behaviour of these new methods consistent with one another, while avoiding creating an unsafe use of varargs, I've opted to change the handling of {{null}} array inputs. I've referred to this in the deprecation comments.

Thoughts on this welcome.

Repository: commons-lang
Updated Branches:
 refs/heads/master 0136218fa -> 795fe5d86


Added new vararg methods to insert values into an array at an index.

This implements LANG-660. This commit also deprecates the add() methods
that inserted a single value into an array at an index.

Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/795fe5d8
Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/795fe5d8
Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/795fe5d8

> Add methods to insert arrays into arrays at an index
> ----------------------------------------------------
>
>                 Key: LANG-660
>                 URL: https://issues.apache.org/jira/browse/LANG-660
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.*
>    Affects Versions: 2.5
>            Reporter: Aaron Digulla
>            Assignee: Duncan Jones
>            Priority: Minor
>             Fix For: 3.6
>
>         Attachments: ArrayUtils.txt, ArrayUtilsAddTest.txt, ArrayUtilsAddTest_addAll.patch, ArrayUtils_addAll.patch
>
>
> Please add methods with this signature: ArrayUtils.addAll(int[] target, int index, int[] source) (i.e. insert an array into an array at a certain position).



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