RFR: 8301991: Convert l10n properties resource bundles to UTF-8 native [v4]
Naoto Sato
naoto at openjdk.org
Fri Mar 17 21:05:18 UTC 2023
On Fri, 17 Mar 2023 20:31:27 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Adjust CF test to read in with UTF-8 to fix failing test
>
> make/jdk/src/classes/build/tools/compileproperties/CompileProperties.java line 226:
>
>> 224: Properties p = new Properties();
>> 225: try {
>> 226: FileInputStream input = new FileInputStream(propertiesPath);
>
> Should this stream be closed in a finally { } block?
or better be `try-with-resources`?
-------------
PR: https://git.openjdk.org/jdk/pull/12726
More information about the serviceability-dev
mailing list