Re: RFR: 8300916: Re-examine the initialization of JNU Charset in StaticProperty [v9]

2023-01-26 Thread Naoto Sato
On Thu, 26 Jan 2023 12:07:20 GMT, Alan Bateman wrote: > Thanks for the updates/iterations, I think you've got this to a good place. > > One thing to think about is having System.initPhase3 read file.encoding and > if not UTF-8, it could call Charset.defaultCharset and if not the expected > val

Re: RFR: 8300916: Re-examine the initialization of JNU Charset in StaticProperty [v9]

2023-01-26 Thread Alan Bateman
On Thu, 26 Jan 2023 00:19:45 GMT, Naoto Sato wrote: >> This issue was found during the review of this PR: >> https://github.com/openjdk/jdk/pull/12132 where `Charset` class was >> loaded/initialized at the phase 1 of the startup process. Since `Charset` >> depends on `StaticProperty`, loading

Re: RFR: 8300916: Re-examine the initialization of JNU Charset in StaticProperty [v9]

2023-01-25 Thread Mandy Chung
On Thu, 26 Jan 2023 00:19:45 GMT, Naoto Sato wrote: >> This issue was found during the review of this PR: >> https://github.com/openjdk/jdk/pull/12132 where `Charset` class was >> loaded/initialized at the phase 1 of the startup process. Since `Charset` >> depends on `StaticProperty`, loading

Re: RFR: 8300916: Re-examine the initialization of JNU Charset in StaticProperty [v9]

2023-01-25 Thread Naoto Sato
> This issue was found during the review of this PR: > https://github.com/openjdk/jdk/pull/12132 where `Charset` class was > loaded/initialized at the phase 1 of the startup process. Since `Charset` > depends on `StaticProperty`, loading of `Charset` class should be delayed. I > basically moved