Re: RFR: 8297561: Redundant index check in String.offsetByCodePoints() [v2]

2022-12-01 Thread Sergey Tsypanov
On Wed, 30 Nov 2022 20:41:47 GMT, Claes Redestad wrote: >> Sergey Tsypanov has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains three additional >>

Re: RFR: 8297561: Redundant index check in String.offsetByCodePoints() [v2]

2022-11-30 Thread Claes Redestad
On Wed, 30 Nov 2022 18:17:47 GMT, Sergey Tsypanov wrote: >> `String.offsetByCodePoints()` delegates to `Character.offsetByCodePoints()` >> which in turn specifies the same exception thrown under the same conditions >> and the implementation does exactly the same checks. This means we can >>

Re: RFR: 8297561: Redundant index check in String.offsetByCodePoints() [v2]

2022-11-30 Thread Roger Riggs
On Wed, 30 Nov 2022 18:17:47 GMT, Sergey Tsypanov wrote: >> `String.offsetByCodePoints()` delegates to `Character.offsetByCodePoints()` >> which in turn specifies the same exception thrown under the same conditions >> and the implementation does exactly the same checks. This means we can >>

Re: RFR: 8297561: Redundant index check in String.offsetByCodePoints() [v2]

2022-11-30 Thread Sergey Tsypanov
> `String.offsetByCodePoints()` delegates to `Character.offsetByCodePoints()` > which in turn specifies the same exception thrown under the same conditions > and the implementation does exactly the same checks. This means we can remove > the check from `String.offsetByCodePoints()` and rely on