brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

conform: Remove unnecessary destroy() calls

Browse Source 
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.
This commit is contained in:
Emmanuele Bassi
2010-10-19 12:32:02 +01:00
parent b5bbdd6cec
commit ce3311df26
1 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tests/conform/test-script-parser.c
View File

@ -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);
}