Re: [9] RFR: 8057646: ClassCircularityError running SQE test

2014-09-09 Thread Alan Bateman
On 09/09/2014 00:04, Naoto Sato wrote: Hello, Please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8057646 The proposed changes are located at: http://cr.openjdk.java.net/~naoto/8057646/webrev.0/ It's not clear to me that this is the right place to address th

Re: [9] RFR: 8057646: ClassCircularityError running SQE test

2014-09-09 Thread Naoto Sato
On 9/9/14, 12:39 AM, Alan Bateman wrote: It's not clear to me that this is the right place to address this issue. Clearly the changes to JRELocaleProviderAdapter to use ServiceLoader have upset the ordering that things are initialized for this error case, but it looks to me that it's always been

Re: [9] RFR: 8057646: ClassCircularityError running SQE test

2014-09-09 Thread Alan Bateman
On 09/09/2014 18:14, Naoto Sato wrote: It's an inherent issue where some init code issues locale sensitive services, such as in this case, *Formatter. So this could happen not only in Security class, but could be anywhere. So I think we still need the change proposed here in order for avoidin

Re: [9] RFR: 8057646: ClassCircularityError running SQE test

2014-09-09 Thread Naoto Sato
OK. I will transfer the issue to security-libs. Naoto On 9/9/14, 12:38 PM, Alan Bateman wrote: On 09/09/2014 18:14, Naoto Sato wrote: It's an inherent issue where some init code issues locale sensitive services, such as in this case, *Formatter. So this could happen not only in Security class