8220614: (bf) Buffer absolute slice methods should use Objects.checkFromIndexSize()

Reviewed-by: alanb, igerasim, rriggs
This commit is contained in:
Brian Burkhalter 2019-03-14 09:43:18 -07:00
parent 818df25a3b
commit e9cb99cbe4
4 changed files with 4 additions and 8 deletions

View File

@ -87,8 +87,7 @@ class ByteBufferAs$Type$Buffer$RW$$BO$ // package-private
@Override
public $Type$Buffer slice(int index, int length) {
Objects.checkIndex(index, limit() + 1);
Objects.checkIndex(length, limit() - index + 1);
Objects.checkFromIndexSize(index, length, limit());
return new ByteBufferAs$Type$Buffer$RW$$BO$(bb,
-1,
0,

View File

@ -220,8 +220,7 @@ class Direct$Type$Buffer$RW$$BO$
@Override
public $Type$Buffer slice(int index, int length) {
Objects.checkIndex(index, limit() + 1);
Objects.checkIndex(length, limit() - index + 1);
Objects.checkFromIndexSize(index, length, limit());
return new Direct$Type$Buffer$RW$$BO$(this,
-1,
0,

View File

@ -115,8 +115,7 @@ class Heap$Type$Buffer$RW$
@Override
public $Type$Buffer slice(int index, int length) {
Objects.checkIndex(index, limit() + 1);
Objects.checkIndex(length, limit() - index + 1);
Objects.checkFromIndexSize(index, length, limit());
return new Heap$Type$Buffer$RW$(hb,
-1,
0,

View File

@ -54,8 +54,7 @@ class StringCharBuffer // package-private
@Override
public CharBuffer slice(int index, int length) {
Objects.checkIndex(index, limit() + 1);
Objects.checkIndex(length, limit() - index + 1);
Objects.checkFromIndexSize(index, length, limit());
return new StringCharBuffer(str,
-1,
0,