* <p>A well-formed locale key has the form
  * <code>[0-9a-zA-Z]{2}</code>.  A well-formed locale type has the
  * form <code>"" | [0-9a-zA-Z]{3,8} ('-' [0-9a-zA-Z]{3,8})*</code> (it
- * can be empty, or a series of subtags 3-8 alphanums in length). A
+ * can be empty, or a series of subtags 3-8 alphanumerics in length).  A
  * well-formed locale attribute has the form
  * <code>[0-9a-zA-Z]{3,8}</code> (it is a single subtag with the same
  * form as a locale type subtag).

This is the only place where I have replaced alphanums with alphanumerics, because the second word is used later in the class specification. For example:

     * <p><b>Note:</b> The <code>Locale</code> class does not provide any
* syntactic restrictions on variant, while BCP 47 requires each variant
     * subtag to be 5 to 8 alphanumerics or a single numeric followed by 3
     * alphanumerics.  The method <code>setVariant</code> throws

I'll revert the change if you want.

Thanks,
SAM

On 22.10.2013 21:34, Naoto Sato wrote:
Hi Sergey,

I looked at the changes wrt i18n, and found one change in Locale class:

221 * can be empty, or a series of subtags 3-8 alphanumerics in length). A

where you modified "alphanums" to "alphanumerics". In fact it's not a typo, as "alphanum" is defined in BCP 47 standard.

Naoto

On 10/22/13 5:22 AM, sergey malenkov wrote:
Hello,

Could you please review the following huge fix:
fix:http://cr.openjdk.java.net/~malenkov/8022746.8.0/
bug:https://bugs.openjdk.java.net/browse/JDK-8022746

I've replaced all typos with correct words in all documentations and
comments.  Our technical writer has already checked it.

Thanks,
SAM


Reply via email to