[Bug libgcj/21736] New: MessageFormat("{0,time,medium}") fails

[Bug libgcj/21736] New: MessageFormat("{0,time,medium}") fails

[gbenson at redhat dot com]

Attempting to create a MessageFormat with a "time" element styled as "medium"
raises an exception.  The error, in MessageFormat.setLocale(), happens because
DateFormat.DEFAULT has the same value as DateFormat.MEDIUM, and the code was
obviously written with the assumption that this is not the case.

-- 
           Summary: MessageFormat("{0,time,medium}") fails
           Product: gcc
           Version: 4.0.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: libgcj
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: gbenson at redhat dot com
                CC: gcc-bugs at gcc dot gnu dot org,java-prs at gcc dot gnu
                    dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[gbenson at redhat dot com]

------- Additional Comments From gbenson at redhat dot com  2005-05-24 10:01 -------
Created an attachment (id=8959)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=8959&action=view)
Testcase


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[gbenson at redhat dot com]

------- Additional Comments From gbenson at redhat dot com  2005-05-24 10:02 -------
Created an attachment (id=8960)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=8960&action=view)
Fix


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[tromey at gcc dot gnu dot org]

------- Additional Comments From tromey at gcc dot gnu dot org  2005-05-24 17:17 -------
I'm handling this.
Thanks for the test case; looks like there is already one in mauve,
but I'll verify that first.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |tromey at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-05-24 18:16 -------
Subject: Bug 21736

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-4_0-branch
Changes by:	tromey@gcc.gnu.org	2005-05-24 18:16:11

Modified files:
	libjava        : ChangeLog 
	libjava/java/text: MessageFormat.java 

Log message:
	2005-05-24  Gary Benson  <gbenson@redhat.com>
	
	PR libgcj/21736:
	* java/text/MessageFormat.java (MessageFormatElement.setLocale):
	Handle DateFormat.DEFAULT case correctly.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.3391.2.75&r2=1.3391.2.76
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/text/MessageFormat.java.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.23&r2=1.23.2.1



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[tromey at gcc dot gnu dot org]

------- Additional Comments From tromey at gcc dot gnu dot org  2005-05-24 18:17 -------
I've checked this in to the 4.0 branch and Classpath.
I'm leaving the PR open for now; I will check this in and close it
once the trunk is out of its slushy state.
Thanks for the patch and test case.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.0.1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[gbenson at redhat dot com]

------- Additional Comments From gbenson at redhat dot com  2005-05-25 08:44 -------
No problem, thanks for handling it :)

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-05-25 15:25 -------
Subject: Bug 21736

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	tromey@gcc.gnu.org	2005-05-25 15:25:39

Modified files:
	libjava        : ChangeLog 
	libjava/java/text: MessageFormat.java 

Log message:
	2005-05-24  Gary Benson  <gbenson@redhat.com>
	
	PR libgcj/21736:
	* java/text/MessageFormat.java (MessageFormatElement.setLocale):
	Handle DateFormat.DEFAULT case correctly.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/ChangeLog.diff?cvsroot=gcc&r1=1.3632&r2=1.3633
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/text/MessageFormat.java.diff?cvsroot=gcc&r1=1.23&r2=1.24



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736

[Bug libgcj/21736] MessageFormat("{0,time,medium}") fails

[tromey at gcc dot gnu dot org]

------- Additional Comments From tromey at gcc dot gnu dot org  2005-05-25 15:26 -------
Fix checked in.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21736