2008/12/31 Andrew Haley <a...@redhat.com>:
> 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")
>

        case RFC822_TIMEZONE_FIELD:
             ^^^^^^^^^^^^^^^^^^^^^
case expressions must be constant expressions

> 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.
>

Yes, leftover from debugging this I guess.
-- 
Andrew :-)

Support Free Java!
Contribute to GNU Classpath and the OpenJDK
http://www.gnu.org/software/classpath
http://openjdk.java.net

PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8

Reply via email to