summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDebarshi Ray <debarshir@gnome.org>2018-10-06 17:52:19 +0200
committerDebarshi Ray <debarshir@gnome.org>2018-10-06 19:34:00 +0200
commit3f14c2b8c4c5f1816e3995acecab4f6a83ab46fd (patch)
treea0e53cede5a4f68deb6d39e98addfa3190f6de51
parent6d254dc701b8b29745edd54ff4e7643a8feeb8a1 (diff)
downloadlibgd-3f14c2b8c4c5f1816e3995acecab4f6a83ab46fd.tar.gz
toggle-pixbuf-renderer: 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. The saves one pointer per class in the heirarchy multiplied by the number of instances of the type, and the function is fast enough because it only does pointer arithmetic. https://bugzilla.gnome.org/show_bug.cgi?id=774709
-rw-r--r--libgd/gd-toggle-pixbuf-renderer.c32
-rw-r--r--libgd/gd-toggle-pixbuf-renderer.h2
2 files changed, 21 insertions, 13 deletions
diff --git a/libgd/gd-toggle-pixbuf-renderer.c b/libgd/gd-toggle-pixbuf-renderer.c
index 92de575..5d43d5e 100644
--- a/libgd/gd-toggle-pixbuf-renderer.c
+++ b/libgd/gd-toggle-pixbuf-renderer.c
@@ -48,13 +48,15 @@ render_check (GdTogglePixbufRenderer *self,
gint xpad,
gint ypad)
{
+ GdTogglePixbufRendererPrivate *priv;
GtkStyleContext *context;
gint check_x, check_y, x_offset;
GtkTextDirection direction;
+ priv = gd_toggle_pixbuf_renderer_get_instance_private (self);
context = gtk_widget_get_style_context (widget);
- if (!self->priv->toggle_visible)
+ if (!priv->toggle_visible)
return;
direction = gtk_widget_get_direction (widget);
@@ -69,7 +71,7 @@ render_check (GdTogglePixbufRenderer *self,
gtk_style_context_save (context);
gtk_style_context_add_class (context, GTK_STYLE_CLASS_CHECK);
- if (self->priv->active)
+ if (priv->active)
gtk_style_context_set_state (context, gtk_widget_get_state_flags (widget) | GTK_STATE_FLAG_CHECKED);
gtk_render_background (context, cr,
@@ -93,9 +95,12 @@ render_activity (GdTogglePixbufRenderer *self,
gint xpad,
gint ypad)
{
+ GdTogglePixbufRendererPrivate *priv;
gint x, y, width, height;
- if (self->priv->pulse == 0)
+ priv = gd_toggle_pixbuf_renderer_get_instance_private (self);
+
+ if (priv->pulse == 0)
return;
width = cell_area->width / 4;
@@ -110,7 +115,7 @@ render_activity (GdTogglePixbufRenderer *self,
GTK_STATE_FLAG_ACTIVE,
widget,
NULL,
- (guint) self->priv->pulse - 1,
+ (guint) priv->pulse - 1,
x, y,
width, height);
G_GNUC_END_IGNORE_DEPRECATIONS;
@@ -173,17 +178,20 @@ gd_toggle_pixbuf_renderer_get_property (GObject *object,
GParamSpec *pspec)
{
GdTogglePixbufRenderer *self = GD_TOGGLE_PIXBUF_RENDERER (object);
+ GdTogglePixbufRendererPrivate *priv;
+
+ priv = gd_toggle_pixbuf_renderer_get_instance_private (self);
switch (property_id)
{
case PROP_ACTIVE:
- g_value_set_boolean (value, self->priv->active);
+ g_value_set_boolean (value, priv->active);
break;
case PROP_TOGGLE_VISIBLE:
- g_value_set_boolean (value, self->priv->toggle_visible);
+ g_value_set_boolean (value, priv->toggle_visible);
break;
case PROP_PULSE:
- g_value_set_uint (value, self->priv->pulse);
+ g_value_set_uint (value, priv->pulse);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec);
@@ -198,17 +206,20 @@ gd_toggle_pixbuf_renderer_set_property (GObject *object,
GParamSpec *pspec)
{
GdTogglePixbufRenderer *self = GD_TOGGLE_PIXBUF_RENDERER (object);
+ GdTogglePixbufRendererPrivate *priv;
+
+ priv = gd_toggle_pixbuf_renderer_get_instance_private (self);
switch (property_id)
{
case PROP_ACTIVE:
- self->priv->active = g_value_get_boolean (value);
+ priv->active = g_value_get_boolean (value);
break;
case PROP_TOGGLE_VISIBLE:
- self->priv->toggle_visible = g_value_get_boolean (value);
+ priv->toggle_visible = g_value_get_boolean (value);
break;
case PROP_PULSE:
- self->priv->pulse = g_value_get_uint (value);
+ priv->pulse = g_value_get_uint (value);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec);
@@ -258,7 +269,6 @@ gd_toggle_pixbuf_renderer_class_init (GdTogglePixbufRendererClass *klass)
static void
gd_toggle_pixbuf_renderer_init (GdTogglePixbufRenderer *self)
{
- self->priv = gd_toggle_pixbuf_renderer_get_instance_private (self);
}
GtkCellRenderer *
diff --git a/libgd/gd-toggle-pixbuf-renderer.h b/libgd/gd-toggle-pixbuf-renderer.h
index fe54cf4..2a7531a 100644
--- a/libgd/gd-toggle-pixbuf-renderer.h
+++ b/libgd/gd-toggle-pixbuf-renderer.h
@@ -57,8 +57,6 @@ typedef struct _GdTogglePixbufRendererPrivate GdTogglePixbufRendererPrivate;
struct _GdTogglePixbufRenderer
{
GtkCellRendererPixbuf parent;
-
- GdTogglePixbufRendererPrivate *priv;
};
struct _GdTogglePixbufRendererClass