diff options
author | Timm Bäder <mail@baedert.org> | 2018-06-10 12:25:44 +0200 |
---|---|---|
committer | Timm Bäder <mail@baedert.org> | 2018-06-16 10:09:12 +0200 |
commit | 1b208eb45764841498f8e875bdab807bbe74a2c0 (patch) | |
tree | 636f30e9cd2679d56aa6585235cc50ef25e16e08 | |
parent | 1528665662ec29718f627cdc55cc66debabaf92e (diff) | |
download | gtk+-1b208eb45764841498f8e875bdab807bbe74a2c0.tar.gz |
combobox: Remove priv pointer
-rw-r--r-- | gtk/gtkcombobox.c | 206 | ||||
-rw-r--r-- | gtk/gtkcombobox.h | 4 |
2 files changed, 103 insertions, 107 deletions
diff --git a/gtk/gtkcombobox.c b/gtk/gtkcombobox.c index 4ed317d7d8..f5844ad572 100644 --- a/gtk/gtkcombobox.c +++ b/gtk/gtkcombobox.c @@ -109,8 +109,7 @@ */ -/* WELCOME, to THE house of evil code */ -struct _GtkComboBoxPrivate +typedef struct { GtkTreeModel *model; @@ -157,7 +156,7 @@ struct _GtkComboBoxPrivate GtkTreeViewRowSeparatorFunc row_separator_func; gpointer row_separator_data; GDestroyNotify row_separator_destroy; -}; +} GtkComboBoxPrivate; /* There are 2 modes to this widget, which can be characterized as follows: * @@ -349,7 +348,7 @@ gtk_combo_box_measure (GtkWidget *widget, int *natural_baseline) { GtkComboBox *combo_box = GTK_COMBO_BOX (widget); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gtk_widget_measure (priv->box, orientation, @@ -364,7 +363,7 @@ gtk_combo_box_size_allocate (GtkWidget *widget, int baseline) { GtkComboBox *combo_box = GTK_COMBO_BOX (widget); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gtk_widget_size_allocate (priv->box, allocation, baseline); @@ -398,7 +397,7 @@ gtk_combo_box_compute_expand (GtkWidget *widget, gboolean *vexpand) { GtkComboBox *combo_box = GTK_COMBO_BOX (widget); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkWidget *child; child = gtk_bin_get_child (GTK_BIN (combo_box)); @@ -877,9 +876,9 @@ gtk_combo_box_cell_editable_init (GtkCellEditableIface *iface) static gboolean gtk_combo_box_row_separator_func (GtkTreeModel *model, GtkTreeIter *iter, - GtkComboBox *combo) + GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (priv->row_separator_func) return priv->row_separator_func (model, iter, priv->row_separator_data); @@ -890,14 +889,11 @@ gtk_combo_box_row_separator_func (GtkTreeModel *model, static void gtk_combo_box_init (GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkStyleContext *context; GtkTreeMenu *menu; GtkEventController *controller; - combo_box->priv = gtk_combo_box_get_instance_private (combo_box); - priv = combo_box->priv; - priv->wrap_width = 0; priv->active = -1; @@ -952,7 +948,7 @@ gtk_combo_box_set_property (GObject *object, GParamSpec *pspec) { GtkComboBox *combo_box = GTK_COMBO_BOX (object); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); switch (prop_id) { @@ -1041,7 +1037,7 @@ gtk_combo_box_get_property (GObject *object, GParamSpec *pspec) { GtkComboBox *combo_box = GTK_COMBO_BOX (object); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); switch (prop_id) { @@ -1112,10 +1108,11 @@ gtk_combo_box_button_toggled (GtkWidget *widget, gpointer data) { GtkComboBox *combo_box = GTK_COMBO_BOX (data); + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (widget))) { - if (!combo_box->priv->popup_in_progress) + if (!priv->popup_in_progress) gtk_combo_box_popup (combo_box); } else @@ -1127,7 +1124,7 @@ gtk_combo_box_button_toggled (GtkWidget *widget, static void gtk_combo_box_create_child (GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkWidget *child; if (priv->has_entry) @@ -1162,7 +1159,7 @@ gtk_combo_box_add (GtkContainer *container, GtkWidget *widget) { GtkComboBox *combo_box = GTK_COMBO_BOX (container); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (priv->box == NULL) { @@ -1205,7 +1202,7 @@ gtk_combo_box_remove (GtkContainer *container, GtkWidget *widget) { GtkComboBox *combo_box = GTK_COMBO_BOX (container); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreePath *path; if (priv->has_entry) { @@ -1247,7 +1244,7 @@ gtk_combo_box_menu_show (GtkWidget *menu, gpointer user_data) { GtkComboBox *combo_box = GTK_COMBO_BOX (user_data); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gtk_combo_box_child_show (menu, user_data); @@ -1262,10 +1259,11 @@ gtk_combo_box_menu_hide (GtkWidget *menu, gpointer user_data) { GtkComboBox *combo_box = GTK_COMBO_BOX (user_data); + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gtk_combo_box_child_hide (menu,user_data); - gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (combo_box->priv->button), + gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (priv->button), FALSE); } @@ -1294,7 +1292,7 @@ static gboolean tree_column_row_is_sensitive (GtkComboBox *combo_box, GtkTreeIter *iter) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (priv->row_separator_func) { @@ -1311,7 +1309,7 @@ static void update_menu_sensitivity (GtkComboBox *combo_box, GtkWidget *menu) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GList *children, *child; GtkWidget *item, *submenu; GtkWidget *cell_view; @@ -1349,7 +1347,7 @@ update_menu_sensitivity (GtkComboBox *combo_box, static void gtk_combo_box_menu_popup (GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gint active_item; GtkWidget *active; @@ -1510,8 +1508,7 @@ gtk_combo_box_popup_for_device (GtkComboBox *combo_box, { /* As above, this currently does not do anything useful, and nothing with the * passed-in device. But the bits that are not blatantly obsolete are kept. */ - - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); g_return_if_fail (GDK_IS_DEVICE (device)); @@ -1519,8 +1516,6 @@ gtk_combo_box_popup_for_device (GtkComboBox *combo_box, if (!gtk_widget_get_realized (GTK_WIDGET (combo_box))) return; - priv = combo_box->priv; - if (gtk_widget_get_mapped (priv->popup_widget)) return; @@ -1536,7 +1531,9 @@ gtk_combo_box_real_popup (GtkComboBox *combo_box) static gboolean gtk_combo_box_real_popdown (GtkComboBox *combo_box) { - if (combo_box->priv->popup_shown) + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + + if (priv->popup_shown) { gtk_combo_box_popdown (combo_box); return TRUE; @@ -1557,19 +1554,17 @@ gtk_combo_box_real_popdown (GtkComboBox *combo_box) void gtk_combo_box_popdown (GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; - gtk_menu_popdown (GTK_MENU (priv->popup_widget)); } static void gtk_combo_box_unset_model (GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (priv->model) { @@ -1606,7 +1601,7 @@ gtk_combo_box_forall (GtkContainer *container, gpointer callback_data) { GtkComboBox *combo_box = GTK_COMBO_BOX (container); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkWidget *child; child = gtk_bin_get_child (GTK_BIN (container)); @@ -1618,7 +1613,7 @@ static void gtk_combo_box_child_show (GtkWidget *widget, GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); priv->popup_shown = TRUE; g_object_notify (G_OBJECT (combo_box), "popup-shown"); @@ -1628,7 +1623,7 @@ static void gtk_combo_box_child_hide (GtkWidget *widget, GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); priv->popup_shown = FALSE; g_object_notify (G_OBJECT (combo_box), "popup-shown"); @@ -1809,7 +1804,7 @@ gtk_combo_box_scroll_controller_scroll (GtkEventControllerScroll *scroll, gdouble dy, GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gboolean found = FALSE; GtkTreeIter iter; GtkTreeIter new_iter; @@ -1833,9 +1828,10 @@ gtk_combo_box_menu_activate (GtkWidget *menu, const gchar *path, GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeIter iter; - if (gtk_tree_model_get_iter_from_string (combo_box->priv->model, &iter, path)) + if (gtk_tree_model_get_iter_from_string (priv->model, &iter, path)) gtk_combo_box_set_active_iter (combo_box, &iter); g_object_set (combo_box, @@ -1847,7 +1843,7 @@ gtk_combo_box_menu_activate (GtkWidget *menu, static void gtk_combo_box_update_sensitivity (GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeIter iter; gboolean sensitive = TRUE; /* fool code checkers */ @@ -1891,7 +1887,7 @@ gtk_combo_box_model_row_deleted (GtkTreeModel *model, gpointer user_data) { GtkComboBox *combo_box = GTK_COMBO_BOX (user_data); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (!gtk_tree_row_reference_valid (priv->active_row)) { @@ -1920,7 +1916,7 @@ gtk_combo_box_model_row_changed (GtkTreeModel *model, gpointer user_data) { GtkComboBox *combo_box = GTK_COMBO_BOX (user_data); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreePath *active_path; /* FIXME this belongs to GtkCellView */ @@ -1959,7 +1955,9 @@ static GtkCellArea * gtk_combo_box_cell_layout_get_area (GtkCellLayout *cell_layout) { GtkComboBox *combo_box = GTK_COMBO_BOX (cell_layout); - return combo_box->priv->area; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + + return priv->area; } /* @@ -2043,9 +2041,10 @@ gtk_combo_box_new_with_model_and_entry (GtkTreeModel *model) gint gtk_combo_box_get_wrap_width (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), -1); - return combo_box->priv->wrap_width; + return priv->wrap_width; } /** @@ -2061,13 +2060,11 @@ void gtk_combo_box_set_wrap_width (GtkComboBox *combo_box, gint width) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); g_return_if_fail (width >= 0); - priv = combo_box->priv; - if (width != priv->wrap_width) { priv->wrap_width = width; @@ -2087,9 +2084,10 @@ gtk_combo_box_set_wrap_width (GtkComboBox *combo_box, gint gtk_combo_box_get_row_span_column (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), -1); - return combo_box->priv->row_column; + return priv->row_column; } /** @@ -2105,13 +2103,11 @@ void gtk_combo_box_set_row_span_column (GtkComboBox *combo_box, gint row_span) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gint col; g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; - col = gtk_tree_model_get_n_columns (priv->model); g_return_if_fail (row_span >= -1 && row_span < col); @@ -2134,9 +2130,11 @@ gtk_combo_box_set_row_span_column (GtkComboBox *combo_box, gint gtk_combo_box_get_column_span_column (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), -1); - return combo_box->priv->col_column; + return priv->col_column; } /** @@ -2152,13 +2150,11 @@ void gtk_combo_box_set_column_span_column (GtkComboBox *combo_box, gint column_span) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gint col; g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; - col = gtk_tree_model_get_n_columns (priv->model); g_return_if_fail (column_span >= -1 && column_span < col); @@ -2186,13 +2182,11 @@ gtk_combo_box_set_column_span_column (GtkComboBox *combo_box, gint gtk_combo_box_get_active (GtkComboBox *combo_box) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); gint result; g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), 0); - priv = combo_box->priv; - if (gtk_tree_row_reference_valid (priv->active_row)) { GtkTreePath *path; @@ -2219,14 +2213,12 @@ void gtk_combo_box_set_active (GtkComboBox *combo_box, gint index_) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreePath *path = NULL; g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); g_return_if_fail (index_ >= -1); - priv = combo_box->priv; - if (priv->model == NULL) { /* Save index, in case the model is set after the index */ @@ -2248,7 +2240,7 @@ static void gtk_combo_box_set_active_internal (GtkComboBox *combo_box, GtkTreePath *path) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreePath *active_path; gint path_cmp; @@ -2319,14 +2311,12 @@ gboolean gtk_combo_box_get_active_iter (GtkComboBox *combo_box, GtkTreeIter *iter) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreePath *path; gboolean result; g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), FALSE); - priv = combo_box->priv; - if (!gtk_tree_row_reference_valid (priv->active_row)) return FALSE; @@ -2376,13 +2366,11 @@ void gtk_combo_box_set_model (GtkComboBox *combo_box, GtkTreeModel *model) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); g_return_if_fail (model == NULL || GTK_IS_TREE_MODEL (model)); - priv = combo_box->priv; - if (model == priv->model) return; @@ -2439,16 +2427,18 @@ out: GtkTreeModel * gtk_combo_box_get_model (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), NULL); - return combo_box->priv->model; + return priv->model; } static void gtk_combo_box_real_move_active (GtkComboBox *combo_box, GtkScrollType scroll) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeIter iter; GtkTreeIter new_iter; gboolean active_iter; @@ -2536,8 +2526,9 @@ gtk_combo_box_mnemonic_activate (GtkWidget *widget, gboolean group_cycling) { GtkComboBox *combo_box = GTK_COMBO_BOX (widget); + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); - if (combo_box->priv->has_entry) + if (priv->has_entry) { GtkWidget* child; @@ -2546,7 +2537,7 @@ gtk_combo_box_mnemonic_activate (GtkWidget *widget, gtk_widget_grab_focus (child); } else - gtk_widget_grab_focus (combo_box->priv->button); + gtk_widget_grab_focus (priv->button); return TRUE; } @@ -2555,8 +2546,9 @@ static void gtk_combo_box_grab_focus (GtkWidget *widget) { GtkComboBox *combo_box = GTK_COMBO_BOX (widget); + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); - if (combo_box->priv->has_entry) + if (priv->has_entry) { GtkWidget *child; @@ -2565,7 +2557,7 @@ gtk_combo_box_grab_focus (GtkWidget *widget) gtk_widget_grab_focus (child); } else - gtk_widget_grab_focus (combo_box->priv->button); + gtk_widget_grab_focus (priv->button); } static void @@ -2580,7 +2572,7 @@ static void gtk_combo_box_destroy (GtkWidget *widget) { GtkComboBox *combo_box = GTK_COMBO_BOX (widget); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (priv->popup_idle_id > 0) { @@ -2674,7 +2666,7 @@ static gchar * gtk_combo_box_format_entry_text (GtkComboBox *combo_box, const gchar *path) { - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeModel *model; GtkTreeIter iter; gchar *text = NULL; @@ -2696,7 +2688,7 @@ static void gtk_combo_box_constructed (GObject *object) { GtkComboBox *combo_box = GTK_COMBO_BOX (object); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); G_OBJECT_CLASS (gtk_combo_box_parent_class)->constructed (object); @@ -2716,7 +2708,7 @@ static void gtk_combo_box_dispose (GObject* object) { GtkComboBox *combo_box = GTK_COMBO_BOX (object); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); if (priv->popup_widget) { @@ -2775,7 +2767,7 @@ gtk_combo_box_start_editing (GtkCellEditable *cell_editable, GdkEvent *event) { GtkComboBox *combo_box = GTK_COMBO_BOX (cell_editable); - GtkComboBoxPrivate *priv = combo_box->priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkWidget *child; priv->is_cell_renderer = TRUE; @@ -2813,12 +2805,10 @@ void gtk_combo_box_set_popup_fixed_width (GtkComboBox *combo_box, gboolean fixed) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; - if (priv->popup_fixed_width != fixed) { priv->popup_fixed_width = fixed; @@ -2839,9 +2829,11 @@ gtk_combo_box_set_popup_fixed_width (GtkComboBox *combo_box, gboolean gtk_combo_box_get_popup_fixed_width (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), FALSE); - return combo_box->priv->popup_fixed_width; + return priv->popup_fixed_width; } @@ -2860,9 +2852,11 @@ gtk_combo_box_get_popup_fixed_width (GtkComboBox *combo_box) AtkObject* gtk_combo_box_get_popup_accessible (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), NULL); - return gtk_widget_get_accessible (combo_box->priv->popup_widget); + return gtk_widget_get_accessible (priv->popup_widget); } /** @@ -2876,9 +2870,11 @@ gtk_combo_box_get_popup_accessible (GtkComboBox *combo_box) GtkTreeViewRowSeparatorFunc gtk_combo_box_get_row_separator_func (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), NULL); - return combo_box->priv->row_separator_func; + return priv->row_separator_func; } /** @@ -2898,12 +2894,10 @@ gtk_combo_box_set_row_separator_func (GtkComboBox *combo_box, gpointer data, GDestroyNotify destroy) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; - if (priv->row_separator_destroy) priv->row_separator_destroy (priv->row_separator_data); @@ -2931,12 +2925,10 @@ void gtk_combo_box_set_button_sensitivity (GtkComboBox *combo_box, GtkSensitivityType sensitivity) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; - if (priv->button_sensitivity != sensitivity) { priv->button_sensitivity = sensitivity; @@ -2962,9 +2954,10 @@ gtk_combo_box_set_button_sensitivity (GtkComboBox *combo_box, GtkSensitivityType gtk_combo_box_get_button_sensitivity (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), FALSE); - return combo_box->priv->button_sensitivity; + return priv->button_sensitivity; } @@ -2979,9 +2972,10 @@ gtk_combo_box_get_button_sensitivity (GtkComboBox *combo_box) gboolean gtk_combo_box_get_has_entry (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), FALSE); - return combo_box->priv->has_entry; + return priv->has_entry; } /** @@ -3001,12 +2995,11 @@ void gtk_combo_box_set_entry_text_column (GtkComboBox *combo_box, gint text_column) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeModel *model; g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; model = gtk_combo_box_get_model (combo_box); g_return_if_fail (text_column >= 0); @@ -3038,9 +3031,11 @@ gtk_combo_box_set_entry_text_column (GtkComboBox *combo_box, gint gtk_combo_box_get_entry_text_column (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), 0); - return combo_box->priv->text_column; + return priv->text_column; } static void @@ -3088,8 +3083,9 @@ gtk_combo_box_buildable_get_internal_child (GtkBuildable *buildable, const gchar *childname) { GtkComboBox *combo_box = GTK_COMBO_BOX (buildable); + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); - if (combo_box->priv->has_entry && strcmp (childname, "entry") == 0) + if (priv->has_entry && strcmp (childname, "entry") == 0) return G_OBJECT (gtk_bin_get_child (GTK_BIN (buildable))); return parent_buildable_iface->get_internal_child (buildable, builder, childname); @@ -3108,13 +3104,11 @@ void gtk_combo_box_set_id_column (GtkComboBox *combo_box, gint id_column) { - GtkComboBoxPrivate *priv; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeModel *model; g_return_if_fail (GTK_IS_COMBO_BOX (combo_box)); - priv = combo_box->priv; - if (id_column != priv->id_column) { model = gtk_combo_box_get_model (combo_box); @@ -3142,9 +3136,11 @@ gtk_combo_box_set_id_column (GtkComboBox *combo_box, gint gtk_combo_box_get_id_column (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), 0); - return combo_box->priv->id_column; + return priv->id_column; } /** @@ -3168,13 +3164,14 @@ gtk_combo_box_get_id_column (GtkComboBox *combo_box) const gchar * gtk_combo_box_get_active_id (GtkComboBox *combo_box) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeModel *model; GtkTreeIter iter; gint column; g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), NULL); - column = combo_box->priv->id_column; + column = priv->id_column; if (column < 0) return NULL; @@ -3218,6 +3215,7 @@ gboolean gtk_combo_box_set_active_id (GtkComboBox *combo_box, const gchar *active_id) { + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); GtkTreeModel *model; GtkTreeIter iter; gboolean match = FALSE; @@ -3231,7 +3229,7 @@ gtk_combo_box_set_active_id (GtkComboBox *combo_box, return TRUE; /* active row was successfully unset */ } - column = combo_box->priv->id_column; + column = priv->id_column; if (column < 0) return FALSE; @@ -3262,7 +3260,9 @@ gtk_combo_box_set_active_id (GtkComboBox *combo_box, } GtkWidget * -gtk_combo_box_get_popup (GtkComboBox *combo) +gtk_combo_box_get_popup (GtkComboBox *combo_box) { - return combo->priv->popup_widget; + GtkComboBoxPrivate *priv = gtk_combo_box_get_instance_private (combo_box); + + return priv->popup_widget; } diff --git a/gtk/gtkcombobox.h b/gtk/gtkcombobox.h index f5172e1166..9f2b9609ec 100644 --- a/gtk/gtkcombobox.h +++ b/gtk/gtkcombobox.h @@ -37,14 +37,10 @@ G_BEGIN_DECLS typedef struct _GtkComboBox GtkComboBox; typedef struct _GtkComboBoxClass GtkComboBoxClass; -typedef struct _GtkComboBoxPrivate GtkComboBoxPrivate; struct _GtkComboBox { GtkBin parent_instance; - - /*< private >*/ - GtkComboBoxPrivate *priv; }; /** |