From ce3311df2683dae46b3b63a760ae87353f9f59b6 Mon Sep 17 00:00:00 2001 From: Emmanuele Bassi Date: Tue, 19 Oct 2010 12:32:02 +0100 Subject: [PATCH] conform: Remove unnecessary destroy() calls Unparented actors are owned by the Script instance, and if that goes away then the actors go away with it. The fact that we needed an explicit destroy() before was a hint of a memory management issue that I blissfully - and regretfully - ignored for the sake of a passing test suite. --- tests/conform/test-script-parser.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tests/conform/test-script-parser.c b/tests/conform/test-script-parser.c index c9301e2c4..c1195e717 100644 --- a/tests/conform/test-script-parser.c +++ b/tests/conform/test-script-parser.c @@ -162,7 +162,6 @@ test_script_child (TestConformSimpleFixture *fixture, g_assert (!focus_ret); g_object_unref (script); - clutter_actor_destroy (CLUTTER_ACTOR (container)); g_free (test_file); } @@ -201,8 +200,6 @@ test_script_single (TestConformSimpleFixture *fixture, g_assert_cmpint (color.alpha, ==, 0xff); g_object_unref (script); - - clutter_actor_destroy (rect); g_free (test_file); } @@ -273,7 +270,6 @@ test_script_object_property (TestConformSimpleFixture *fixture, g_assert (CLUTTER_IS_BIN_LAYOUT (manager)); g_object_unref (script); - clutter_actor_destroy (CLUTTER_ACTOR (actor)); g_free (test_file); } @@ -306,7 +302,6 @@ test_script_named_object (TestConformSimpleFixture *fixture, g_assert (clutter_box_layout_get_vertical (CLUTTER_BOX_LAYOUT (manager))); g_object_unref (script); - clutter_actor_destroy (CLUTTER_ACTOR (actor)); g_free (test_file); }