RFR: 8306597: Improve string formatting in EquivMapsGenerator.java [v3]
Naoto Sato
naoto at openjdk.org
Fri May 12 16:43:55 UTC 2023
On Thu, 11 May 2023 22:23:47 GMT, Justin Lu <jlu at openjdk.org> wrote:
>> Please review changes to `EquivMapsGenerator.java` (which is used to generate the Locale equivalencies for the JDK).
>>
>> The file previously used large concatenated Strings, which are now replaced with text blocks, in addition to some cleanup. No functionality is changed, the Locale equivalencies builds the same.
>
> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove more ws in text block
make/jdk/src/classes/build/tools/generatelsrequivmaps/EquivMapsGenerator.java line 344:
> 342: }
> 343:
> 344: private static final String footerText = " }\n\n}";
I'd prefer this one also be uppercased (and moved to a more suitable location).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13935#discussion_r1192593829
More information about the core-libs-dev
mailing list