Re: [Nautilus-list] eel memleak patch

On Wednesday, August 22, 2001, at 11:33  PM, Anders Carlsson wrote:

here's another patch that fixes some memleaks in eel.

Looks good. Please feel free to commit (both branches if possible) after fixing one thing:

Index: eel/eel-font-manager.c
RCS file: /cvs/gnome/eel/eel/eel-font-manager.c,v
retrieving revision
diff -u -r1.13.2.3 eel-font-manager.c
--- eel/eel-font-manager.c	2001/08/14 18:35:51
+++ eel/eel-font-manager.c	2001/08/23 06:30:47
@@ -1467,6 +1467,7 @@
EEL_CHECK_STRING_RESULT (font_description_get_char_set (description), "iso8859-1");

 	font_table_list_free (font_table_list);
+	g_free (original_current_dir);

I suggest simply removing the unused original_current_dir variable.

    -- Darin

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