Re: [11] RFR: JDK-8198385: Remove property sun.locale.formatasdefault
Thanks, Alan. There is no test case affected by this change, so I added "noreg" label to the jira issue. Naoto On 2/22/18 5:34 AM, Alan Bateman wrote: On 21/02/2018 21:13, naoto.s...@oracle.com wrote: Hello, Please review the fix to the following issue: https://bugs.openjdk.java.net/browse/JDK-8198385 The proposed changeset is located at: http://cr.openjdk.java.net/~naoto/8198385/webrev.00/ The property was introduced in JDK7 for the backward compatibility, which at this point is no longer needed. Corresponding CSR is already approved. This looks okay. Are there are tests to be adjusted/removed as part of this? -Alan
Re: [11] RFR: JDK-8198385: Remove property sun.locale.formatasdefault
On 21/02/2018 21:13, naoto.s...@oracle.com wrote: Hello, Please review the fix to the following issue: https://bugs.openjdk.java.net/browse/JDK-8198385 The proposed changeset is located at: http://cr.openjdk.java.net/~naoto/8198385/webrev.00/ The property was introduced in JDK7 for the backward compatibility, which at this point is no longer needed. Corresponding CSR is already approved. This looks okay. Are there are tests to be adjusted/removed as part of this? -Alan
Re: [11] RFR: JDK-8198385: Remove property sun.locale.formatasdefault
Hi Naoto, +1 Brian On Feb 21, 2018, at 1:13 PM, naoto.s...@oracle.com wrote: > Please review the fix to the following issue: > > https://bugs.openjdk.java.net/browse/JDK-8198385 > > The proposed changeset is located at: > > http://cr.openjdk.java.net/~naoto/8198385/webrev.00/ > > The property was introduced in JDK7 for the backward compatibility, which at > this point is no longer needed. Corresponding CSR is already approved.
[11] RFR: JDK-8198385: Remove property sun.locale.formatasdefault
Hello, Please review the fix to the following issue: https://bugs.openjdk.java.net/browse/JDK-8198385 The proposed changeset is located at: http://cr.openjdk.java.net/~naoto/8198385/webrev.00/ The property was introduced in JDK7 for the backward compatibility, which at this point is no longer needed. Corresponding CSR is already approved. Naoto