[Gtk-sharp-list] PATCH: speed up treeview and managed values
John Luke
jluke@users.sourceforge.net
Sun, 15 Feb 2004 02:53:27 -0500
On Sun, 2004-02-15 at 01:05, Ben Maurer wrote:
> Hello,
>
> The patch is at:
> http://devservices.go-mono.com/~benm/patches/gtksharp-treemodel-managedvalue.patch
>
> Overall, this made a noticable difference when loading a large project
> (in this case, the MCS compiler) in MonoDevelop. The time improved and
> memory was down by a few MB (according to top).
>
A few comments after glancing over the patch:
- I believe it is still considered appropriate to actually attach a
patch, at least so it is preserved and not lost.
- The rest of Gtk# and GTK+ is LGPL licensed why did you submit these
with GPL headers.
- I'm not crazy about special casing certain things to C code solely for
performance unless it is very necessary. Hopefully you looked into the
future maintainability, considered the NodeStore implementation, and GTK
2.4 changes.