RFR: 8301991: Convert l10n properties resource bundles to UTF-8 native [v6]
Naoto Sato
naoto at openjdk.org
Thu May 11 21:51:13 UTC 2023
On Thu, 11 May 2023 20:21:57 GMT, Justin Lu <jlu at openjdk.org> wrote:
>> This PR converts Unicode sequences to UTF-8 native in .properties file. (Excluding the Unicode space and tab sequence). The conversion was done using native2ascii.
>>
>> In addition, the build logic is adjusted to support reading in the .properties files as UTF-8 during the conversion from .properties file to .java ListResourceBundle file.
>
> Justin Lu has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 16 commits:
>
> - Convert the merged master changes to UTF-8
> - Merge master and fix conflicts
> - Close streams when finished loading into props
> - Adjust CF test to read in with UTF-8 to fix failing test
> - Reconvert CS.properties to UTF-8
> - Revert all changes to CurrencySymbols.properties
> - Bug6204853 should not be converted
> - Copyright year for CompileProperties
> - Redo translation for CS.properties
> - Spot convert CurrencySymbols.properties
> - ... and 6 more: https://git.openjdk.org/jdk/compare/4386d42d...f15b373a
I think this is fine, as those properties files are JDK's own. I believe the benefit of moving to UTF-8 outweighs the issue you wrote, which can be remedied by changing the encoding in the IDEs.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/12726#issuecomment-1544722480
More information about the core-libs-dev
mailing list