[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: PING: Re: [cp-patches] RFC: Fix for "XXX" use in java.util.Currency
From: |
Michael Koch |
Subject: |
Re: PING: Re: [cp-patches] RFC: Fix for "XXX" use in java.util.Currency |
Date: |
Mon, 11 Apr 2005 08:11:00 +0200 |
User-agent: |
mutt-ng 1.5.9-r243i (Debian) |
On Mon, Apr 11, 2005 at 08:46:42AM +0200, Jeroen Frijters wrote:
> Andrew John Hughes wrote:
> > On Mon, 2005-03-28 at 01:41 +0100, Andrew John Hughes wrote:
> > > The attached patch allows examples such as the following:
> > >
> > > NumberFormat.getCurrencyInstance(Locale.ENGLISH).getCurrency()
> > >
> > > to succeed with the result of a Currency instance containing "XXX"
> > > rather than throwing a NullPointerException. This is Sun's
> > behaviour,
> > > which is made possible by allowing "XXX" to be supplied to
> > > Currency.getInstance(String). This is something we have discussed
> > > before, which has lead to the introduction of the String version
> > > of this value in java.text.DecimalFormatSymbols (11 Jan -
> > Fix for Bug #11545).
> > >
> > > Should we allow this horrible design decision to creep into
> > > GNU Classpath? Comments please.
> > >
> > > Changelog:
> > >
> > > 2005-03-28 Andrew John Hughes <address@hidden>
> > >
> > > * java/text/DecimalFormatSymbols.java:
> > > Added retrieval of "XXX" instance in place of null.
> > > * java/util/Currency.java,
> > > (Currency(String)): New constructor for the XXX special case.
> > > (getInstance(String)): Allow special case of "XXX".
> > >
> > Does anyone have any comments on this patch, posted March
> > 28th? Should I commit it or not?
>
> I like it and I vote for committing it.
/me too.
Michael
--
Escape the Java Trap with GNU Classpath!
http://www.gnu.org/philosophy/java-trap.html
Join the community at http://planet.classpath.org/