RFR: 8356439: Rename JavaLangAccess::*NoRepl methods [v7]
Volkan Yazici
vyazici at openjdk.org
Thu Aug 28 10:46:02 UTC 2025
On Wed, 27 Aug 2025 23:18:11 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Volkan Yazici has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Improve "sneaky throws"
>> - Improve comment style
>
> src/java.base/share/classes/java/lang/String.java line 890:
>
>> 888:
>> 889: private static <E extends Exception> byte[] encodeWithEncoder(
>> 890: Charset cs, byte coder, byte[] val, Class<E> exceptionClass)
>
> Since `exceptionClass` is a dummy argument that can't actually control the type of exception thrown by this method, that it being null or not is the only thing that matters, I recommend documenting/commenting about this fact, as this is otherwise potentially confusing. Maybe a name to indicate this nature (I came up with `replacementSwitch` but this is for sure horrible) would help too I guess?
I think you have a point – I've improved the docs in fa46a81bdc2.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26413#discussion_r2307007890
More information about the security-dev
mailing list