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 [EMAIL PROTECTED]

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

--- shadow/77633        2007-05-28 16:55:24.000000000 -0400
+++ shadow/77633.tmp.11867      2007-05-29 08:58:38.000000000 -0400
@@ -63,6 +63,22 @@
 .NET 2.0 throws a FormatException
 
 ------- Additional Comments From [EMAIL PROTECTED]  2007-05-28 16:55 -------
 Created an attachment (id=19343)
 Proposal to fix 77633.  Throws the appropriate exception for the appropriate 
runtime version (2.0 versus 1.1) for DateTime.Parse()
 
+
+------- Additional Comments From [EMAIL PROTECTED]  2007-05-29 08:58 -------
+You could just wrap the entire test methods with
+#if NET_2_0
+
+test_only_for_2_0() { ... } 
+
+#else
+
+test_only_for_1_1() { ... }
+
+#endif
+
+Other than that the patch looks fine. (though I'm unlikely to use
+const for every exception message as we don't do that in other classes
+...)
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to