From 273fa042a82f92ea1b794cbc59eb41af704596ad Mon Sep 17 00:00:00 2001 From: Matthias Clasen Date: Mon, 27 May 2019 00:28:49 -0400 Subject: [PATCH] radio menu item: Drop the priv pointer --- gtk/gtkradiomenuitem.c | 43 ++++++++++++++++++------------------------ 1 file changed, 18 insertions(+), 25 deletions(-) diff --git a/gtk/gtkradiomenuitem.c b/gtk/gtkradiomenuitem.c index 01bedbc908..261f0f44b5 100644 --- a/gtk/gtkradiomenuitem.c +++ b/gtk/gtkradiomenuitem.c @@ -74,14 +74,12 @@ * with name radio, which gets the .left or .right style class. */ -typedef struct _GtkRadioMenuItemPrivate GtkRadioMenuItemPrivate; -typedef struct _GtkRadioMenuItemClass GtkRadioMenuItemClass; +typedef struct _GtkRadioMenuItemPrivate GtkRadioMenuItemPrivate; +typedef struct _GtkRadioMenuItemClass GtkRadioMenuItemClass; struct _GtkRadioMenuItem { GtkCheckMenuItem check_menu_item; - - GtkRadioMenuItemPrivate *priv; }; struct _GtkRadioMenuItemClass @@ -189,14 +187,12 @@ void gtk_radio_menu_item_set_group (GtkRadioMenuItem *radio_menu_item, GSList *group) { - GtkRadioMenuItemPrivate *priv; + GtkRadioMenuItemPrivate *priv = gtk_radio_menu_item_get_instance_private (radio_menu_item); GtkWidget *old_group_singleton = NULL; GtkWidget *new_group_singleton = NULL; g_return_if_fail (GTK_IS_RADIO_MENU_ITEM (radio_menu_item)); - priv = radio_menu_item->priv; - if (priv->group == group) return; @@ -211,11 +207,10 @@ gtk_radio_menu_item_set_group (GtkRadioMenuItem *radio_menu_item, for (slist = priv->group; slist; slist = slist->next) { - GtkRadioMenuItem *tmp_item; - - tmp_item = slist->data; + GtkRadioMenuItem *tmp_item = slist->data; + GtkRadioMenuItemPrivate *tmp_priv = gtk_radio_menu_item_get_instance_private (tmp_item); - tmp_item->priv->group = priv->group; + tmp_priv->group = priv->group; } } @@ -230,11 +225,10 @@ gtk_radio_menu_item_set_group (GtkRadioMenuItem *radio_menu_item, for (slist = group; slist; slist = slist->next) { - GtkRadioMenuItem *tmp_item; - - tmp_item = slist->data; + GtkRadioMenuItem *tmp_item = slist->data; + GtkRadioMenuItemPrivate *tmp_priv = gtk_radio_menu_item_get_instance_private (tmp_item); - tmp_item->priv->group = priv->group; + tmp_priv->group = priv->group; } _gtk_check_menu_item_set_active (GTK_CHECK_MENU_ITEM (radio_menu_item), FALSE); @@ -396,9 +390,11 @@ gtk_radio_menu_item_new_with_label_from_widget (GtkRadioMenuItem *group, GSList* gtk_radio_menu_item_get_group (GtkRadioMenuItem *radio_menu_item) { + GtkRadioMenuItemPrivate *priv = gtk_radio_menu_item_get_instance_private (radio_menu_item); + g_return_val_if_fail (GTK_IS_RADIO_MENU_ITEM (radio_menu_item), NULL); - return radio_menu_item->priv->group; + return priv->group; } static void @@ -457,10 +453,7 @@ gtk_radio_menu_item_class_init (GtkRadioMenuItemClass *klass) static void gtk_radio_menu_item_init (GtkRadioMenuItem *radio_menu_item) { - GtkRadioMenuItemPrivate *priv; - - radio_menu_item->priv = gtk_radio_menu_item_get_instance_private (radio_menu_item); - priv = radio_menu_item->priv; + GtkRadioMenuItemPrivate *priv = gtk_radio_menu_item_get_instance_private (radio_menu_item); priv->group = g_slist_prepend (NULL, radio_menu_item); gtk_check_menu_item_set_draw_as_radio (GTK_CHECK_MENU_ITEM (radio_menu_item), TRUE); @@ -470,9 +463,8 @@ static void gtk_radio_menu_item_destroy (GtkWidget *widget) { GtkRadioMenuItem *radio_menu_item = GTK_RADIO_MENU_ITEM (widget); - GtkRadioMenuItemPrivate *priv = radio_menu_item->priv; + GtkRadioMenuItemPrivate *priv = gtk_radio_menu_item_get_instance_private (radio_menu_item); GtkWidget *old_group_singleton = NULL; - GtkRadioMenuItem *tmp_menu_item; GSList *tmp_list; gboolean was_in_group; @@ -486,10 +478,11 @@ gtk_radio_menu_item_destroy (GtkWidget *widget) while (tmp_list) { - tmp_menu_item = tmp_list->data; + GtkRadioMenuItem *tmp_item = tmp_list->data; + GtkRadioMenuItemPrivate *tmp_priv = gtk_radio_menu_item_get_instance_private (tmp_item); tmp_list = tmp_list->next; - tmp_menu_item->priv->group = priv->group; + tmp_priv->group = priv->group; } /* this radio menu item is no longer in the group */ @@ -507,7 +500,7 @@ static void gtk_radio_menu_item_activate (GtkMenuItem *menu_item) { GtkRadioMenuItem *radio_menu_item = GTK_RADIO_MENU_ITEM (menu_item); - GtkRadioMenuItemPrivate *priv = radio_menu_item->priv; + GtkRadioMenuItemPrivate *priv = gtk_radio_menu_item_get_instance_private (radio_menu_item); GtkCheckMenuItem *check_menu_item = GTK_CHECK_MENU_ITEM (menu_item); GtkCheckMenuItem *tmp_menu_item; GSList *tmp_list;