[Mono-bugs] [Bug 58256][Nor] Changed - [PATCH] Replace Doesn't Replace ' Correctly

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Tue, 1 Jun 2004 08:17:37 -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 kjambunathan@novell.com.

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

--- shadow/58256	2004-06-01 08:09:32.000000000 -0400
+++ shadow/58256.tmp.25280	2004-06-01 08:17:37.000000000 -0400
@@ -123,6 +123,24 @@
 
 
 ------- Additional Comments From kjambunathan@novell.com  2004-06-01 08:09 -------
 Created an attachment (id=7954)
 Complete Patch & Some Test Cases
 
+
+------- Additional Comments From kjambunathan@novell.com  2004-06-01 08:17 -------
+Some Comments about my consolidated patch:
+
+1) MS .NET doesn't honour the escape chars in replacement patterns, as
+documented by SDK. But we behave correctly according to SDK. It's a
+moot point whether the reported problem need be fixed. Currently
+leaving this behaviour of us untouched.
+
+2) Mono's Regex.Unescape() was broken; Fixed the same and submitted a
+test case.
+
+3) We weren't handling \cA - \cZ correctly. It has been fixed; Refer 
+http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vsintro7/html/_pluslang_ascii_character_codes_chart_1.asp
+for the ASCII map.
+
+4) \b is handled differently in depending upon the context in which it
+is used in a Regex. Included a mini test case to exercise the same.