On Thu, 17 Apr 2025 00:30:00 GMT, Stuart Marks <sma...@openjdk.org> wrote:
>> Remove the #ifdef MACOSX from platform-independent System.c, which (on Mac) >> sets native.encoding from sprops.encoding or (on non-Mac Unix and Windows) >> sets native.encoding from sprops.sun_jnu_encoding. After this removal, the >> native.encoding property will be set from sprops.encoding and the >> sun.jnu.encoding property will be set from sprops.sun_jnu_encoding. >> >> Change the windows java_props_md.c so that it initializes >> sprops.sun_jnu_encoding and then copies that value to sprops.encoding. >> Previously, sprops.encoding was initialized with data from other Windows >> APIs, but this value was ignored by platform-independent code, which was >> kind of confusing. >> >> There are no changes to the Unix (Mac & non-Mac) platform-specific code. It >> already has code to set sprops.encoding and sprops.sun_jnu_encoding >> properly, including macOS-specific code to set the latter to UTF-8 >> unconditionally. >> >> There should be no behavior changes on any platform. >> >> I changed variable names in windows/java_props_md.c to make it clear that >> the values stored into them are unused. Of course there are other >> possibilities here, such as changing the interface to SetupI18nProps(). I'm >> open to suggestions. > > Stuart Marks has updated the pull request incrementally with one additional > commit since the last revision: > > Remove encoding parameter from SetupI18nProps. I probably would have waited to integrate until the person who requested changes had a chance to re-approve. ------------- PR Comment: https://git.openjdk.org/jdk/pull/24607#issuecomment-2813794530
