The Mauve test java.text.MessageFormat.format14 demonstrates
that the integer format used by MessageFormat should use the
formatter returned by NumberFormat.getIntegerInstance.  This
is further reinforced by the online specification.  This is probably
a case of our MessageFormat being outdated due to changes in 1.4.
This corrects it.

ChangeLog:

2009-01-02  Andrew John Hughes  <gnu_and...@member.fsf.org>

        * java/text/MessageFormat.java:
        (setLocale(Locale)): Integer format should use
        NumberFormat.getIntegerInstance.

-- 
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
Index: java/text/MessageFormat.java
===================================================================
RCS file: /sources/classpath/classpath/java/text/MessageFormat.java,v
retrieving revision 1.29
diff -u -u -r1.29 MessageFormat.java
--- java/text/MessageFormat.java        31 Dec 2008 11:28:12 -0000      1.29
+++ java/text/MessageFormat.java        2 Jan 2009 12:41:31 -0000
@@ -99,12 +99,7 @@
               else if (style.equals("percent"))
                 format = NumberFormat.getPercentInstance(loc);
               else if (style.equals("integer"))
-                {
-                  NumberFormat nf = NumberFormat.getNumberInstance(loc);
-                  nf.setMaximumFractionDigits(0);
-                  nf.setGroupingUsed(false);
-                  format = nf;
-                }
+               format = NumberFormat.getIntegerInstance(loc);
               else
                 {
                   format = NumberFormat.getNumberInstance(loc);

Reply via email to