> `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 removes `NoRepl` suffix from method names and consistently uses > `throws CCE` in method footprints. (b4845109e18 passes `tier1,2`.)
Volkan Yazici 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 seven additional commits since the last revision: - Group `String` methods by `doReplace` argument - Merge remote-tracking branch 'upstream/master' into jlaNoRepl - Replace `requireNonNull` with implicit null checks - Merge remote-tracking branch 'upstream/master' into jlaNoRepl - Improve docs of touched methods and add NPE checks - Convert IAE-throwing methods into CCE-throwing ones - Rename `JavaLangAccess::*NoRepl` methods ------------- Changes: - all: https://git.openjdk.org/jdk/pull/26413/files - new: https://git.openjdk.org/jdk/pull/26413/files/5f555a68..1fb8582e Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=26413&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26413&range=00-01 Stats: 18798 lines in 552 files changed: 11544 ins; 5896 del; 1358 mod Patch: https://git.openjdk.org/jdk/pull/26413.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/26413/head:pull/26413 PR: https://git.openjdk.org/jdk/pull/26413
