RFR: 8356439: Rename JavaLangAccess::*NoRepl methods [v10]
Alan Bateman
alanb at openjdk.org
Sat Aug 30 14:04:46 UTC 2025
On Fri, 29 Aug 2025 13:48:52 GMT, Volkan Yazici <vyazici at openjdk.org> 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 two additional commits since the last revision:
>
> - Improve Javadoc of touched `encode*()` methods
> - Rename `exceptionClass` to `exClass`
Thanks for the updates, I don't have any more comments.
-------------
Marked as reviewed by alanb (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26413#pullrequestreview-3171050605
More information about the security-dev
mailing list