Review request for 8050968: Extension class loader initialization fails on Win7 x64 zh_TW

[David Holmes]

Hi Mandy,

On 18/07/2014 2:57 PM, Mandy Chung wrote:
>
> On 7/17/2014 9:56 PM, Mandy Chung wrote:
>> This is a regression caused by JDK-8038177 that changed
>> sun.nio.cs.ext.ExtendedCharsets to use lambda.  It turns out that
>> ExtendedCharsets is loaded during the initialization of the extension
>> class loader when opening jre/lib/ext/meta-index on windows with system
>> locale loading extended charsets and caused the initialization error.
>>
>> The fix is not to avoid lambda in ExtendedCharsets getting the system
>
> s/avoid/use

This looks okay to me.

It is unfortunately very hard to know how introducing a lambda 
expression can affect the initialization order, particularly when it may 
depend on locales etc.

David
-----

> Mandy
>
>> property value.  At some point in the future, I'd like to take a closer
>> look at the system initialization path and what we can do better.
>>
>> Webrev at:
>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8050968/webrev.00/
>>
>> thanks
>> Mandy
>>
>