commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: svn commit: r628021 - in /commons/proper/configuration/branches/configuration2_experimental: src/main/java/org/apache/commons/configuration2/plist/ src/test/java/org/apache/commons/configuration2/plist/ xdocs/
Date Fri, 15 Feb 2008 13:23:48 GMT
In 15/02/2008, ebourg@apache.org <ebourg@apache.org> wrote:
> Author: ebourg
>  Date: Fri Feb 15 03:31:07 2008
>  New Revision: 628021
>
>  URL: http://svn.apache.org/viewvc?rev=628021&view=rev
>  Log:
>  The calendar objects are now formatted with their own time zone
>  Removed the Java 1.3 workaround in PropertyListConfiguration to parse and format the
dates without a SimpleDateFormat
>

BTW, SimpleDateFormat is not synchronized, so having package-protected
instances (whether static or not) may cause problems if the class is
intended for multithreaded use.

Also, the serialVersionUID should probably be changed as a new
instance field has been added.

Perhaps consider using org.apache.commons.lang.time.FastDateFormat
from Commons Lang for at least the formatting part?

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message