From d15a54fc30933f62728b3b3e15fc3175ea0ef501 Mon Sep 17 00:00:00 2001 From: Harish Krishnaswamy Date: Mon, 20 Nov 2006 16:41:45 +0000 Subject: Remove unreachable code and fix the leak of basename. 2006-11-20 Harish Krishnaswamy * e-icon-factory.c: (load_icon), (e_icon_factory_get_icon): Remove unreachable code and fix the leak of basename. svn path=/trunk/; revision=33007 --- e-util/e-icon-factory.c | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) (limited to 'e-util/e-icon-factory.c') diff --git a/e-util/e-icon-factory.c b/e-util/e-icon-factory.c index 2a5cfb35f5..5471b5adab 100644 --- a/e-util/e-icon-factory.c +++ b/e-util/e-icon-factory.c @@ -124,6 +124,7 @@ load_icon (const char *icon_key, const char *icon_name, int size, int scale) dent, basename, NULL); + g_free (basename); if ((unscaled = gdk_pixbuf_new_from_file (filename, NULL))) break; } @@ -308,18 +309,8 @@ e_icon_factory_get_icon (const char *icon_name, int icon_size) pthread_mutex_lock (&lock); if (!(icon = g_hash_table_lookup (name_to_icon, icon_key))) { - if (!(icon = load_icon (icon_key, icon_name, size, TRUE))) { - g_warning ("Icon not found -- %s", icon_name); - - /* Create an empty icon so that we don't keep spitting - out the same warning over and over, every time this - icon is requested. */ - - icon = icon_new (icon_key, NULL); - g_hash_table_insert (name_to_icon, icon->name, icon); - } else { - g_hash_table_insert (name_to_icon, icon->name, icon); - } + icon = load_icon (icon_key, icon_name, size, TRUE); + g_hash_table_insert (name_to_icon, icon->name, icon); } if ((pixbuf = icon->pixbuf)) { -- cgit v1.2.3