[gtksourceview/wip/chergert/gsv-gtk4: 16/118] completion: modernize GtkSourceCompletionItem



commit 74419ddcdc02c7857399119a68d0ff15200c30b5
Author: Christian Hergert <chergert redhat com>
Date:   Wed Jan 8 19:43:11 2020 -0800

    completion: modernize GtkSourceCompletionItem
    
    Use G_DECLARE and other modernizations

 gtksourceview/gtksourceautocleanups.h   |   1 -
 gtksourceview/gtksourcecompletionitem.c | 169 +++++++++++++++++++-------------
 gtksourceview/gtksourcecompletionitem.h |  63 +++++-------
 3 files changed, 121 insertions(+), 112 deletions(-)
---
diff --git a/gtksourceview/gtksourceautocleanups.h b/gtksourceview/gtksourceautocleanups.h
index aa8070c3..5e65f496 100644
--- a/gtksourceview/gtksourceautocleanups.h
+++ b/gtksourceview/gtksourceautocleanups.h
@@ -28,7 +28,6 @@ G_BEGIN_DECLS
 
 #ifndef __GI_SCANNER__
 
-G_DEFINE_AUTOPTR_CLEANUP_FUNC(GtkSourceCompletionItem, g_object_unref)
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(GtkSourceFile, g_object_unref)
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(GtkSourceFileLoader, g_object_unref)
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(GtkSourceFileSaver, g_object_unref)
diff --git a/gtksourceview/gtksourcecompletionitem.c b/gtksourceview/gtksourcecompletionitem.c
index 0b4054d6..e81ae72c 100644
--- a/gtksourceview/gtksourcecompletionitem.c
+++ b/gtksourceview/gtksourcecompletionitem.c
@@ -33,7 +33,7 @@
 #include "gtksourcecompletionitem.h"
 #include "gtksourcecompletionproposal.h"
 
-struct _GtkSourceCompletionItemPrivate
+typedef struct
 {
        gchar *label;
        gchar *markup;
@@ -42,7 +42,7 @@ struct _GtkSourceCompletionItemPrivate
        gchar *icon_name;
        GIcon *gicon;
        gchar *info;
-};
+} GtkSourceCompletionItemPrivate;
 
 enum
 {
@@ -59,57 +59,78 @@ enum
 static void gtk_source_completion_proposal_iface_init (gpointer g_iface, gpointer iface_data);
 
 G_DEFINE_TYPE_WITH_CODE (GtkSourceCompletionItem,
-                        gtk_source_completion_item,
-                        G_TYPE_OBJECT,
-                        G_ADD_PRIVATE (GtkSourceCompletionItem)
-                        G_IMPLEMENT_INTERFACE (GTK_SOURCE_TYPE_COMPLETION_PROPOSAL,
-                                               gtk_source_completion_proposal_iface_init))
+                         gtk_source_completion_item,
+                         G_TYPE_OBJECT,
+                         G_ADD_PRIVATE (GtkSourceCompletionItem)
+                         G_IMPLEMENT_INTERFACE (GTK_SOURCE_TYPE_COMPLETION_PROPOSAL,
+                                                gtk_source_completion_proposal_iface_init))
 
 static gchar *
 gtk_source_completion_proposal_get_label_impl (GtkSourceCompletionProposal *proposal)
 {
-       return g_strdup (GTK_SOURCE_COMPLETION_ITEM (proposal)->priv->label);
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (proposal);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
+       return g_strdup (priv->label);
 }
 
 static gchar *
 gtk_source_completion_proposal_get_markup_impl (GtkSourceCompletionProposal *proposal)
 {
-       return g_strdup (GTK_SOURCE_COMPLETION_ITEM (proposal)->priv->markup);
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (proposal);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
+       return g_strdup (priv->markup);
 }
 
 static gchar *
 gtk_source_completion_proposal_get_text_impl (GtkSourceCompletionProposal *proposal)
 {
-       return g_strdup (GTK_SOURCE_COMPLETION_ITEM (proposal)->priv->text);
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (proposal);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
+       return g_strdup (priv->text);
 }
 
 static GdkPixbuf *
 gtk_source_completion_proposal_get_icon_impl (GtkSourceCompletionProposal *proposal)
 {
-       return GTK_SOURCE_COMPLETION_ITEM (proposal)->priv->icon;
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (proposal);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
+       return priv->icon;
 }
 
 static const gchar *
 gtk_source_completion_proposal_get_icon_name_impl (GtkSourceCompletionProposal *proposal)
 {
-       return GTK_SOURCE_COMPLETION_ITEM (proposal)->priv->icon_name;
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (proposal);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
+       return priv->icon_name;
 }
 
 static GIcon *
 gtk_source_completion_proposal_get_gicon_impl (GtkSourceCompletionProposal *proposal)
 {
-       return GTK_SOURCE_COMPLETION_ITEM (proposal)->priv->gicon;
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (proposal);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
+       return priv->gicon;
 }
 
 static gchar *
 gtk_source_completion_proposal_get_info_impl (GtkSourceCompletionProposal *proposal)
 {
-       return g_strdup (GTK_SOURCE_COMPLETION_ITEM (proposal)->priv->info);
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (proposal);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
+       return g_strdup (priv->info);
 }
 
 static void
 gtk_source_completion_proposal_iface_init (gpointer g_iface,
-                                          gpointer iface_data)
+                                           gpointer iface_data)
 {
        GtkSourceCompletionProposalIface *iface = g_iface;
 
@@ -127,9 +148,10 @@ static void
 gtk_source_completion_item_dispose (GObject *object)
 {
        GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (object);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
 
-       g_clear_object (&item->priv->icon);
-       g_clear_object (&item->priv->gicon);
+       g_clear_object (&priv->icon);
+       g_clear_object (&priv->gicon);
 
        G_OBJECT_CLASS (gtk_source_completion_item_parent_class)->dispose (object);
 }
@@ -138,56 +160,54 @@ static void
 gtk_source_completion_item_finalize (GObject *object)
 {
        GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (object);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
 
-       g_free (item->priv->label);
-       g_free (item->priv->markup);
-       g_free (item->priv->text);
-       g_free (item->priv->icon_name);
-       g_free (item->priv->info);
+       g_free (priv->label);
+       g_free (priv->markup);
+       g_free (priv->text);
+       g_free (priv->icon_name);
+       g_free (priv->info);
 
        G_OBJECT_CLASS (gtk_source_completion_item_parent_class)->finalize (object);
 }
 
 static void
 gtk_source_completion_item_get_property (GObject    *object,
-                                        guint       prop_id,
-                                        GValue     *value,
-                                        GParamSpec *pspec)
+                                         guint       prop_id,
+                                         GValue     *value,
+                                         GParamSpec *pspec)
 {
-       GtkSourceCompletionItem *item;
-
-       g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (object));
-
-       item = GTK_SOURCE_COMPLETION_ITEM (object);
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (object);
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
 
        switch (prop_id)
        {
                case PROP_LABEL:
-                       g_value_set_string (value, item->priv->label);
+                       g_value_set_string (value, priv->label);
                        break;
 
                case PROP_MARKUP:
-                       g_value_set_string (value, item->priv->markup);
+                       g_value_set_string (value, priv->markup);
                        break;
 
                case PROP_TEXT:
-                       g_value_set_string (value, item->priv->text);
+                       g_value_set_string (value, priv->text);
                        break;
 
                case PROP_ICON:
-                       g_value_set_object (value, item->priv->icon);
+                       g_value_set_object (value, priv->icon);
                        break;
 
                case PROP_ICON_NAME:
-                       g_value_set_string (value, item->priv->icon_name);
+                       g_value_set_string (value, priv->icon_name);
                        break;
 
                case PROP_GICON:
-                       g_value_set_object (value, item->priv->gicon);
+                       g_value_set_object (value, priv->gicon);
                        break;
 
                case PROP_INFO:
-                       g_value_set_string (value, item->priv->info);
+                       g_value_set_string (value, priv->info);
                        break;
 
                default:
@@ -204,15 +224,11 @@ emit_changed (GtkSourceCompletionItem *item)
 
 static void
 gtk_source_completion_item_set_property (GObject      *object,
-                                        guint         prop_id,
-                                        const GValue *value,
-                                        GParamSpec   *pspec)
+                                         guint         prop_id,
+                                         const GValue *value,
+                                         GParamSpec   *pspec)
 {
-       GtkSourceCompletionItem *item;
-
-       g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (object));
-
-       item = GTK_SOURCE_COMPLETION_ITEM (object);
+       GtkSourceCompletionItem *item = GTK_SOURCE_COMPLETION_ITEM (object);
 
        switch (prop_id)
        {
@@ -366,7 +382,6 @@ gtk_source_completion_item_class_init (GtkSourceCompletionItemClass *klass)
 static void
 gtk_source_completion_item_init (GtkSourceCompletionItem *item)
 {
-       item->priv = gtk_source_completion_item_get_instance_private (item);
 }
 
 /**
@@ -393,14 +408,16 @@ gtk_source_completion_item_new (void)
  */
 void
 gtk_source_completion_item_set_label (GtkSourceCompletionItem *item,
-                                     const gchar             *label)
+                                      const gchar             *label)
 {
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
        g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (item));
 
-       if (g_strcmp0 (item->priv->label, label) != 0)
+       if (g_strcmp0 (priv->label, label) != 0)
        {
-               g_free (item->priv->label);
-               item->priv->label = g_strdup (label);
+               g_free (priv->label);
+               priv->label = g_strdup (label);
 
                emit_changed (item);
                g_object_notify (G_OBJECT (item), "label");
@@ -416,14 +433,16 @@ gtk_source_completion_item_set_label (GtkSourceCompletionItem *item,
  */
 void
 gtk_source_completion_item_set_markup (GtkSourceCompletionItem *item,
-                                      const gchar             *markup)
+                                       const gchar             *markup)
 {
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
        g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (item));
 
-       if (g_strcmp0 (item->priv->markup, markup) != 0)
+       if (g_strcmp0 (priv->markup, markup) != 0)
        {
-               g_free (item->priv->markup);
-               item->priv->markup = g_strdup (markup);
+               g_free (priv->markup);
+               priv->markup = g_strdup (markup);
 
                emit_changed (item);
                g_object_notify (G_OBJECT (item), "markup");
@@ -439,14 +458,16 @@ gtk_source_completion_item_set_markup (GtkSourceCompletionItem *item,
  */
 void
 gtk_source_completion_item_set_text (GtkSourceCompletionItem *item,
-                                    const gchar             *text)
+                                     const gchar             *text)
 {
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
        g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (item));
 
-       if (g_strcmp0 (item->priv->text, text) != 0)
+       if (g_strcmp0 (priv->text, text) != 0)
        {
-               g_free (item->priv->text);
-               item->priv->text = g_strdup (text);
+               g_free (priv->text);
+               priv->text = g_strdup (text);
 
                emit_changed (item);
                g_object_notify (G_OBJECT (item), "text");
@@ -462,12 +483,14 @@ gtk_source_completion_item_set_text (GtkSourceCompletionItem *item,
  */
 void
 gtk_source_completion_item_set_icon (GtkSourceCompletionItem *item,
-                                    GdkPixbuf               *icon)
+                                     GdkPixbuf               *icon)
 {
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
        g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (item));
        g_return_if_fail (icon == NULL || GDK_IS_PIXBUF (icon));
 
-       if (g_set_object (&item->priv->icon, icon))
+       if (g_set_object (&priv->icon, icon))
        {
                emit_changed (item);
                g_object_notify (G_OBJECT (item), "icon");
@@ -483,14 +506,16 @@ gtk_source_completion_item_set_icon (GtkSourceCompletionItem *item,
  */
 void
 gtk_source_completion_item_set_icon_name (GtkSourceCompletionItem *item,
-                                         const gchar             *icon_name)
+                                          const gchar             *icon_name)
 {
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
        g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (item));
 
-       if (g_strcmp0 (item->priv->icon_name, icon_name) != 0)
+       if (g_strcmp0 (priv->icon_name, icon_name) != 0)
        {
-               g_free (item->priv->icon_name);
-               item->priv->icon_name = g_strdup (icon_name);
+               g_free (priv->icon_name);
+               priv->icon_name = g_strdup (icon_name);
 
                emit_changed (item);
                g_object_notify (G_OBJECT (item), "icon-name");
@@ -506,12 +531,14 @@ gtk_source_completion_item_set_icon_name (GtkSourceCompletionItem *item,
  */
 void
 gtk_source_completion_item_set_gicon (GtkSourceCompletionItem *item,
-                                     GIcon                   *gicon)
+                                      GIcon                   *gicon)
 {
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
        g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (item));
        g_return_if_fail (gicon == NULL || G_IS_ICON (gicon));
 
-       if (g_set_object (&item->priv->gicon, gicon))
+       if (g_set_object (&priv->gicon, gicon))
        {
                emit_changed (item);
                g_object_notify (G_OBJECT (item), "gicon");
@@ -527,14 +554,16 @@ gtk_source_completion_item_set_gicon (GtkSourceCompletionItem *item,
  */
 void
 gtk_source_completion_item_set_info (GtkSourceCompletionItem *item,
-                                    const gchar             *info)
+                                     const gchar             *info)
 {
+       GtkSourceCompletionItemPrivate *priv = gtk_source_completion_item_get_instance_private (item);
+
        g_return_if_fail (GTK_SOURCE_IS_COMPLETION_ITEM (item));
 
-       if (g_strcmp0 (item->priv->info, info) != 0)
+       if (g_strcmp0 (priv->info, info) != 0)
        {
-               g_free (item->priv->info);
-               item->priv->info = g_strdup (info);
+               g_free (priv->info);
+               priv->info = g_strdup (info);
 
                emit_changed (item);
                g_object_notify (G_OBJECT (item), "info");
diff --git a/gtksourceview/gtksourcecompletionitem.h b/gtksourceview/gtksourcecompletionitem.h
index 750786ef..6eeb465f 100644
--- a/gtksourceview/gtksourcecompletionitem.h
+++ b/gtksourceview/gtksourcecompletionitem.h
@@ -30,60 +30,41 @@
 
 G_BEGIN_DECLS
 
-#define GTK_SOURCE_TYPE_COMPLETION_ITEM                        (gtk_source_completion_item_get_type ())
-#define GTK_SOURCE_COMPLETION_ITEM(obj)                        (G_TYPE_CHECK_INSTANCE_CAST ((obj), 
GTK_SOURCE_TYPE_COMPLETION_ITEM, GtkSourceCompletionItem))
-#define GTK_SOURCE_COMPLETION_ITEM_CLASS(klass)                (G_TYPE_CHECK_CLASS_CAST ((klass), 
GTK_SOURCE_TYPE_COMPLETION_ITEM, GtkSourceCompletionItemClass))
-#define GTK_SOURCE_IS_COMPLETION_ITEM(obj)             (G_TYPE_CHECK_INSTANCE_TYPE ((obj), 
GTK_SOURCE_TYPE_COMPLETION_ITEM))
-#define GTK_SOURCE_IS_COMPLETION_ITEM_CLASS(klass)     (G_TYPE_CHECK_CLASS_TYPE ((klass), 
GTK_SOURCE_TYPE_COMPLETION_ITEM))
-#define GTK_SOURCE_COMPLETION_ITEM_GET_CLASS(obj)      (G_TYPE_INSTANCE_GET_CLASS ((obj), 
GTK_SOURCE_TYPE_COMPLETION_ITEM, GtkSourceCompletionItemClass))
+#define GTK_SOURCE_TYPE_COMPLETION_ITEM (gtk_source_completion_item_get_type())
 
-typedef struct _GtkSourceCompletionItemClass   GtkSourceCompletionItemClass;
-typedef struct _GtkSourceCompletionItemPrivate GtkSourceCompletionItemPrivate;
-
-struct _GtkSourceCompletionItem {
-       GObject parent;
-
-       GtkSourceCompletionItemPrivate *priv;
-};
+GTK_SOURCE_AVAILABLE_IN_ALL
+G_DECLARE_DERIVABLE_TYPE (GtkSourceCompletionItem, gtk_source_completion_item, GTK_SOURCE, COMPLETION_ITEM, 
GObject)
 
-struct _GtkSourceCompletionItemClass {
+struct _GtkSourceCompletionItemClass
+{
        GObjectClass parent_class;
 
-       gpointer padding[10];
+       /*< private >*/
+       gpointer _reserved[10];
 };
 
-GTK_SOURCE_AVAILABLE_IN_ALL
-GType                   gtk_source_completion_item_get_type            (void) G_GNUC_CONST;
-
 GTK_SOURCE_AVAILABLE_IN_4_0
-GtkSourceCompletionItem *gtk_source_completion_item_new                        (void);
-
+GtkSourceCompletionItem *gtk_source_completion_item_new           (void);
 GTK_SOURCE_AVAILABLE_IN_3_24
-void                    gtk_source_completion_item_set_label           (GtkSourceCompletionItem *item,
-                                                                        const gchar             *label);
-
+void                     gtk_source_completion_item_set_label     (GtkSourceCompletionItem *item,
+                                                                   const gchar             *label);
 GTK_SOURCE_AVAILABLE_IN_3_24
-void                    gtk_source_completion_item_set_markup          (GtkSourceCompletionItem *item,
-                                                                        const gchar             *markup);
-
+void                     gtk_source_completion_item_set_markup    (GtkSourceCompletionItem *item,
+                                                                   const gchar             *markup);
 GTK_SOURCE_AVAILABLE_IN_3_24
-void                    gtk_source_completion_item_set_text            (GtkSourceCompletionItem *item,
-                                                                        const gchar             *text);
-
+void                     gtk_source_completion_item_set_text      (GtkSourceCompletionItem *item,
+                                                                   const gchar             *text);
 GTK_SOURCE_AVAILABLE_IN_3_24
-void                    gtk_source_completion_item_set_icon            (GtkSourceCompletionItem *item,
-                                                                        GdkPixbuf               *icon);
-
+void                     gtk_source_completion_item_set_icon      (GtkSourceCompletionItem *item,
+                                                                   GdkPixbuf               *icon);
 GTK_SOURCE_AVAILABLE_IN_3_24
-void                    gtk_source_completion_item_set_icon_name       (GtkSourceCompletionItem *item,
-                                                                        const gchar             *icon_name);
-
+void                     gtk_source_completion_item_set_icon_name (GtkSourceCompletionItem *item,
+                                                                   const gchar             *icon_name);
 GTK_SOURCE_AVAILABLE_IN_3_24
-void                    gtk_source_completion_item_set_gicon           (GtkSourceCompletionItem *item,
-                                                                        GIcon                   *gicon);
-
+void                     gtk_source_completion_item_set_gicon     (GtkSourceCompletionItem *item,
+                                                                   GIcon                   *gicon);
 GTK_SOURCE_AVAILABLE_IN_3_24
-void                    gtk_source_completion_item_set_info            (GtkSourceCompletionItem *item,
-                                                                        const gchar             *info);
+void                     gtk_source_completion_item_set_info      (GtkSourceCompletionItem *item,
+                                                                   const gchar             *info);
 
 G_END_DECLS


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