Here's a patch for a small regression in GregorianCalendar, spotted my
Mark.

2005-02-17  Sven de Marothy  <[EMAIL PROTECTED]>

        * java/util/GregorianCalendar.java      
        (add): Don't set fields directly anymore. Use set()

Index: java/util/GregorianCalendar.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/util/GregorianCalendar.java,v
retrieving revision 1.38
diff -u -r1.38 GregorianCalendar.java
--- java/util/GregorianCalendar.java	14 Feb 2005 16:39:03 -0000	1.38
+++ java/util/GregorianCalendar.java	17 Feb 2005 18:54:47 -0000
@@ -923,13 +923,13 @@
 	    fields[MONTH] += 12;
 	    fields[YEAR]--;
 	  }
-	isTimeSet = false;
 	int maxDay = getActualMaximum(DAY_OF_MONTH);
 	if (fields[DAY_OF_MONTH] > maxDay)
 	  {
 	    fields[DAY_OF_MONTH] = maxDay;
-	    isTimeSet = false;
 	  }
+	set(YEAR, fields[YEAR]);
+	set(MONTH, fields[MONTH]);
 	break;
       case DAY_OF_MONTH:
       case DAY_OF_YEAR:
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to