[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jakarta-commons-dev
Subject:    [jira] Closed: (LANG-368) FastDateFormat getDateInstance() and
From:       "Henri Yandell (JIRA)" <jira () apache ! org>
Date:       2007-10-31 3:59:51
Message-ID: 2658059.1193803191259.JavaMail.jira () brutus
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/LANG-368?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Henri Yandell closed LANG-368.
------------------------------

    Resolution: Fixed

Thanks Sebb, applied.

svn ci -m "Applying Sebb's test and fix from LANG-368 - fixing it so that \
FastDateFormat getDateInstance and getDateTimeInstance continue to work if \
Locale.getDefault() changes" 

src/test/org/apache/commons/lang/time/FastDateFormatTest.java 
src/java/org/apache/commons/lang/time/FastDateFormat.java
Sending        src/java/org/apache/commons/lang/time/FastDateFormat.java
Sending        src/test/org/apache/commons/lang/time/FastDateFormatTest.java
Transmitting file data ..
Committed revision 590551.

> FastDateFormat getDateInstance() and getDateTimeInstance() assume \
>                 Locale.getDefault() won't change
> --------------------------------------------------------------------------------------------------
>  
> Key: LANG-368
> URL: https://issues.apache.org/jira/browse/LANG-368
> Project: Commons Lang
> Issue Type: Bug
> Reporter: Sebb
> Attachments: FastDateFormat.patch, FastDateFormatTest.patch
> 
> 
> The FastDateFormat getDateInstance() and getDateTimeInstance()  methods create the \
> HashMap key from various items including the locale. If the locale is null, then it \
> is not made part of the key, but the stored object is created using the current \
> default locale. If the Locale is changed subsequently, then the wrong locale is \
> applied. Patch for test case to follow.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic