RFR [8014066] Mistake in documentation of ArrayList#removeRange

Ivan Gerasimov ivan.gerasimov at oracle.com
Mon Mar 17 06:03:37 UTC 2014


On 17.03.2014 2:52, Ulf Zibis wrote:
> Am 16.03.2014 23:37, schrieb Ivan Gerasimov:
>> Here is yet another iteration of the fix:
>> http://cr.openjdk.java.net/~igerasim/8014066/3/webrev/
>>
>> 2)
>> Kept the check for 'fromIndex > toIndex' in removeRange().
>> While I understand that this should not add anything significant to 
>> the current code, as currently removeRange() is always called with 
>> valid arguments.
>> However, if it is stated in the spec that in case of 'fromIndex > 
>> toIndex' an exception is thrown, I believe it should be thrown, 
>> otherwise why it's stated?
>
> Isn't this a perfect situation to use an assert statement? Needless to 
> say, then the spec must be adapted.
>

Yeah, it might be more fair to write that it's assumed that the passed 
arguments are valid and use asserts as you suggest.
Though I think that the current trend is to do as little modifications 
as possible.

Sincerely yours,
Ivan

> -Ulf
>
>
>




More information about the core-libs-dev mailing list