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=77530

--- shadow/77530        2006-02-21 07:36:46.000000000 -0500
+++ shadow/77530.tmp.11145      2006-02-21 12:18:02.000000000 -0500
@@ -191,6 +191,19 @@
 ------- Additional Comments From [EMAIL PROTECTED]  2006-02-21 07:36 -------
 Hi. I don't see the fix for 3).
 
 + Write ("return FromEnum ((long) val, new string[] {...
 
 This will create a new string[] each time it is called.
+
+------- Additional Comments From [EMAIL PROTECTED]  2006-02-21 12:18 -------
+You're correct, the string[] itself is created each time, but it is 
+created from an array that is cached.
+
+Do you really think we should cache the string (meaning new string[] 
+{ "EnumConstantName1", "EnumConstantName2", ... }) ? Isn't it enough 
+to cache the string[] that is used to construct the code, instead of 
+the code itself ?
+
+What's your opinion on the usefulness of the three arrays (for enum 
+names, xml names, values) I added on EnumMap ? Do you think its ok to 
+initialize these in the ctor, instead of using lazy init ?
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to