Re: [PATCH] Compliance of DecimalFormatSymbols

[Dalibor Topic]

Salut Guilhem,

Guilhem Lavaux wrote:
> Hi,
>
> Here is an adaptation of the former patch concerning java/text.
> This one is small and concerns only java/text/DecimalFormatSymbols.
> According to the official specification, if the serial version is greater
> than 2, you have to read the locale field. This is what is done here.

Looks good to me beside just a small point. Could you write some mauve 
tests to go with it, please?


> Regards,
> Guilhem.
>
> ChangeLog entry:
>
> 2003-11-21 Guilhem Lavaux <address@hidden>
>
>   * java/text/DecimalFormatSymbols.java    (locale): New field.
>   (serialVersionOnStream): Upgraded to number 2
>   (readObject): updated to assign locale if it wasn't by the
>   serializer.

Lacks one entry:
(DecimalFormatSymbols (Locale)): Set locale.

Other than that, looks fine to me. I'm waiting for the ChangeLog police, 
though ;)

cheers,
dalibor topic