changeset 51421:ef57958c7c51 jdk-12+7

8209047: "Illegal pattern character 'B'" IllegalArgumentException with Burmese locales Summary: Replaced time patterns for Burmese locale from CLDR 29's patterns. Reviewed-by: naoto
author rgoel
date Thu, 16 Aug 2018 16:06:54 +0530
parents 6bb7b8d0da76
children 41257a58a588
files src/jdk.localedata/share/classes/sun/util/cldr/resources/common/main/my.xml test/jdk/java/text/Format/DateFormat/Bug8209047.java
diffstat 2 files changed, 55 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/jdk.localedata/share/classes/sun/util/cldr/resources/common/main/my.xml	Thu Aug 16 09:46:09 2018 +0200
+++ b/src/jdk.localedata/share/classes/sun/util/cldr/resources/common/main/my.xml	Thu Aug 16 16:06:54 2018 +0530
@@ -1446,14 +1446,17 @@
 							<pattern>z HH:mm:ss</pattern>
 						</timeFormat>
 					</timeFormatLength>
+					<!--Pattern for medium and short replaced with CLDR 29's patterns 
+					    as character 'B' is currently not supported in SimpleDateFormat and java.time.DateTimeFormatter 
+					    classes. This is a restriction until JDK-8209175 is resolved.-->
 					<timeFormatLength type="medium">
 						<timeFormat>
-							<pattern>B HH:mm:ss</pattern>
+							<pattern>HH:mm:ss</pattern>
 						</timeFormat>
 					</timeFormatLength>
 					<timeFormatLength type="short">
 						<timeFormat>
-							<pattern>B H:mm</pattern>
+							<pattern>H:mm</pattern>
 						</timeFormat>
 					</timeFormatLength>
 				</timeFormats>
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/test/jdk/java/text/Format/DateFormat/Bug8209047.java	Thu Aug 16 16:06:54 2018 +0530
@@ -0,0 +1,50 @@
+/*
+ * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+ /*
+ * @test
+ * @bug 8209047
+ * @modules jdk.localedata
+ * @summary Check that DateFormat's getDateInstance, getTimeInstance and getDateTimeInstance works with all locales.
+ */
+import java.text.DateFormat;
+import java.util.Arrays;
+import java.util.Locale;
+import java.util.Set;
+
+public class Bug8209047 {
+
+    public static void main(String[] args) {
+        Set<Integer> styles = Set.of(DateFormat.FULL, DateFormat.LONG, DateFormat.MEDIUM, DateFormat.SHORT);
+        Arrays.stream(Locale.getAvailableLocales()).forEach(locale -> {
+            styles.forEach(style -> {
+                try {
+                    DateFormat.getDateInstance(style, locale);
+                    DateFormat.getTimeInstance(style, locale);
+                    DateFormat.getDateTimeInstance(style, style, locale);
+                } catch (IllegalArgumentException ex) {
+                    throw new RuntimeException("Getting DateFormat instance failed for locale " + locale, ex);
+                }
+            });
+        });
+    }
+}