--- old/src/java.base/share/classes/java/util/Locale.java 2017-11-10 15:30:24.195439060 -0800
+++ new/src/java.base/share/classes/java/util/Locale.java 2017-11-10 15:30:23.853432667 -0800
@@ -48,6 +48,7 @@
import java.text.MessageFormat;
import java.util.concurrent.ConcurrentHashMap;
import java.util.spi.LocaleNameProvider;
+import java.util.stream.Collectors;
import sun.security.action.GetPropertyAction;
import sun.util.locale.BaseLocale;
@@ -62,6 +63,7 @@
import sun.util.locale.provider.LocaleProviderAdapter;
import sun.util.locale.provider.LocaleResources;
import sun.util.locale.provider.LocaleServiceProviderPool;
+import sun.util.locale.provider.TimeZoneNameUtility;
/**
* A Locale
object represents a specific geographical, political,
@@ -665,10 +667,12 @@
/**
* Display types for retrieving localized names from the name providers.
*/
- private static final int DISPLAY_LANGUAGE = 0;
- private static final int DISPLAY_COUNTRY = 1;
- private static final int DISPLAY_VARIANT = 2;
- private static final int DISPLAY_SCRIPT = 3;
+ private static final int DISPLAY_LANGUAGE = 0;
+ private static final int DISPLAY_COUNTRY = 1;
+ private static final int DISPLAY_VARIANT = 2;
+ private static final int DISPLAY_SCRIPT = 3;
+ private static final int DISPLAY_UEXT_KEY = 4;
+ private static final int DISPLAY_UEXT_TYPE = 5;
/**
* Private constructor used by getInstance method
@@ -942,11 +946,14 @@
variant = props.getProperty("user.variant", "");
}
- return getInstance(language, script, country, variant, null);
+ return getInstance(language, script, country, variant,
+ getDefaultExtensions(props.getProperty("user.extensions", ""))
+ .orElse(null));
}
private static Locale initDefault(Locale.Category category) {
Properties props = GetPropertyAction.privilegedGetProperties();
+
return getInstance(
props.getProperty(category.languageKey,
defaultLocale.getLanguage()),
@@ -956,7 +963,22 @@
defaultLocale.getCountry()),
props.getProperty(category.variantKey,
defaultLocale.getVariant()),
- null);
+ getDefaultExtensions(props.getProperty(category.extensionsKey, ""))
+ .orElse(defaultLocale.getLocaleExtensions()));
+ }
+
+ private static Optional getDefaultExtensions(String extensionsProp) {
+ LocaleExtensions exts = null;
+
+ try {
+ exts = new InternalLocaleBuilder()
+ .setExtensions(extensionsProp)
+ .getLocaleExtensions();
+ } catch (LocaleSyntaxException e) {
+ // just ignore this incorrect property
+ }
+
+ return Optional.ofNullable(exts);
}
/**
@@ -1771,7 +1793,7 @@
* @exception NullPointerException if inLocale
is null
*/
public String getDisplayLanguage(Locale inLocale) {
- return getDisplayString(baseLocale.getLanguage(), inLocale, DISPLAY_LANGUAGE);
+ return getDisplayString(baseLocale.getLanguage(), null, inLocale, DISPLAY_LANGUAGE);
}
/**
@@ -1801,7 +1823,7 @@
* @since 1.7
*/
public String getDisplayScript(Locale inLocale) {
- return getDisplayString(baseLocale.getScript(), inLocale, DISPLAY_SCRIPT);
+ return getDisplayString(baseLocale.getScript(), null, inLocale, DISPLAY_SCRIPT);
}
/**
@@ -1844,29 +1866,24 @@
* @exception NullPointerException if inLocale
is null
*/
public String getDisplayCountry(Locale inLocale) {
- return getDisplayString(baseLocale.getRegion(), inLocale, DISPLAY_COUNTRY);
+ return getDisplayString(baseLocale.getRegion(), null, inLocale, DISPLAY_COUNTRY);
}
- private String getDisplayString(String code, Locale inLocale, int type) {
- if (code.length() == 0) {
- return "";
- }
+ private String getDisplayString(String code, String cat, Locale inLocale, int type) {
+ Objects.requireNonNull(inLocale);
+ Objects.requireNonNull(code);
- if (inLocale == null) {
- throw new NullPointerException();
+ if (code.isEmpty()) {
+ return "";
}
LocaleServiceProviderPool pool =
LocaleServiceProviderPool.getPool(LocaleNameProvider.class);
- String key = (type == DISPLAY_VARIANT ? "%%"+code : code);
+ String rbKey = (type == DISPLAY_VARIANT ? "%%"+code : code);
String result = pool.getLocalizedObject(
LocaleNameGetter.INSTANCE,
- inLocale, key, type, code);
- if (result != null) {
- return result;
- }
-
- return code;
+ inLocale, rbKey, type, code, cat);
+ return result != null ? result : code;
}
/**
@@ -1894,29 +1911,31 @@
if (baseLocale.getVariant().length() == 0)
return "";
- LocaleResources lr = LocaleProviderAdapter.forJRE().getLocaleResources(inLocale);
+ LocaleResources lr = LocaleProviderAdapter
+ .getResourceBundleBased()
+ .getLocaleResources(inLocale);
String names[] = getDisplayVariantArray(inLocale);
// Get the localized patterns for formatting a list, and use
// them to format the list.
return formatList(names,
- lr.getLocaleName("ListPattern"),
lr.getLocaleName("ListCompositionPattern"));
}
/**
* Returns a name for the locale that is appropriate for display to the
* user. This will be the values returned by getDisplayLanguage(),
- * getDisplayScript(), getDisplayCountry(), and getDisplayVariant() assembled
- * into a single string. The non-empty values are used in order,
- * with the second and subsequent names in parentheses. For example:
+ * getDisplayScript(), getDisplayCountry(), getDisplayVariant() and
+ * optional Unicode extensions
+ * assembled into a single string. The non-empty values are used in order, with
+ * the second and subsequent names in parentheses. For example:
*
- * language (script, country, variant)
- * language (country)
- * language (variant)
- * script (country)
- * country
+ * language (script, country, variant(, extension)*)
+ * language (country(, extension)*)
+ * language (variant(, extension)*)
+ * script (country(, extension)*)
+ * country (extension)*
*
* depending on which fields are specified in the locale. If the
* language, script, country, and variant fields are all empty,
@@ -1931,16 +1950,17 @@
/**
* Returns a name for the locale that is appropriate for display
* to the user. This will be the values returned by
- * getDisplayLanguage(), getDisplayScript(),getDisplayCountry(),
- * and getDisplayVariant() assembled into a single string.
- * The non-empty values are used in order,
- * with the second and subsequent names in parentheses. For example:
+ * getDisplayLanguage(), getDisplayScript(),getDisplayCountry()
+ * getDisplayVariant(), and optional
+ * Unicode extensions assembled into a single string. The non-empty
+ * values are used in order, with the second and subsequent names in
+ * parentheses. For example:
*
- * language (script, country, variant)
- * language (country)
- * language (variant)
- * script (country)
- * country
+ * language (script, country, variant(, extension)*)
+ * language (country(, extension)*)
+ * language (variant(, extension)*)
+ * script (country(, extension)*)
+ * country (extension)*
*
* depending on which fields are specified in the locale. If the
* language, script, country, and variant fields are all empty,
@@ -1951,7 +1971,9 @@
* @throws NullPointerException if inLocale
is null
*/
public String getDisplayName(Locale inLocale) {
- LocaleResources lr = LocaleProviderAdapter.forJRE().getLocaleResources(inLocale);
+ LocaleResources lr = LocaleProviderAdapter
+ .getResourceBundleBased()
+ .getLocaleResources(inLocale);
String languageName = getDisplayLanguage(inLocale);
String scriptName = getDisplayScript(inLocale);
@@ -1960,7 +1982,6 @@
// Get the localized patterns for formatting a display name.
String displayNamePattern = lr.getLocaleName("DisplayNamePattern");
- String listPattern = lr.getLocaleName("ListPattern");
String listCompositionPattern = lr.getLocaleName("ListCompositionPattern");
// The display name consists of a main name, followed by qualifiers.
@@ -1977,7 +1998,7 @@
if (variantNames.length == 0) {
return "";
} else {
- return formatList(variantNames, listPattern, listCompositionPattern);
+ return formatList(variantNames, listCompositionPattern);
}
}
ArrayList names = new ArrayList<>(4);
@@ -1994,6 +2015,16 @@
names.addAll(Arrays.asList(variantNames));
}
+ // add Unicode extensions
+ if (localeExtensions != null) {
+ localeExtensions.getUnicodeLocaleAttributes().stream()
+ .map(key -> getDisplayString(key, null, inLocale, DISPLAY_UEXT_KEY))
+ .forEach(names::add);
+ localeExtensions.getUnicodeLocaleKeys().stream()
+ .map(key -> getDisplayKeyTypeExtensionString(key, lr, inLocale))
+ .forEach(names::add);
+ }
+
// The first one in the main name
mainName = names.get(0);
@@ -2014,7 +2045,7 @@
// list case, but this is more efficient, and we want it to be
// efficient since all the language-only locales will not have any
// qualifiers.
- qualifierNames.length != 0 ? formatList(qualifierNames, listPattern, listCompositionPattern) : null
+ qualifierNames.length != 0 ? formatList(qualifierNames, listCompositionPattern) : null
};
if (displayNamePattern != null) {
@@ -2121,74 +2152,78 @@
// For each variant token, lookup the display name. If
// not found, use the variant name itself.
for (int i=0; i 3) {
- MessageFormat format = new MessageFormat(listCompositionPattern);
- stringList = composeList(format, stringList);
+ switch (stringList.length) {
+ case 0:
+ return "";
+ case 1:
+ return stringList[0];
+ default:
+ return Arrays.stream(stringList).reduce("",
+ (s1, s2) -> {
+ if (s1.equals("")) {
+ return s2;
+ }
+ if (s2.equals("")) {
+ return s1;
+ }
+ return MessageFormat.format(pattern, s1, s2);
+ });
}
-
- // Rebuild the argument list with the list length as the first element
- Object[] args = new Object[stringList.length + 1];
- System.arraycopy(stringList, 0, args, 1, stringList.length);
- args[0] = stringList.length;
-
- // Format it using the pattern in the resource
- MessageFormat format = new MessageFormat(listPattern);
- return format.format(args);
- }
-
- /**
- * Given a list of strings, return a list shortened to three elements.
- * Shorten it by applying the given format to the first two elements
- * recursively.
- * @param format a format which takes two arguments
- * @param list a list of strings
- * @return if the list is three elements or shorter, the same list;
- * otherwise, a new list of three elements.
- */
- private static String[] composeList(MessageFormat format, String[] list) {
- if (list.length <= 3) return list;
-
- // Use the given format to compose the first two elements into one
- String[] listItems = { list[0], list[1] };
- String newItem = format.format(listItems);
-
- // Form a new list one element shorter
- String[] newList = new String[list.length-1];
- System.arraycopy(list, 2, newList, 1, newList.length-1);
- newList[0] = newItem;
-
- // Recurse
- return composeList(format, newList);
}
// Duplicate of sun.util.locale.UnicodeLocaleExtension.isKey in order to
@@ -2345,9 +2380,10 @@
Locale locale,
String key,
Object... params) {
- assert params.length == 2;
+ assert params.length == 3;
int type = (Integer)params[0];
String code = (String)params[1];
+ String cat = (String)params[2];
switch(type) {
case DISPLAY_LANGUAGE:
@@ -2358,6 +2394,10 @@
return localeNameProvider.getDisplayVariant(code, locale);
case DISPLAY_SCRIPT:
return localeNameProvider.getDisplayScript(code, locale);
+ case DISPLAY_UEXT_KEY:
+ return localeNameProvider.getDisplayUnicodeExtensionKey(code, locale);
+ case DISPLAY_UEXT_TYPE:
+ return localeNameProvider.getDisplayUnicodeExtensionType(code, cat, locale);
default:
assert false; // shouldn't happen
}
@@ -2384,7 +2424,8 @@
DISPLAY("user.language.display",
"user.script.display",
"user.country.display",
- "user.variant.display"),
+ "user.variant.display",
+ "user.extensions.display"),
/**
* Category used to represent the default locale for
@@ -2393,19 +2434,23 @@
FORMAT("user.language.format",
"user.script.format",
"user.country.format",
- "user.variant.format");
+ "user.variant.format",
+ "user.extensions.format");
- Category(String languageKey, String scriptKey, String countryKey, String variantKey) {
+ Category(String languageKey, String scriptKey, String countryKey,
+ String variantKey, String extensionsKey) {
this.languageKey = languageKey;
this.scriptKey = scriptKey;
this.countryKey = countryKey;
this.variantKey = variantKey;
+ this.extensionsKey = extensionsKey;
}
final String languageKey;
final String scriptKey;
final String countryKey;
final String variantKey;
+ final String extensionsKey;
}
/**