On Fri, 1 Sep 2023 23:12:28 GMT, Justin Lu <j...@openjdk.org> wrote:

> Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8315558) 
> which is a conformance change to document some exceptions in the 
> specification of java.text.StringCharacterIterator.

src/java.base/share/classes/java/text/StringCharacterIterator.java line 158:

> 156:      * Implements CharacterIterator.setIndex() for String.
> 157:      *
> 158:      * @throws IllegalArgumentException if {@code p} is an invalid index

It'd be clearer if we clarify what the `invalid` means

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15547#discussion_r1316222951

Reply via email to