[gnome-online-accounts] provider: Drop the priv pointer
- From: Debarshi Ray <debarshir src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-online-accounts] provider: Drop the priv pointer
- Date: Tue, 7 Feb 2017 15:57:45 +0000 (UTC)
commit 24215a45a7858c6ce9a059b3fb001ac4719acbc1
Author: Debarshi Ray <debarshir gnome org>
Date: Tue Feb 7 16:35:18 2017 +0100
provider: Drop the priv pointer
The current GObject recommendation is to use the generated
get_instance_private function instead of a separate priv pointer in the
instance struct. This saves one pointer per class in the hierarchy
multiplied by the number of instances of the type, and the function is
fast enough because it only does pointer arithmetic.
src/goabackend/goaprovider-priv.h | 1 -
src/goabackend/goaprovider.c | 24 ++++++++++++++++++------
2 files changed, 18 insertions(+), 7 deletions(-)
---
diff --git a/src/goabackend/goaprovider-priv.h b/src/goabackend/goaprovider-priv.h
index 9bdcbcd..078618f 100644
--- a/src/goabackend/goaprovider-priv.h
+++ b/src/goabackend/goaprovider-priv.h
@@ -40,7 +40,6 @@ struct _GoaProvider
{
/*< private >*/
GObject parent_instance;
- GoaProviderPrivate *priv;
};
/**
diff --git a/src/goabackend/goaprovider.c b/src/goabackend/goaprovider.c
index 478d32a..5242e49 100644
--- a/src/goabackend/goaprovider.c
+++ b/src/goabackend/goaprovider.c
@@ -186,10 +186,13 @@ goa_provider_get_property (GObject *object,
GParamSpec *pspec)
{
GoaProvider *self = GOA_PROVIDER (object);
+ GoaProviderPrivate *priv;
+
+ priv = goa_provider_get_instance_private (self);
switch (property_id) {
case PROP_PRESEED_DATA:
- g_value_set_variant (value, self->priv->preseed_data);
+ g_value_set_variant (value, priv->preseed_data);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec);
@@ -219,8 +222,11 @@ static void
goa_provider_dispose (GObject *object)
{
GoaProvider *self = GOA_PROVIDER (object);
+ GoaProviderPrivate *priv;
+
+ priv = goa_provider_get_instance_private (self);
- g_clear_pointer (&self->priv->preseed_data, g_variant_unref);
+ g_clear_pointer (&priv->preseed_data, g_variant_unref);
G_OBJECT_CLASS (goa_provider_parent_class)->dispose (object);
}
@@ -228,7 +234,6 @@ goa_provider_dispose (GObject *object)
static void
goa_provider_init (GoaProvider *self)
{
- self->priv = goa_provider_get_instance_private (self);
}
static void
@@ -1397,9 +1402,13 @@ void
goa_provider_set_preseed_data (GoaProvider *self,
GVariant *preseed_data)
{
- g_clear_pointer (&self->priv->preseed_data, g_variant_unref);
+ GoaProviderPrivate *priv;
+
+ priv = goa_provider_get_instance_private (self);
+
+ g_clear_pointer (&priv->preseed_data, g_variant_unref);
if (preseed_data != NULL)
- self->priv->preseed_data = g_variant_ref_sink (preseed_data);
+ priv->preseed_data = g_variant_ref_sink (preseed_data);
g_object_notify (G_OBJECT (self), "preseed-data");
}
@@ -1415,7 +1424,10 @@ goa_provider_set_preseed_data (GoaProvider *self,
GVariant *
goa_provider_get_preseed_data (GoaProvider *self)
{
- return self->priv->preseed_data;
+ GoaProviderPrivate *priv;
+
+ priv = goa_provider_get_instance_private (self);
+ return priv->preseed_data;
}
/* ---------------------------------------------------------------------------------------------------- */
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]