At 12:55 AM 12/19/2004, Curt Arnold wrote:

CachedDateFormat is pretty interesting. However, I suspect that it may
not work properly in case the data format passed by the user causes
the length of the returned data to change over time. For example, if
the format is 'yyyy-MMMMM-dd HH:mm:ss' and the cache is initialized on
4th of July, it will work properly for the remainder of the month but
start returning erroneous results on August. Do you concur with this
analysis?

As for the locale support in PatternLayout, it is probably
overkill. It makes the code harder to understand and to maintain. I'd
wait for someone using a different numbering system to contact us
before adding doing it on our own.

Add TimeZone and locale for PatternLayout, remove obs DateFormat
http://issues.eu.apache.org/bugzilla/show_bug.cgi?id=32064

This last one I need to review after the recent discussions on jakarta-commons. It may be more appropriate to specify locale on the appender instead of the layout.


--
Ceki Gülcü

  The complete log4j manual: http://qos.ch/log4j/



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to