[Mono-bugs] [Bug 74521][Min] Changed - [monodis] escapes Array definitions..

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Mon, 18 Apr 2005 15:55:34 -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 rharinath@novell.com.

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

--- shadow/74521	2005-04-16 12:32:11.000000000 -0400
+++ shadow/74521.tmp.14788	2005-04-18 15:55:34.000000000 -0400
@@ -55,6 +55,23 @@
 Monodis should follow one of the above approaches by default and maybe
 have the other available as an option.
 So, the patch improves the conformance to the first approach.
 
 ------- Additional Comments From miguel@ximian.com  2005-04-16 12:32 -------
 Thanks.  This patch can be applied.
+
+------- Additional Comments From rharinath@novell.com  2005-04-18 15:55 -------
+Ankit and I discussed it, and deduced that it probably is a monodis
+bug.  The quoting of Foo[,] should be
+
+  'Foo'[,]
+
+not
+
+  'Foo[,]'
+
+From what I could make out, the latter is an identifier named
+'Foo[,]', quoted to prevent the special meaning of '['.
+
+The original patch is incomplete, and I think we should instead fix
+'get_escaped_name' to recurse down the ElementType before printing the
+escaped name.