RFR: 8266666: Implementation for snippets [v23]
Jonathan Gibbons
jjg at openjdk.java.net
Mon Sep 13 19:02:59 UTC 2021
On Mon, 13 Sep 2021 18:34:59 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
>> Pavel Rappo has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove trailing whitespace to satisfy jcheck
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/taglets/snippet/ParseException.java line 50:
>
>> 48: * @param position the approximate position
>> 49: */
>> 50: public ParseException(String message, int position) {
>
> I know it is common to use English-only strings in exceptions that are not intended to be seen in normal use by an end user, but here, you're using `ParseException` to contain content that is intended to appear in end-user error messages, triggered by errors in the content provided by an end-user. As such, it is close to a firm requirement that these messages can be localized.
One possible solution at this time that would be minimally invasive would be to replace `new ParseException(String message, int pos)` with use of a method that localizes the string first, as in,
ParseException newParseException(int pos, String resourceKey, Object... args)
so the code becomes something like
throw newParseException(pos, key, args);
-------------
PR: https://git.openjdk.java.net/jdk/pull/4795
More information about the compiler-dev
mailing list