In message "[kaffe] Bug report: java.text.SimpleDateFormat#parse" on 05/01/25, Ito Kazumitsu <[EMAIL PROTECTED]> writes:
> You might have found that test/regression/DateFormatTest.java fails > recently. > > This is becuase of a bug in java.text.SimpleDateFormat#parse. And this is my patch. I am afraid whether or not this patch is applied, there may be cases where java.text.SimpleDateFormat#parse does not handle DST correctly, but this patch will work fine at least for those time zones which does not have DST. --- java/text/SimpleDateFormat.java.orig Tue Jan 25 14:50:18 2005 +++ java/text/SimpleDateFormat.java Tue Jan 25 16:35:03 2005 @@ -840,7 +840,7 @@ // We need a special case for the timezone, because it // uses a different data structure than the other cases. is_numeric = false; - calendar_field = Calendar.DST_OFFSET; + calendar_field = Calendar.ZONE_OFFSET; String[][] zoneStrings = formatData.getZoneStrings(); int zoneCount = zoneStrings.length; int index = pos.getIndex(); @@ -868,8 +868,8 @@ found_zone = true; saw_timezone = true; TimeZone tz = TimeZone.getTimeZone (strings[0]); - calendar.set (Calendar.ZONE_OFFSET, tz.getRawOffset ()); - offset = tz.getDSTSavings(); + calendar.set (Calendar.DST_OFFSET, tz.getDSTSavings()); + offset = tz.getRawOffset (); pos.setIndex(index + strings[k].length()); break; } _______________________________________________ kaffe mailing list kaffe@kaffe.org http://kaffe.org/cgi-bin/mailman/listinfo/kaffe