From 6881aa5ca235ee0737c2a409ffab966a10e5971e Mon Sep 17 00:00:00 2001 From: Christian Hergert Date: Mon, 24 Feb 2020 22:36:27 +0000 Subject: [PATCH 2/3] clutter: avoid g_signal_emit_by_name() from ClutterActor g_signal_emit_by_name() is used to emit signals on ClutterContainer when actors are removed or added. It happens to do various interface lookups which are a bit unneccessary and can allocate memory. Simply using emission wrappers makes all of that go away. https://gitlab.gnome.org/GNOME/mutter/merge_requests/1083 --- clutter/clutter/cally/cally-actor.c | 5 +-- clutter/clutter/clutter-actor.c | 17 ++++++++-- clutter/clutter/clutter-actor.h | 5 ++- clutter/clutter/clutter-container-private.h | 36 +++++++++++++++++++++ clutter/clutter/clutter-container.c | 21 ++++++++++++ 5 files changed, 78 insertions(+), 6 deletions(-) create mode 100644 clutter/clutter/clutter-container-private.h diff --git a/clutter/clutter/cally/cally-actor.c b/clutter/clutter/cally/cally-actor.c index 548615f48..517969625 100644 --- a/clutter/clutter/cally/cally-actor.c +++ b/clutter/clutter/cally/cally-actor.c @@ -604,10 +604,11 @@ cally_actor_real_remove_actor (ClutterActor *container, g_return_val_if_fail (CLUTTER_IS_ACTOR (actor), 0); atk_parent = ATK_OBJECT (data); - atk_child = clutter_actor_get_accessible (actor); - if (atk_child) + if (clutter_actor_has_accessible (actor)) { + atk_child = clutter_actor_get_accessible (actor); + g_value_init (&values.old_value, G_TYPE_POINTER); g_value_set_pointer (&values.old_value, atk_parent); diff --git a/clutter/clutter/clutter-actor.c b/clutter/clutter/clutter-actor.c index 803f76aae..93d0a93ef 100644 --- a/clutter/clutter/clutter-actor.c +++ b/clutter/clutter/clutter-actor.c @@ -624,7 +624,7 @@ #include "clutter-color-static.h" #include "clutter-color.h" #include "clutter-constraint-private.h" -#include "clutter-container.h" +#include "clutter-container-private.h" #include "clutter-content-private.h" #include "clutter-debug.h" #include "clutter-easing.h" @@ -4372,7 +4372,7 @@ clutter_actor_remove_child_internal (ClutterActor *self, /* we need to emit the signal before dropping the reference */ if (emit_actor_removed) - g_signal_emit_by_name (self, "actor-removed", child); + _clutter_container_emit_actor_removed (CLUTTER_CONTAINER (self), child); if (notify_first_last) { @@ -13060,7 +13060,7 @@ clutter_actor_add_child_internal (ClutterActor *self, } if (emit_actor_added) - g_signal_emit_by_name (self, "actor-added", child); + _clutter_container_emit_actor_added (CLUTTER_CONTAINER (self), child); if (notify_first_last) { @@ -21448,3 +21448,14 @@ clutter_actor_create_texture_paint_node (ClutterActor *self, return node; } + +gboolean +clutter_actor_has_accessible (ClutterActor *actor) +{ + g_return_val_if_fail (CLUTTER_IS_ACTOR (actor), FALSE); + + if (CLUTTER_ACTOR_GET_CLASS (actor)->has_accessible) + return CLUTTER_ACTOR_GET_CLASS (actor)->has_accessible (actor); + + return TRUE; +} diff --git a/clutter/clutter/clutter-actor.h b/clutter/clutter/clutter-actor.h index 7d2168af1..d286f2843 100644 --- a/clutter/clutter/clutter-actor.h +++ b/clutter/clutter/clutter-actor.h @@ -296,10 +296,11 @@ struct _ClutterActorClass gboolean (* touch_event) (ClutterActor *self, ClutterTouchEvent *event); + gboolean (* has_accessible) (ClutterActor *self); /*< private >*/ /* padding for future expansion */ - gpointer _padding_dummy[26]; + gpointer _padding_dummy[25]; }; /** @@ -369,6 +370,8 @@ CLUTTER_EXPORT const gchar * clutter_actor_get_name (ClutterActor *self); CLUTTER_EXPORT AtkObject * clutter_actor_get_accessible (ClutterActor *self); +CLUTTER_EXPORT +gboolean clutter_actor_has_accessible (ClutterActor *self); CLUTTER_EXPORT gboolean clutter_actor_is_visible (ClutterActor *self); diff --git a/clutter/clutter/clutter-container-private.h b/clutter/clutter/clutter-container-private.h new file mode 100644 index 000000000..d619a6531 --- /dev/null +++ b/clutter/clutter/clutter-container-private.h @@ -0,0 +1,36 @@ +/* + * Clutter. + * + * An OpenGL based 'interactive canvas' library. + * + * Copyright 2020 Red Hat, Inc. + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library. If not, see . + */ + +#ifndef __CLUTTER_CONTAINER_PRIVATE_H__ +#define __CLUTTER_CONTAINER_PRIVATE_H__ + +#include + +G_BEGIN_DECLS + +void _clutter_container_emit_actor_added (ClutterContainer *container, + ClutterActor *actor); +void _clutter_container_emit_actor_removed (ClutterContainer *container, + ClutterActor *actor); + +G_END_DECLS + +#endif /* __CLUTTER_CONTAINER_PRIVATE_H__ */ diff --git a/clutter/clutter/clutter-container.c b/clutter/clutter/clutter-container.c index 0f611ae55..79abb5b4f 100644 --- a/clutter/clutter/clutter-container.c +++ b/clutter/clutter/clutter-container.c @@ -37,6 +37,7 @@ #include "clutter-actor-private.h" #include "clutter-child-meta.h" +#include "clutter-container-private.h" #include "clutter-debug.h" #include "clutter-main.h" #include "clutter-marshal.h" @@ -1446,3 +1447,23 @@ clutter_container_child_notify (ClutterContainer *container, child, pspec); } + +void +_clutter_container_emit_actor_added (ClutterContainer *container, + ClutterActor *actor) +{ + g_return_if_fail (CLUTTER_IS_CONTAINER (container)); + g_return_if_fail (CLUTTER_IS_ACTOR (actor)); + + g_signal_emit (container, container_signals[ACTOR_ADDED], 0, actor); +} + +void +_clutter_container_emit_actor_removed (ClutterContainer *container, + ClutterActor *actor) +{ + g_return_if_fail (CLUTTER_IS_CONTAINER (container)); + g_return_if_fail (CLUTTER_IS_ACTOR (actor)); + + g_signal_emit (container, container_signals[ACTOR_REMOVED], 0, actor); +} -- 2.26.0