permission to commit patch for #168679 ?



I patched this bug today; the patch does change memory management somewhat, so perhaps is not risk-free, however it does seem to be correct and is in response to a significant panel/applet leak.

The bug:
http://bugzilla.gnome.org/show_bug.cgi?id=168679

The patch:
http://bugzilla.gnome.org/attachment.cgi?id=38093&action=view

comment from Federico:

/------- Additional Comment #13 <http://bugzilla.gnome.org/show_bug.cgi?id=168679#c13> From Federico Mena Quintero <mailto:federico ximian com> 2005-03-01 12:54 ------- /

The patch is fine; indeed, since the "any" is not being allocated from a
skeleton, its value doesn't get freed.  The patch should go on CVS.


Comments? I'm happy to commit and re-dist if you guys give the (modern, not Roman) thumbs-up.

- Bill



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]