[balsa/gtk3] Just destroy the children, no need to recurse



commit 1df8158dfd39a306506036fae6e186bbae9c6eea
Author: Peter Bloomfield <PeterBloomfield bellsouth net>
Date:   Sat Feb 5 12:14:26 2011 -0500

    Just destroy the children, no need to recurse
    
    	* src/balsa-message.c (hide_all_parts): just destroy the
    	children, no need to recurse deeper.

 ChangeLog           |    7 ++++++-
 src/balsa-message.c |   11 +----------
 2 files changed, 7 insertions(+), 11 deletions(-)
---
diff --git a/ChangeLog b/ChangeLog
index 4a5d0f8..ad7f054 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,11 +1,16 @@
 2011-02-05  Peter Bloomfield
 
+	* src/balsa-message.c (hide_all_parts): just destroy the
+	children, no need to recurse deeper.
+
+2011-02-05  Peter Bloomfield
+
 	* src/balsa-mblist.c (bmbl_init): remove redundant and crashing
 	column-set-visible.
 
 2011-02-05  Peter Bloomfield
 
-	* configure.in: GtkHTML-4 is the new GtkHtml-3.
+	* configure.in: GtkHtml-4 is the new GtkHtml-3.
 	* libbalsa/html.c (libbalsa_html_new), (lbh_size_request_cb):
 	ditto; do not connect to "size-request" signal.
 
diff --git a/src/balsa-message.c b/src/balsa-message.c
index e63e1d9..5bb9dc4 100644
--- a/src/balsa-message.c
+++ b/src/balsa-message.c
@@ -2205,15 +2205,6 @@ gtk_tree_hide_func(GtkTreeModel * model, GtkTreePath * path,
 }
 
 static void
-bm_hide_all_helper(GtkWidget * widget, gpointer data)
-{
-    if (GTK_IS_CONTAINER(widget))
-	gtk_container_foreach(GTK_CONTAINER(widget), bm_hide_all_helper,
-			      NULL);
-    gtk_widget_destroy(widget);
-}
-
-static void
 hide_all_parts(BalsaMessage * bm)
 {
     if (bm->current_part) {
@@ -2227,7 +2218,7 @@ hide_all_parts(BalsaMessage * bm)
     }
 
     gtk_container_foreach(GTK_CONTAINER(bm->bm_widget->container),
-			  bm_hide_all_helper, NULL);
+                          (GtkCallback) gtk_widget_destroy, NULL);
 }
 
 /*



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