Re: [Nautilus-list] eel memleak patch

[Darin Adler <darin bentspoon com>]

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 1.13.2.3
> diff -u -r1.13.2.3 eel-font-manager.c
> --- eel/eel-font-manager.c	2001/08/14 18:35:51	1.13.2.3
> +++ 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