RFR [8014066] Mistake in documentation of ArrayList#removeRange (original) (raw)

Martin Buchholz martinrb at google.com
Fri Mar 14 06:38:05 UTC 2014

On Thu, Mar 13, 2014 at 3:59 PM, Doug Lea <dl at cs.oswego.edu> wrote:

On 03/13/2014 12:34 PM, Martin Buchholz wrote:

I notice there are zero jtreg tests for removeRange. That should be fixed.

I notice there is a removeRange in CopyOnWriteArrayList, but it is package-private instead of "protected", which seems like an oversight. Can Doug remember any history on that? CopyOnWriteArrayList does not extend AbstractList, but its sublist does. The sublist relies on COWAL.removeRange only for clear. So COWAL sublist clearing uses the same idea as AbstractList, and gives it the same name, but it is not the same AbstractList method, so need not be protected.

Ahh OK, I think the party line for users is if they want to remove a range of elements from a list, use list.subList(fromIndex, toIndex).clear (), so there's no advantage in making COWAL.removeRange a public interface.

More information about the core-libs-dev mailing list