[cp-patches] FYI: GregorianCalendar tweak

Gary Benson gbenson at redhat.com
Wed Apr 4 15:34:57 UTC 2007


Hi all,

This patch removes a redundant call to complete() in one of
java.util.GregorianCalendar's constructors.  setTimeInMillis()
fills in both the time and the fields.

Cheers,
Gary
-------------- next part --------------
Index: ChangeLog
===================================================================
RCS file: /cvsroot/classpath/classpath/ChangeLog,v
retrieving revision 1.9208
diff -u -r1.9208 ChangeLog
--- ChangeLog	4 Apr 2007 11:38:46 -0000	1.9208
+++ ChangeLog	4 Apr 2007 15:31:36 -0000
@@ -1,3 +1,8 @@
+2007-04-04  Gary Benson  <gbenson at redhat.com>
+
+	* java/util/GregorianCalendar.java
+	(GregorianCalendar(TimeZone, Locale)): Remove redundant complete().
+
 2007-04-04  Roman Kennke  <roman at kennke.org>
 
 	* java/nio/channels/spi/SelectorProvider.java
Index: java/util/GregorianCalendar.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/util/GregorianCalendar.java,v
retrieving revision 1.47
diff -u -r1.47 GregorianCalendar.java
--- java/util/GregorianCalendar.java	23 Dec 2006 22:32:59 -0000	1.47
+++ java/util/GregorianCalendar.java	4 Apr 2007 15:31:36 -0000
@@ -223,7 +223,6 @@
   {
     this(zone, locale, false);
     setTimeInMillis(System.currentTimeMillis());
-    complete();
   }
 
   /**


More information about the Classpath-patches mailing list