https://bugzilla.novell.com/show_bug.cgi?id=420312
User [EMAIL PROTECTED] added comment https://bugzilla.novell.com/show_bug.cgi?id=420312#c2 Andy Hume <[EMAIL PROTECTED]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|[EMAIL PROTECTED] | --- Comment #2 from Andy Hume <[EMAIL PROTECTED]> 2008-09-29 09:34:35 MDT --- Created an attachment (id=242314) --> (https://bugzilla.novell.com/attachment.cgi?id=242314) Patch #2 Redone. Hopefully this is as you imagined. As before, still has debug logging and not quite per coding standards. The checking for well-known assemblies seems a bit odd but there seems not other simple way to get and compare the PublicKeyToken with its well-known values... Note that when used by PropertyGrid the debug logging from the constructor is seen with _every_ scroll of the PG (when showing a object with such an item collection); which shows that PG is recreating the editor for every action (when scrolling: to call PaintValue). I guess this falls into need for more caching that you had highlighted earlier. For this case it seems that the editor should only be instantiated *once* by the GridEntry -- its not something that changes after the object is first loaded. So GetEditor should store the value and return it, something like: .. bool doneGetEditor; UITypeEditor _editor; private UITypeEditor GetEditor () { if (doneGetEditor) return _editor; doneGetEditor = true; .. Similarly for GetConverter? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. _______________________________________________ mono-bugs maillist - mono-bugs@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-bugs