RFR: 8356439: Rename JavaLangAccess::*NoRepl methods [v6]
Volkan Yazici
vyazici at openjdk.org
Mon Aug 25 07:51:52 UTC 2025
On Fri, 22 Aug 2025 14:25:19 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Volkan Yazici has updated the pull request incrementally with five additional commits since the last revision:
>>
>> - Renamed to `malformedASCII`
>> - Improve exception parametrization
>> - Document parametrization on the exception type
>> - Avoid using links in the Javadoc title line
>> - Rename `NoReplTest` and fix its copyright year
>
> src/java.base/share/classes/java/lang/String.java line 1316:
>
>> 1314:
>> 1315: @SuppressWarnings("unchecked")
>> 1316: private static <E extends Exception> E malformedInputException(int off, int nb, Class<E> exceptionClass) {
>
> The argument name can be "_", since it is ignored and is unused in the body.
> Comment the method as always throwing MalformedInputException.
> Similarly in the other malformedInput methods*.
I've removed the need to pass a `Class<E>` to these exception factories, and improved their documentation in 584e6e213ec.
> The argument name can be "_", since it is ignored and is unused in the body.
For the record, AFAIK, this is not possible for method arguments. Quoting [§6.1 (Names > Declarations)](https://docs.oracle.com/javase/specs/jls/se24/html/jls-6.html#jls-6.1):
> If a declaration does not include an identifier, but instead includes the keyword _ (underscore), then the entity cannot be referred to by name. The following kinds of entity may be declared using an underscore:
>
> * A local variable, one of the following: ...
> * An exception parameter of an exception handler declared in a catch clause of a try statement
> * A formal parameter of a lambda expression
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26413#discussion_r2297377015
More information about the security-dev
mailing list