Re: [cp-patches] FYI: Fix regression in SimpleDateFormat

[Andrew Haley]

Andrew John Hughes wrote:
> The standard characters set in SimpleDateFormat should match
> the length of the localised characters obtained from our locale
> data.
> 
> ChangeLog:
> 
> 2008-12-30  Andrew John Hughes  <gnu_and...@member.fsf.org>
> 
>       * java/text/SimpleDateFormat.java:
>       (standardChars): Use standard characters from CLDR.
>       (RFC822_TIMEZONE_FIELD): Fixed to match new standard
>       characters.
> 
> 

Surely this

+  private static final int RFC822_TIMEZONE_FIELD = 23;

is

   private static final int RFC822_TIMEZONE_FIELD = standardChars.indexOf("Z")

And this

@@ -576,6 +576,7 @@
            int j = oldChars.indexOf(ch);
            if (j >= 0)
              ch = newChars.charAt(j);
+
          }
        buf.append(ch);
       }

is a typo.

Andrew.