[Xfce4-commits] <garcon:master> Plug another leak.

Nick Schermer noreply at xfce.org
Tue Feb 16 19:12:02 CET 2010


Updating branch refs/heads/master
         to 1dae4cdea34c3d9e7f0ed1d1d4d2958784f9061d (commit)
       from 12ec0729009c74537ce4142a74ffb325a108bd2a (commit)

commit 1dae4cdea34c3d9e7f0ed1d1d4d2958784f9061d
Author: Nick Schermer <nick at xfce.org>
Date:   Tue Feb 16 19:04:22 2010 +0100

    Plug another leak.
    
    Because the function had no return value, we never
    fully traversed the tree.

 garcon/garcon-menu-node.c |   11 ++++++++++-
 1 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/garcon/garcon-menu-node.c b/garcon/garcon-menu-node.c
index cbbd7de..51e8436 100644
--- a/garcon/garcon-menu-node.c
+++ b/garcon/garcon-menu-node.c
@@ -795,13 +795,22 @@ garcon_menu_node_tree_copy (GNode *tree)
 
 
 
+static gboolean
+free_children (GNode *tree)
+{
+  garcon_menu_node_tree_free_data (tree);
+  return FALSE;
+}
+
+
+
 void
 garcon_menu_node_tree_free (GNode *tree)
 {
   if (tree != NULL)
     {
       g_node_traverse (tree, G_IN_ORDER, G_TRAVERSE_ALL, -1,
-                       (GNodeTraverseFunc) garcon_menu_node_tree_free_data, NULL);
+                       (GNodeTraverseFunc) free_children, NULL);
 
       g_node_destroy (tree);
     }



More information about the Xfce4-commits mailing list