Merged in default (pull request #409)

Wed, 31 Oct 2018 01:45:24 +0000

author
Gary Kramlich <grim@reaperworld.com>
date
Wed, 31 Oct 2018 01:45:24 +0000
changeset 39249
917e9d27be24
parent 39247
ee65e9be3a5f (current diff)
parent 39248
9541bd9eed1f (diff)
child 39252
4d0acb9e4b15

Merged in default (pull request #409)

Port Finch away from deprecated g_type_class_add_private()

Approved-by: Elliott Sales de Andrade
Approved-by: Gary Kramlich
Approved-by: Eion Robb

--- a/finch/gntmedia.c	Sun Oct 14 19:40:06 2018 -0400
+++ b/finch/gntmedia.c	Wed Oct 31 01:45:24 2018 +0000
@@ -75,16 +75,13 @@
 	PurpleConversation *conv;
 };
 
-#define FINCH_MEDIA_GET_PRIVATE(obj) (G_TYPE_INSTANCE_GET_PRIVATE((obj), FINCH_TYPE_MEDIA, FinchMediaPrivate))
-
+static GType finch_media_get_type (void);
 static void finch_media_class_init (FinchMediaClass *klass);
 static void finch_media_init (FinchMedia *media);
 static void finch_media_finalize (GObject *object);
 static void finch_media_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec);
 static void finch_media_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec);
 
-static GntBoxClass *parent_class = NULL;
-
 enum {
 	MESSAGE,
 	LAST_SIGNAL
@@ -96,35 +93,12 @@
 	PROP_MEDIA,
 };
 
-static GType
-finch_media_get_type(void)
-{
-	static GType type = 0;
-
-	if (type == 0) {
-		static const GTypeInfo info = {
-			sizeof(FinchMediaClass),
-			NULL,
-			NULL,
-			(GClassInitFunc) finch_media_class_init,
-			NULL,
-			NULL,
-			sizeof(FinchMedia),
-			0,
-			(GInstanceInitFunc) finch_media_init,
-			NULL
-		};
-		type = g_type_register_static(GNT_TYPE_BOX, "FinchMedia", &info, 0);
-	}
-	return type;
-}
-
+G_DEFINE_TYPE_WITH_PRIVATE(FinchMedia, finch_media, GNT_TYPE_BOX);
 
 static void
 finch_media_class_init (FinchMediaClass *klass)
 {
 	GObjectClass *gobject_class = (GObjectClass*)klass;
-	parent_class = g_type_class_peek_parent(klass);
 
 	gobject_class->finalize = finch_media_finalize;
 	gobject_class->set_property = finch_media_set_property;
@@ -141,15 +115,13 @@
 					G_SIGNAL_RUN_LAST, 0, NULL, NULL,
 					g_cclosure_marshal_VOID__STRING,
 					G_TYPE_NONE, 1, G_TYPE_STRING);
-
-	g_type_class_add_private(klass, sizeof(FinchMediaPrivate));
 }
 
 
 static void
 finch_media_init (FinchMedia *media)
 {
-	media->priv = FINCH_MEDIA_GET_PRIVATE(media);
+	media->priv = finch_media_get_instance_private(media);
 
 	media->priv->calling = gnt_label_new(_("Calling..."));
 	media->priv->hangup = gnt_button_new(_("Hangup"));
--- a/finch/gntplugin.c	Sun Oct 14 19:40:06 2018 -0400
+++ b/finch/gntplugin.c	Wed Oct 31 01:45:24 2018 +0000
@@ -37,9 +37,6 @@
 #include "gntplugin.h"
 #include "gntrequest.h"
 
-#define FINCH_PLUGIN_INFO_GET_PRIVATE(obj) \
-	(G_TYPE_INSTANCE_GET_PRIVATE((obj), FINCH_TYPE_PLUGIN_INFO, FinchPluginInfoPrivate))
-
 typedef struct
 {
 	FinchPluginPrefFrameCb pref_frame_cb;
@@ -75,6 +72,9 @@
 	} u;
 } FinchPluginUiData;
 
+G_DEFINE_TYPE_WITH_PRIVATE(FinchPluginInfo, finch_plugin_info,
+		PURPLE_TYPE_PLUGIN_INFO);
+
 static GntWidget *process_pref_frame(PurplePluginPrefFrame *frame);
 
 /* Set method for GObject properties */
@@ -82,7 +82,8 @@
 finch_plugin_info_set_property(GObject *obj, guint param_id, const GValue *value,
 		GParamSpec *pspec)
 {
-	FinchPluginInfoPrivate *priv = FINCH_PLUGIN_INFO_GET_PRIVATE(obj);
+	FinchPluginInfoPrivate *priv = finch_plugin_info_get_instance_private(
+			FINCH_PLUGIN_INFO(obj));
 
 	switch (param_id) {
 		case PROP_GNT_PREF_FRAME_CB:
@@ -99,7 +100,8 @@
 finch_plugin_info_get_property(GObject *obj, guint param_id, GValue *value,
 		GParamSpec *pspec)
 {
-	FinchPluginInfoPrivate *priv = FINCH_PLUGIN_INFO_GET_PRIVATE(obj);
+	FinchPluginInfoPrivate *priv = finch_plugin_info_get_instance_private(
+			FINCH_PLUGIN_INFO(obj));
 
 	switch (param_id) {
 		case PROP_GNT_PREF_FRAME_CB:
@@ -111,13 +113,16 @@
 	}
 }
 
+static void
+finch_plugin_info_init(FinchPluginInfo *info)
+{
+}
+
 /* Class initializer function */
 static void finch_plugin_info_class_init(FinchPluginInfoClass *klass)
 {
 	GObjectClass *obj_class = G_OBJECT_CLASS(klass);
 
-	g_type_class_add_private(klass, sizeof(FinchPluginInfoPrivate));
-
 	/* Setup properties */
 	obj_class->get_property = finch_plugin_info_get_property;
 	obj_class->set_property = finch_plugin_info_set_property;
@@ -130,25 +135,6 @@
 		                     G_PARAM_STATIC_STRINGS));
 }
 
-GType
-finch_plugin_info_get_type(void)
-{
-	static GType type = 0;
-
-	if (G_UNLIKELY(type == 0)) {
-		static const GTypeInfo info = {
-			.class_size = sizeof(FinchPluginInfoClass),
-			.class_init = (GClassInitFunc)finch_plugin_info_class_init,
-			.instance_size = sizeof(FinchPluginInfo),
-		};
-
-		type = g_type_register_static(PURPLE_TYPE_PLUGIN_INFO,
-		                              "FinchPluginInfo", &info, 0);
-	}
-
-	return type;
-}
-
 FinchPluginInfo *
 finch_plugin_info_new(const char *first_property, ...)
 {
@@ -189,7 +175,8 @@
 		return FALSE;
 
 	if (FINCH_IS_PLUGIN_INFO(info))
-		priv = FINCH_PLUGIN_INFO_GET_PRIVATE(info);
+		priv = finch_plugin_info_get_instance_private(
+				FINCH_PLUGIN_INFO(info));
 
 	ret = ((priv && priv->pref_frame_cb) ||
 			purple_plugin_info_get_pref_frame_cb(info) ||
@@ -388,7 +375,8 @@
 	purple_plugin_info_set_ui_data(info, ui_data);
 
 	if (FINCH_IS_PLUGIN_INFO(info))
-		priv = FINCH_PLUGIN_INFO_GET_PRIVATE(info);
+		priv = finch_plugin_info_get_instance_private(
+				FINCH_PLUGIN_INFO(info));
 
 	if (priv && priv->pref_frame_cb != NULL)
 	{
--- a/finch/libgnt/gntprogressbar.c	Sun Oct 14 19:40:06 2018 -0400
+++ b/finch/libgnt/gntprogressbar.c	Wed Oct 31 01:45:24 2018 +0000
@@ -38,15 +38,13 @@
 	GntWidget parent;
 };
 
-#define GNT_PROGRESS_BAR_GET_PRIVATE(o)   (G_TYPE_INSTANCE_GET_PRIVATE ((o), GNT_TYPE_PROGRESS_BAR, GntProgressBarPrivate))
-
-static GntWidgetClass *parent_class = NULL;
-
+G_DEFINE_TYPE_WITH_PRIVATE (GntProgressBar, gnt_progress_bar, GNT_TYPE_WIDGET);
 
 static void
 gnt_progress_bar_draw (GntWidget *widget)
 {
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (GNT_PROGRESS_BAR (widget));
+	GntProgressBarPrivate *priv = gnt_progress_bar_get_instance_private (
+			GNT_PROGRESS_BAR (widget));
 	gchar progress[8];
 	gint start, end, i, pos;
 	int color;
@@ -115,23 +113,20 @@
 }
 
 static void
-gnt_progress_bar_class_init (gpointer klass, gpointer class_data)
+gnt_progress_bar_class_init (GntProgressBarClass *klass)
 {
-	GObjectClass *g_class = G_OBJECT_CLASS (klass);
+	GntWidgetClass *wclass = GNT_WIDGET_CLASS(klass);
 
-	parent_class = GNT_WIDGET_CLASS (klass);
-
-	g_type_class_add_private (g_class, sizeof (GntProgressBarPrivate));
-
-	parent_class->draw = gnt_progress_bar_draw;
-	parent_class->size_request = gnt_progress_bar_size_request;
+	wclass->draw = gnt_progress_bar_draw;
+	wclass->size_request = gnt_progress_bar_size_request;
 }
 
 static void
-gnt_progress_bar_init (GTypeInstance *instance, gpointer g_class)
+gnt_progress_bar_init (GntProgressBar *progress_bar)
 {
-	GntWidget *widget = GNT_WIDGET (instance);
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (GNT_PROGRESS_BAR (widget));
+	GntWidget *widget = GNT_WIDGET (progress_bar);
+	GntProgressBarPrivate *priv =
+			gnt_progress_bar_get_instance_private (progress_bar);
 
 	gnt_widget_set_take_focus (widget, FALSE);
 	GNT_WIDGET_SET_FLAGS (widget, GNT_WIDGET_NO_BORDER | GNT_WIDGET_NO_SHADOW | GNT_WIDGET_GROW_X);
@@ -142,31 +137,6 @@
 	priv->show_value = TRUE;
 }
 
-GType
-gnt_progress_bar_get_type (void)
-{
-	static GType type = 0;
-
-	if (type == 0) {
-		static const GTypeInfo info = {
-			sizeof (GntProgressBarClass),
-			NULL,                         /* base_init */
-			NULL,                         /* base_finalize */
-			gnt_progress_bar_class_init,  /* class_init */
-			NULL,                         /* class_finalize */
-			NULL,                         /* class_data */
-			sizeof (GntProgressBar),
-			0,                            /* n_preallocs */
-			gnt_progress_bar_init,        /* instance_init */
-			NULL                          /* value_table */
-		};
-
-		type = g_type_register_static (GNT_TYPE_WIDGET, "GntProgressBar", &info, 0);
-	}
-
-	return type;
-}
-
 GntWidget *
 gnt_progress_bar_new (void)
 {
@@ -177,7 +147,8 @@
 void
 gnt_progress_bar_set_fraction (GntProgressBar *pbar, gdouble fraction)
 {
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (pbar);
+	GntProgressBarPrivate *priv =
+			gnt_progress_bar_get_instance_private (pbar);
 
 	if (fraction > 1.0)
 		priv->fraction = 1.0;
@@ -194,7 +165,8 @@
 gnt_progress_bar_set_orientation (GntProgressBar *pbar,
 		GntProgressBarOrientation orientation)
 {
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (pbar);
+	GntProgressBarPrivate *priv =
+			gnt_progress_bar_get_instance_private (pbar);
 	GntWidget *widget = GNT_WIDGET(pbar);
 
 	priv->orientation = orientation;
@@ -218,28 +190,32 @@
 void
 gnt_progress_bar_set_show_progress (GntProgressBar *pbar, gboolean show)
 {
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (pbar);
+	GntProgressBarPrivate *priv =
+			gnt_progress_bar_get_instance_private (pbar);
 	priv->show_value = show;
 }
 
 gdouble
 gnt_progress_bar_get_fraction (GntProgressBar *pbar)
 {
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (pbar);
+	GntProgressBarPrivate *priv =
+			gnt_progress_bar_get_instance_private (pbar);
 	return priv->fraction;
 }
 
 GntProgressBarOrientation
 gnt_progress_bar_get_orientation (GntProgressBar *pbar)
 {
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (pbar);
+	GntProgressBarPrivate *priv =
+			gnt_progress_bar_get_instance_private (pbar);
 	return priv->orientation;
 }
 
 gboolean
 gnt_progress_bar_get_show_progress (GntProgressBar *pbar)
 {
-	GntProgressBarPrivate *priv = GNT_PROGRESS_BAR_GET_PRIVATE (pbar);
+	GntProgressBarPrivate *priv =
+			gnt_progress_bar_get_instance_private (pbar);
 	return priv->show_value;
 }
 

mercurial