From 27f0fdd18fe567ca38f0f66d5005c9768b3b8067 Mon Sep 17 00:00:00 2001 From: Matthias Clasen Date: Wed, 29 Sep 2021 01:38:43 -0400 Subject: [PATCH] Don't inline deprecation warnings The code that emits property deprecation warnings rarely run, and doesn't need to be inlined in all our property setters. --- gobject/gobject.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/gobject/gobject.c b/gobject/gobject.c index d16f27cbb..d02200b44 100644 --- a/gobject/gobject.c +++ b/gobject/gobject.c @@ -1490,16 +1490,13 @@ g_object_thaw_notify (GObject *object) } static void -consider_issuing_property_deprecation_warning (const GParamSpec *pspec) +maybe_issue_property_deprecation_warning (const GParamSpec *pspec) { static GHashTable *already_warned_table; static const gchar *enable_diagnostic; static GMutex already_warned_lock; gboolean already; - if (!(pspec->flags & G_PARAM_DEPRECATED)) - return; - if (g_once_init_enter (&enable_diagnostic)) { const gchar *value = g_getenv ("G_ENABLE_DIAGNOSTIC"); @@ -1539,6 +1536,13 @@ consider_issuing_property_deprecation_warning (const GParamSpec *pspec) g_type_name (pspec->owner_type), pspec->name); } +static inline void +consider_issuing_property_deprecation_warning (const GParamSpec *pspec) +{ + if (G_UNLIKELY (pspec->flags & G_PARAM_DEPRECATED)) + maybe_issue_property_deprecation_warning (pspec); +} + static inline void object_get_property (GObject *object, GParamSpec *pspec,