[gnome-photos] searchbar: Drop the priv pointer



commit 9eb0afc51e74b4a5c7af9f078ff44ea0d536cab2
Author: Debarshi Ray <debarshir gnome org>
Date:   Wed Apr 20 10:55:00 2016 +0200

    searchbar: 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.

 src/photos-searchbar.c |   63 ++++++++++++++++++++++++++++++++++++-----------
 src/photos-searchbar.h |    1 -
 2 files changed, 48 insertions(+), 16 deletions(-)
---
diff --git a/src/photos-searchbar.c b/src/photos-searchbar.c
index b9bbdcf..a0808a9 100644
--- a/src/photos-searchbar.c
+++ b/src/photos-searchbar.c
@@ -59,7 +59,11 @@ G_DEFINE_TYPE_WITH_PRIVATE (PhotosSearchbar, photos_searchbar, GTK_TYPE_REVEALER
 static void
 photos_searchbar_change_state (PhotosSearchbar *self, GVariant *value)
 {
-  g_simple_action_set_state (G_SIMPLE_ACTION (self->priv->search), value);
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
+
+  g_simple_action_set_state (G_SIMPLE_ACTION (priv->search), value);
 
   if (g_variant_get_boolean (value))
     photos_searchbar_show (self);
@@ -71,7 +75,9 @@ photos_searchbar_change_state (PhotosSearchbar *self, GVariant *value)
 static void
 photos_searchbar_default_hide (PhotosSearchbar *self)
 {
-  PhotosSearchbarPrivate *priv = self->priv;
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
 
   priv->in = FALSE;
   gtk_revealer_set_reveal_child (GTK_REVEALER (self), FALSE);
@@ -84,9 +90,11 @@ photos_searchbar_default_hide (PhotosSearchbar *self)
 static void
 photos_searchbar_default_show (PhotosSearchbar *self)
 {
-  PhotosSearchbarPrivate *priv = self->priv;
+  PhotosSearchbarPrivate *priv;
   GdkDevice *event_device;
 
+  priv = photos_searchbar_get_instance_private (self);
+
   event_device = gtk_get_current_event_device ();
   gtk_revealer_set_reveal_child (GTK_REVEALER (self), TRUE);
   priv->in = TRUE;
@@ -99,10 +107,13 @@ photos_searchbar_default_show (PhotosSearchbar *self)
 static void
 photos_searchbar_enable_search (PhotosSearchbar *self, gboolean enable)
 {
+  PhotosSearchbarPrivate *priv;
   GVariant *state;
 
+  priv = photos_searchbar_get_instance_private (self);
+
   state = g_variant_new ("b", enable);
-  g_action_change_state (self->priv->search, state);
+  g_action_change_state (priv->search, state);
 }
 
 
@@ -152,14 +163,19 @@ photos_is_tab_event (PhotosSearchbar *self, GdkEventKey *event)
 static void
 photos_searchbar_preedit_changed (PhotosSearchbar *self, const gchar *preedit)
 {
-  self->priv->preedit_changed = TRUE;
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
+  priv->preedit_changed = TRUE;
 }
 
 
 static void
 photos_searchbar_search_changed (PhotosSearchbar *self)
 {
-  PhotosSearchbarPrivate *priv = self->priv;
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
 
   if (priv->search_change_blocked)
     return;
@@ -172,11 +188,13 @@ static void
 photos_searchbar_constructed (GObject *object)
 {
   PhotosSearchbar *self = PHOTOS_SEARCHBAR (object);
-  PhotosSearchbarPrivate *priv = self->priv;
+  PhotosSearchbarPrivate *priv;
   GApplication *app;
   GtkToolItem *item;
   GVariant *state;
 
+  priv = photos_searchbar_get_instance_private (self);
+
   G_OBJECT_CLASS (photos_searchbar_parent_class)->constructed (object);
 
   PHOTOS_SEARCHBAR_GET_CLASS (self)->create_search_widgets (self);
@@ -215,7 +233,9 @@ static void
 photos_searchbar_dispose (GObject *object)
 {
   PhotosSearchbar *self = PHOTOS_SEARCHBAR (object);
-  PhotosSearchbarPrivate *priv = self->priv;
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
 
   if (priv->search_state_id != 0)
     {
@@ -235,8 +255,7 @@ photos_searchbar_init (PhotosSearchbar *self)
   PhotosSearchbarPrivate *priv;
   GtkStyleContext *context;
 
-  self->priv = photos_searchbar_get_instance_private (self);
-  priv = self->priv;
+  priv = photos_searchbar_get_instance_private (self);
 
   priv->toolbar = gtk_toolbar_new ();
   context = gtk_widget_get_style_context (priv->toolbar);
@@ -277,7 +296,7 @@ photos_searchbar_new (void)
 gboolean
 photos_searchbar_handle_event (PhotosSearchbar *self, GdkEventKey *event)
 {
-  PhotosSearchbarPrivate *priv = self->priv;
+  PhotosSearchbarPrivate *priv;
   gboolean is_escape;
   gboolean is_keynav;
   gboolean is_space;
@@ -287,6 +306,8 @@ photos_searchbar_handle_event (PhotosSearchbar *self, GdkEventKey *event)
   gchar *new_text = NULL;
   gchar *old_text = NULL;
 
+  priv = photos_searchbar_get_instance_private (self);
+
   if (gtk_widget_get_parent (GTK_WIDGET (self)) == NULL)
     goto out;
 
@@ -367,28 +388,40 @@ photos_searchbar_hide (PhotosSearchbar *self)
 gboolean
 photos_searchbar_is_focus (PhotosSearchbar *self)
 {
-  return gtk_widget_is_focus (self->priv->search_entry);
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
+  return gtk_widget_is_focus (priv->search_entry);
 }
 
 
 void
 photos_searchbar_set_search_change_blocked (PhotosSearchbar *self, gboolean search_change_blocked)
 {
-  self->priv->search_change_blocked = search_change_blocked;
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
+  priv->search_change_blocked = search_change_blocked;
 }
 
 
 void
 photos_searchbar_set_search_container (PhotosSearchbar *self, GtkWidget *search_container)
 {
-  self->priv->search_container = search_container;
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
+  priv->search_container = search_container;
 }
 
 
 void
 photos_searchbar_set_search_entry (PhotosSearchbar *self, GtkWidget *search_entry)
 {
-  self->priv->search_entry = search_entry;
+  PhotosSearchbarPrivate *priv;
+
+  priv = photos_searchbar_get_instance_private (self);
+  priv->search_entry = search_entry;
 }
 
 
diff --git a/src/photos-searchbar.h b/src/photos-searchbar.h
index 5070ee7..29f71a6 100644
--- a/src/photos-searchbar.h
+++ b/src/photos-searchbar.h
@@ -58,7 +58,6 @@ typedef struct _PhotosSearchbarPrivate PhotosSearchbarPrivate;
 struct _PhotosSearchbar
 {
   GtkRevealer parent_instance;
-  PhotosSearchbarPrivate *priv;
 };
 
 struct _PhotosSearchbarClass


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]