On Thu, 28 Aug 2025 10:46:01 GMT, Volkan Yazici <[email protected]> wrote:
>> `NoRepl`-suffixed `String` methods denote methods that do not replace
>> invalid characters, but throw `CharacterCodingException` on encounter. This
>> behavior cannot easily be derived from the method footprints, has been a
>> source of confusion for maintainers, and is not uniformly adopted, e.g.,
>> `newStringUTF8NoRepl()` and `getBytesUTF8NoRepl()` does *not* throw `CCE`.
>> This PR replaces the `NoRepl` suffix with `NoReplacement` in method names
>> and consistently uses `throws CCE` in method footprints.
>
> Volkan Yazici has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Improve docs
src/java.base/share/classes/java/lang/String.java line 1336:
> 1334:
> 1335: /**
> 1336: * {@return a new {@link MalformedInputException} for the sub-range
> denoted by specified {@code offset} and {@code length}}
Some of the very long lines make it hard to look at the changes side-by-side,
would you mind trimming some of these back to make it easier for reviewers?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26413#discussion_r2307325258
