[Mono-bugs] [Bug 40910][Maj] Changed - Bad "Z" mask parsing in DateTime

bugzilla-daemon@rocky.ximian.com bugzilla-daemon@rocky.ximian.com
Fri, 11 Apr 2003 19:36:18 -0400 (EDT)


Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by gonzalo@ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=40910

--- shadow/40910	Fri Apr 11 00:35:13 2003
+++ shadow/40910.tmp.18909	Fri Apr 11 19:36:18 2003
@@ -1,13 +1,13 @@
 Bug#: 40910
 Product: Mono/Class Libraries
 Version: unspecified
 OS: All
 OS Details: 
-Status: REOPENED   
-Resolution: 
+Status: RESOLVED   
+Resolution: FIXED
 Severity: Unknown
 Priority: Major
 Component: CORLIB
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: spouliot@videotron.ca               
 QAContact: mono-bugs@ximian.com
@@ -155,6 +155,24 @@
 Can someone do regression testing on that patch? My shell is not 
 working yet ;-).
 
 Also, the specs are very unclear on how to handle 'Z' vs 'z', sp 
 tells me that it works either way on Windows/MS Runtime. I think we 
 should just emulate Microsoft on this.
+
+------- Additional Comments From gonzalo@ximian.com  2003-04-11 19:36 -------
+That patch does not work (it also has a typo ;-).
+
+I run that program on windows and get:
+----------------
+~ $ ./dtsp.exe
+Input: 20030407210500Z  Mask: yyyyMMddHHmmssZ
+ParseExact (localtime) 07/04/2003 23:05:00
+ToUniversalTime() 07/04/2003 21:05:00
+----------------
+
+(notice that my timezone is GMT+2).
+
+I have a 1 line patch that fixes this the I didn't apply last night
+because some unit test were failing. It turns out that those tests
+also fail without my little patch, so I'll apply it.
+