On Tue, 27 Jul 2021 02:46:53 GMT, Yi Yang <yy...@openjdk.org> wrote: >> I meant >> >> if (imageIndex < minIndex) >> throw new IndexOutOfBoundsException(); >> >> can't it be replaced with Objects.checkFromToIndex(minIndex, imageIndex, .. > > Hi, sorry for the late response. checkFromToIndex has many constraints, this > code may not satisfy those constraints, so I don't think we should do that in > this PR.(Maybe we can file a new PR to do replacement carefully)
OK...The subject line of this PR states that checkFromToIndex will also be used but I see only checkIndex so I intervened. Anyways, the code in question is in test so not much of an issue. ------------- PR: https://git.openjdk.java.net/jdk/pull/4718