Thanks for the review~
Valerie
On 7/10/2018 11:16 AM, Anthony Scarpino wrote:
Ok, I'm fine with what you have here.
thanks
Tony
On 07/09/2018 05:42 PM, Valerie Peng wrote:
Hi Tony,
The purpose of the if-block is to ensure that
ArrayIndexOutOfBoundsException is thrown instead of
IndexOutOfBoundsException. Objects.checkFromIndexSize(...) API is
specified to throw IndexOutOfBoundsException which is more general
and can be thrown when an index of some sort (such as to an array, to
a string, or to a vector) is out of range.
ArrayIndexOutOfBoundsException is a subclass of
IndexOutOfBoundsException and is for array index out of bounds.
Thanks,
Valerie
On 7/9/2018 3:14 PM, Anthony Scarpino wrote:
On 07/03/2018 02:03 PM, Valerie Peng wrote:
Hi Tony,
Would you have time to review this? Instead of doing the bounds
check per block, the changes move the bounds check up one level.
Bug: https://bugs.openjdk.java.net/browse/JDK-8179098
Webrev: http://cr.openjdk.java.net/~valeriep/8179098/webrev.00/
Thanks,
Valerie
ArrayUtil.java: Line 48
- It seems like your not throwing any
ArrayIndexOutOfBoundsExceptions, is that intentional? The method in
the try-catch seems to only throw IndexOutOfBoundsException, whole
if() seems confusing to me.
thanks
Tony
