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 <[EMAIL PROTECTED]> > > > > > > * 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/ _______________________________________________ Classpath-patches mailing list Classpath-patches@gnu.org http://lists.gnu.org/mailman/listinfo/classpath-patches