[gnome-shell] Remove dangerous explicit destroy
- From: Cosimo Cecchi <cosimoc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] Remove dangerous explicit destroy
- Date: Tue, 6 Jun 2017 19:40:04 +0000 (UTC)
commit ad2cb227858effa57ce977c77744c9e0781ec1a3
Author: Cosimo Cecchi <cosimo endlessm com>
Date: Wed Jun 5 13:54:09 2013 -0700
Remove dangerous explicit destroy
There's no need to explicitly destroy the ClutterText actor inside the
label; doing so is actually harmful, as it will break the normal
reference cycle between container and children.
As StLabel doesn't hold any extra reference to the ClutterText actor and
just uses clutter_actor_add_actor() to add it to itself, let the normal
container dispose cycle run to dispose of the reference.
https://bugzilla.gnome.org/show_bug.cgi?id=783483
src/st/st-label.c | 6 ------
1 files changed, 0 insertions(+), 6 deletions(-)
---
diff --git a/src/st/st-label.c b/src/st/st-label.c
index 95ba895..a7893b0 100644
--- a/src/st/st-label.c
+++ b/src/st/st-label.c
@@ -180,12 +180,6 @@ st_label_dispose (GObject *object)
{
StLabelPrivate *priv = ST_LABEL (object)->priv;
- if (priv->label)
- {
- clutter_actor_destroy (priv->label);
- priv->label = NULL;
- }
-
g_clear_pointer (&priv->text_shadow_pipeline, cogl_object_unref);
G_OBJECT_CLASS (st_label_parent_class)->dispose (object);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]