luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2024-12-25 15:06:14 +01:00
Code Issues Packages Projects Releases Wiki Activity

reimplemented floating flag handling by atomically accessing a reserved

Browse Source 
Mon Sep 11 12:12:45 2006  Tim Janik  <timj@imendio.com>

        * gparam.c (g_param_spec_sink): reimplemented floating flag handling by
        atomically accessing a reserved GData bit, fixes #330556.
This commit is contained in:
Tim Janik 2006-09-11 10:14:05 +00:00 committed by Tim Janik
parent e9da546b81
commit 831cda3ca7
2 changed files with 14 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
gobject/ChangeLog
View File

@ -1,3 +1,8 @@
Mon Sep 11 12:12:45 2006 Tim Janik <timj@imendio.com>
* gparam.c (g_param_spec_sink): reimplemented floating flag handling by
atomically accessing a reserved GData bit, fixes #330556.
Tue Aug 29 13:27:33 2006 Tim Janik <timj@gtk.org> Tue Aug 29 13:27:33 2006 Tim Janik <timj@gtk.org>
* glib-mkenums.in: fixed to print usage and version info on STDOUT * glib-mkenums.in: fixed to print usage and version info on STDOUT

17
gobject/gparam.c
View File

@ -31,6 +31,7 @@
/* --- defines --- */ /* --- defines --- */
#define PARAM_FLOATING_FLAG 0x2
#define G_PARAM_USER_MASK (~0 << G_PARAM_USER_SHIFT) #define G_PARAM_USER_MASK (~0 << G_PARAM_USER_SHIFT)
#define PSPEC_APPLIES_TO_VALUE(pspec, value) (G_TYPE_CHECK_VALUE_TYPE ((value), G_PARAM_SPEC_VALUE_TYPE (pspec))) #define PSPEC_APPLIES_TO_VALUE(pspec, value) (G_TYPE_CHECK_VALUE_TYPE ((value), G_PARAM_SPEC_VALUE_TYPE (pspec)))
#define G_SLOCK(mutex) g_static_mutex_lock (mutex) #define G_SLOCK(mutex) g_static_mutex_lock (mutex)
@ -62,10 +63,6 @@ static gchar* value_param_lcopy_value (const GValue *value,
guint collect_flags); guint collect_flags);
/* --- variables --- */
static GQuark quark_floating = 0;
/* --- functions --- */ /* --- functions --- */
void void
g_param_type_init (void) g_param_type_init (void)
@ -122,8 +119,6 @@ static void
g_param_spec_class_init (GParamSpecClass *class, g_param_spec_class_init (GParamSpecClass *class,
gpointer class_data) gpointer class_data)
{ {
quark_floating = g_quark_from_static_string ("GParamSpec-floating");
class->value_type = G_TYPE_NONE; class->value_type = G_TYPE_NONE;
class->finalize = g_param_spec_finalize; class->finalize = g_param_spec_finalize;
class->value_set_default = NULL; class->value_set_default = NULL;
@ -142,9 +137,10 @@ g_param_spec_init (GParamSpec *pspec,
pspec->value_type = class->value_type; pspec->value_type = class->value_type;
pspec->owner_type = 0; pspec->owner_type = 0;
pspec->qdata = NULL; pspec->qdata = NULL;
g_datalist_init (&pspec->qdata);
g_datalist_set_flags (&pspec->qdata, PARAM_FLOATING_FLAG);
pspec->ref_count = 1; pspec->ref_count = 1;
pspec->param_id = 0; pspec->param_id = 0;
g_datalist_id_set_data (&pspec->qdata, quark_floating, GUINT_TO_POINTER (TRUE));
} }
static void static void
@ -194,10 +190,15 @@ g_param_spec_unref (GParamSpec *pspec)
void void
g_param_spec_sink (GParamSpec *pspec) g_param_spec_sink (GParamSpec *pspec)
{ {
gpointer oldvalue;
g_return_if_fail (G_IS_PARAM_SPEC (pspec)); g_return_if_fail (G_IS_PARAM_SPEC (pspec));
g_return_if_fail (pspec->ref_count > 0); g_return_if_fail (pspec->ref_count > 0);
if (g_datalist_id_remove_no_notify (&pspec->qdata, quark_floating)) do
oldvalue = g_atomic_pointer_get (&pspec->qdata);
while (!g_atomic_pointer_compare_and_exchange ((void**) &pspec->qdata, oldvalue,
(gpointer) ((gsize) oldvalue & ~(gsize) PARAM_FLOATING_FLAG)));
if ((gsize) oldvalue & PARAM_FLOATING_FLAG)
g_param_spec_unref (pspec); g_param_spec_unref (pspec);
} }