Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Date: Sat, 24 Oct 2015 13:42:28 +0000 (UTC)
From: "Sebb (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Message-ID: <JIRA.12907645.1445694132000.50043.1445694147998@Atlassian.JIRA>
In-Reply-To: <JIRA.12907645.1445694132000@Atlassian.JIRA>
References: <JIRA.12907645.1445694132000@Atlassian.JIRA>
 <JIRA.12907645.1445694132265@arcas>
Subject: [jira] [Created] (LANG-1178) ArrayUtils.removeAll(Object array,
 int... indices) should do the clone, not its callers
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Sebb created LANG-1178:
--------------------------

             Summary: ArrayUtils.removeAll(Object array, int... indices) should do the clone, not its callers
                 Key: LANG-1178
                 URL: https://issues.apache.org/jira/browse/LANG-1178
             Project: Commons Lang
          Issue Type: Bug
            Reporter: Sebb


The method ArrayUtils.removeAll(Object array, int... indices) currently sorts the input indices array. Therefore the array needs to be cloned; this is currently done by the callers.

However the sort is an implementation detail of the method, so should be done by the method itself, not by the callers, which is fragile (easy to overlook when creating a new method) and unnecessary.

This would also allow the method to be more easily changed to a different implementation that does not need to sort the array (e.g. using BitSet)


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