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 <[EMAIL PROTECTED]>

  * 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



_______________________________________________
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath

Reply via email to