2008-11-04 Claudio Saavedra <csaavedra@igalia.com>
authorClaudio Saavedra <csaavedra@igalia.com>
Tue, 4 Nov 2008 11:19:36 +0000 (11:19 +0000)
committerClaudio Saavedra <csaavedra@igalia.com>
Tue, 4 Nov 2008 11:19:36 +0000 (11:19 +0000)
Reverting following commit:

2008-10-13  Alberto Garcia  <agarcia@igalia.com>

  * src/hildon-app-menu-private.h
  * src/hildon-app-menu.[ch]
  * src/hildon-button.[ch]
  * src/hildon-check-button.[ch]
  * src/hildon-entry.[ch]
  * src/hildon-pannable-area.[ch]
  * src/hildon-text-view.[ch]:
  Added private field to the object's structure.

because it causes an ABI breakage. We will need to introduce these
changes later at some point. See NB#91636 for an extensive
explanation and status.

ChangeLog
src/hildon-app-menu.c
src/hildon-button.c
src/hildon-button.h
src/hildon-text-view.c
src/hildon-text-view.h

index 77e37be..d978cc3 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,24 @@
 2008-11-04  Claudio Saavedra  <csaavedra@igalia.com>
 
+       Reverting following commit:
+
+       2008-10-13  Alberto Garcia  <agarcia@igalia.com>
+
+         * src/hildon-app-menu-private.h
+         * src/hildon-app-menu.[ch]
+         * src/hildon-button.[ch]
+         * src/hildon-check-button.[ch]
+         * src/hildon-entry.[ch]
+         * src/hildon-pannable-area.[ch]
+         * src/hildon-text-view.[ch]:
+         Added private field to the object's structure.
+
+       because it causes an ABI breakage. We will need to introduce these
+       changes later at some point. See NB#91636 for an extensive
+       explanation and status.
+
+2008-11-04  Claudio Saavedra  <csaavedra@igalia.com>
+
        Patch contributed by Daniel Borgmann (danielb@openismus.com)
 
        * src/hildon-window.c: (paint_toolbar): Update toolbar sizes and
index c1a446b..4355da0 100644 (file)
@@ -161,7 +161,7 @@ hildon_app_menu_insert                          (HildonAppMenu *menu,
     g_return_if_fail (HILDON_IS_APP_MENU (menu));
     g_return_if_fail (GTK_IS_BUTTON (item));
 
-    priv = menu->priv;
+    priv = HILDON_APP_MENU_GET_PRIVATE(menu);
 
     /* Add the item to the menu */
     gtk_widget_show (GTK_WIDGET (item));
@@ -229,7 +229,7 @@ hildon_app_menu_reorder_child                   (HildonAppMenu *menu,
     g_return_if_fail (GTK_IS_BUTTON (item));
     g_return_if_fail (position >= 0);
 
-    priv = menu->priv;
+    priv = HILDON_APP_MENU_GET_PRIVATE (menu);
     old_position = g_list_index (priv->buttons, item);
 
     g_return_if_fail (old_position >= 0);
@@ -257,7 +257,7 @@ hildon_app_menu_add_filter                      (HildonAppMenu *menu,
     g_return_if_fail (HILDON_IS_APP_MENU (menu));
     g_return_if_fail (GTK_IS_BUTTON (filter));
 
-    priv = menu->priv;
+    priv = HILDON_APP_MENU_GET_PRIVATE(menu);
 
     /* Add the filter to the menu */
     gtk_widget_show (GTK_WIDGET (filter));
@@ -286,7 +286,7 @@ hildon_app_menu_set_columns                     (HildonAppMenu *menu,
     g_return_if_fail (HILDON_IS_APP_MENU (menu));
     g_return_if_fail (columns > 0);
 
-    priv = menu->priv;
+    priv = HILDON_APP_MENU_GET_PRIVATE (menu);
 
     if (columns != priv->columns) {
         priv->columns = columns;
@@ -330,7 +330,9 @@ item_visibility_changed                         (GtkWidget     *item,
                                                  GParamSpec    *arg1,
                                                  HildonAppMenu *menu)
 {
-    hildon_app_menu_repack_items (menu, g_list_index (menu->priv->buttons, item));
+    HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE (menu);
+
+    hildon_app_menu_repack_items (menu, g_list_index (priv->buttons, item));
 }
 
 static void
@@ -351,7 +353,7 @@ remove_item_from_list                           (GList    **list,
 static void
 hildon_app_menu_map                             (GtkWidget *widget)
 {
-    HildonAppMenuPrivate *priv = HILDON_APP_MENU (widget)->priv;
+    HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE(widget);
 
     GTK_WIDGET_CLASS (hildon_app_menu_parent_class)->map (widget);
 
@@ -387,7 +389,7 @@ hildon_app_menu_map                             (GtkWidget *widget)
 static void
 hildon_app_menu_unmap                           (GtkWidget *widget)
 {
-    HildonAppMenuPrivate *priv = HILDON_APP_MENU (widget)->priv;
+    HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE(widget);
 
     /* Remove the grab */
     if (priv->transfer_window != NULL) {
@@ -470,7 +472,7 @@ hildon_app_menu_button_press                    (GtkWidget *widget,
                                                  GdkEventButton *event)
 {
     int x, y;
-    HildonAppMenuPrivate *priv = HILDON_APP_MENU (widget)->priv;
+    HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE(widget);
 
     gdk_window_get_position (widget->window, &x, &y);
 
@@ -489,7 +491,7 @@ static gboolean
 hildon_app_menu_button_release                  (GtkWidget *widget,
                                                  GdkEventButton *event)
 {
-    HildonAppMenuPrivate *priv = HILDON_APP_MENU (widget)->priv;
+    HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE(widget);
 
     if (priv->pressed_outside) {
         int x, y;
@@ -585,7 +587,9 @@ hildon_app_menu_apply_style                     (GtkWidget *widget)
     GdkScreen *screen;
     gint width;
     guint horizontal_spacing, vertical_spacing, inner_border, external_border;
-    HildonAppMenuPrivate *priv = HILDON_APP_MENU (widget)->priv;
+    HildonAppMenuPrivate *priv;
+
+    priv = HILDON_APP_MENU_GET_PRIVATE (widget);
 
     gtk_widget_style_get (widget,
                           "horizontal-spacing", &horizontal_spacing,
@@ -621,7 +625,7 @@ hildon_app_menu_style_set                       (GtkWidget *widget,
 static void
 hildon_app_menu_repack_filters                  (HildonAppMenu *menu)
 {
-    HildonAppMenuPrivate *priv = menu->priv;
+    HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE(menu);
     GList *iter;
 
     for (iter = priv->filters; iter != NULL; iter = iter->next) {
@@ -651,10 +655,12 @@ static void
 hildon_app_menu_repack_items                    (HildonAppMenu *menu,
                                                  gint           start_from)
 {
-    HildonAppMenuPrivate *priv = menu->priv;
+    HildonAppMenuPrivate *priv;
     gint row, col;
     GList *iter;
 
+    priv = HILDON_APP_MENU_GET_PRIVATE(menu);
+
     /* Remove buttons from their parent */
     if (start_from != -1) {
         for (iter = g_list_nth (priv->buttons, start_from); iter != NULL; iter = iter->next) {
@@ -706,8 +712,6 @@ hildon_app_menu_init                            (HildonAppMenu *menu)
     GtkWidget *alignment;
     HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE(menu);
 
-    menu->priv = priv;
-
     /* Initialize private variables */
     priv->parent_window = NULL;
     priv->transfer_window = NULL;
@@ -743,7 +747,7 @@ hildon_app_menu_init                            (HildonAppMenu *menu)
 static void
 hildon_app_menu_finalize                        (GObject *object)
 {
-    HildonAppMenuPrivate *priv = HILDON_APP_MENU (object)->priv;
+    HildonAppMenuPrivate *priv = HILDON_APP_MENU_GET_PRIVATE(object);
 
     if (priv->transfer_window)
         gdk_window_destroy (priv->transfer_window);
index 0fe4726..6b04ea7 100644 (file)
@@ -84,6 +84,8 @@ G_DEFINE_TYPE                                   (HildonButton, hildon_button, GT
                                                 (G_TYPE_INSTANCE_GET_PRIVATE ((obj), \
                                                 HILDON_TYPE_BUTTON, HildonButtonPrivate));
 
+typedef struct                                  _HildonButtonPrivate HildonButtonPrivate;
+
 struct                                          _HildonButtonPrivate
 {
     GtkLabel *title;
@@ -166,7 +168,7 @@ hildon_button_style_set                         (GtkWidget *widget,
                                                  GtkStyle  *previous_style)
 {
     guint horizontal_spacing, vertical_spacing, image_spacing;
-    HildonButtonPrivate *priv = HILDON_BUTTON (widget)->priv;
+    HildonButtonPrivate *priv = HILDON_BUTTON_GET_PRIVATE (widget);
 
     if (GTK_WIDGET_CLASS (hildon_button_parent_class)->style_set)
         GTK_WIDGET_CLASS (hildon_button_parent_class)->style_set (widget, previous_style);
@@ -191,7 +193,7 @@ hildon_button_style_set                         (GtkWidget *widget,
 static void
 hildon_button_finalize                          (GObject *object)
 {
-    HildonButtonPrivate *priv = HILDON_BUTTON (object)->priv;
+    HildonButtonPrivate *priv = HILDON_BUTTON_GET_PRIVATE (object);
 
     g_object_unref (priv->alignment);
     g_object_unref (priv->label_box);
@@ -278,8 +280,6 @@ hildon_button_init                              (HildonButton *self)
 {
     HildonButtonPrivate *priv = HILDON_BUTTON_GET_PRIVATE (self);
 
-    self->priv = priv;
-
     priv->title = GTK_LABEL (gtk_label_new (NULL));
     priv->value = GTK_LABEL (gtk_label_new (NULL));
     priv->alignment = gtk_alignment_new (0.5, 0.5, 0, 0);
@@ -314,10 +314,14 @@ void
 hildon_button_add_title_size_group              (HildonButton *button,
                                                  GtkSizeGroup *size_group)
 {
+    HildonButtonPrivate *priv;
+
     g_return_if_fail (HILDON_IS_BUTTON (button));
     g_return_if_fail (GTK_IS_SIZE_GROUP (size_group));
 
-    gtk_size_group_add_widget (size_group, GTK_WIDGET (button->priv->title));
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    gtk_size_group_add_widget (size_group, GTK_WIDGET (priv->title));
 }
 
 /**
@@ -331,10 +335,14 @@ void
 hildon_button_add_value_size_group              (HildonButton *button,
                                                  GtkSizeGroup *size_group)
 {
+    HildonButtonPrivate *priv;
+
     g_return_if_fail (HILDON_IS_BUTTON (button));
     g_return_if_fail (GTK_IS_SIZE_GROUP (size_group));
 
-    gtk_size_group_add_widget (size_group, GTK_WIDGET (button->priv->value));
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    gtk_size_group_add_widget (size_group, GTK_WIDGET (priv->value));
 }
 
 /**
@@ -349,12 +357,16 @@ void
 hildon_button_add_image_size_group              (HildonButton *button,
                                                  GtkSizeGroup *size_group)
 {
+    HildonButtonPrivate *priv;
+
     g_return_if_fail (HILDON_IS_BUTTON (button));
     g_return_if_fail (GTK_IS_SIZE_GROUP (size_group));
 
-    g_return_if_fail (GTK_IS_WIDGET (button->priv->image));
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    g_return_if_fail (GTK_IS_WIDGET (priv->image));
 
-    gtk_size_group_add_widget (size_group, GTK_WIDGET (button->priv->image));
+    gtk_size_group_add_widget (size_group, GTK_WIDGET (priv->image));
 }
 
 /**
@@ -439,7 +451,9 @@ static void
 hildon_button_set_arrangement                   (HildonButton            *button,
                                                  HildonButtonArrangement  arrangement)
 {
-    HildonButtonPrivate *priv = button->priv;
+    HildonButtonPrivate *priv;
+
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
 
     /* Pack everything */
     if (arrangement == HILDON_BUTTON_ARRANGEMENT_VERTICAL) {
@@ -480,7 +494,7 @@ hildon_button_set_title                         (HildonButton *button,
 
     g_return_if_fail (HILDON_IS_BUTTON (button));
 
-    priv = button->priv;
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
     gtk_label_set_text (priv->title, title);
 
     /* If the button has no title, hide the label so the value is
@@ -516,7 +530,7 @@ hildon_button_set_value                         (HildonButton *button,
 
     g_return_if_fail (HILDON_IS_BUTTON (button));
 
-    priv = button->priv;
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
     gtk_label_set_text (priv->value, value);
 
     /* If the button has no value, hide the label so the title is
@@ -544,9 +558,13 @@ hildon_button_set_value                         (HildonButton *button,
 const gchar *
 hildon_button_get_title                         (HildonButton *button)
 {
+    HildonButtonPrivate *priv;
+
     g_return_val_if_fail (HILDON_IS_BUTTON (button), NULL);
 
-    return gtk_label_get_text (button->priv->title);
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    return gtk_label_get_text (priv->title);
 }
 
 /**
@@ -562,9 +580,13 @@ hildon_button_get_title                         (HildonButton *button)
 const gchar *
 hildon_button_get_value                         (HildonButton *button)
 {
+    HildonButtonPrivate *priv;
+
     g_return_val_if_fail (HILDON_IS_BUTTON (button), NULL);
 
-    return gtk_label_get_text (button->priv->value);
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    return gtk_label_get_text (priv->value);
 }
 
 /**
@@ -601,7 +623,7 @@ hildon_button_set_image                         (HildonButton *button,
     g_return_if_fail (HILDON_IS_BUTTON (button));
     g_return_if_fail (!image || GTK_IS_WIDGET (image));
 
-    priv = button->priv;
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
 
     /* Return if there's nothing to do */
     if (image == priv->image)
@@ -627,9 +649,13 @@ hildon_button_set_image                         (HildonButton *button,
 GtkWidget *
 hildon_button_get_image                         (HildonButton *button)
 {
+    HildonButtonPrivate *priv;
+
     g_return_val_if_fail (HILDON_IS_BUTTON (button), NULL);
 
-    return button->priv->image;
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    return priv->image;
 }
 
 /**
@@ -644,14 +670,18 @@ void
 hildon_button_set_image_position                (HildonButton    *button,
                                                  GtkPositionType  position)
 {
+    HildonButtonPrivate *priv;
+
     g_return_if_fail (HILDON_IS_BUTTON (button));
     g_return_if_fail (position == GTK_POS_LEFT || position == GTK_POS_RIGHT);
 
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
     /* Return if there's nothing to do */
-    if (button->priv->image_position == position)
+    if (priv->image_position == position)
         return;
 
-    button->priv->image_position = position;
+    priv->image_position = position;
 
     hildon_button_construct_child (button);
 }
@@ -675,10 +705,13 @@ hildon_button_set_alignment                     (HildonButton *button,
                                                  gfloat        xscale,
                                                  gfloat        yscale)
 {
+    HildonButtonPrivate *priv;
     GtkWidget *child;
 
     g_return_if_fail (HILDON_IS_BUTTON (button));
 
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
     child = gtk_bin_get_child (GTK_BIN (button));
 
     if (GTK_IS_ALIGNMENT (child)) {
@@ -702,9 +735,13 @@ hildon_button_set_title_alignment               (HildonButton *button,
                                                  gfloat        xalign,
                                                  gfloat        yalign)
 {
+    HildonButtonPrivate *priv;
+
     g_return_if_fail (HILDON_IS_BUTTON (button));
 
-    gtk_misc_set_alignment (GTK_MISC (button->priv->title), xalign, yalign);
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    gtk_misc_set_alignment (GTK_MISC (priv->title), xalign, yalign);
 }
 
 /**
@@ -722,9 +759,13 @@ hildon_button_set_value_alignment               (HildonButton *button,
                                                  gfloat        xalign,
                                                  gfloat        yalign)
 {
+    HildonButtonPrivate *priv;
+
     g_return_if_fail (HILDON_IS_BUTTON (button));
 
-    gtk_misc_set_alignment (GTK_MISC (button->priv->value), xalign, yalign);
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
+
+    gtk_misc_set_alignment (GTK_MISC (priv->value), xalign, yalign);
 }
 
 /**
@@ -746,7 +787,7 @@ hildon_button_set_image_alignment               (HildonButton *button,
 
     g_return_if_fail (HILDON_IS_BUTTON (button));
 
-    priv = button->priv;
+    priv = HILDON_BUTTON_GET_PRIVATE (button);
 
     /* Return if there's nothing to do */
     if (priv->image_xalign == xalign && priv->image_yalign == yalign)
@@ -761,7 +802,7 @@ hildon_button_set_image_alignment               (HildonButton *button,
 static void
 hildon_button_construct_child                   (HildonButton *button)
 {
-    HildonButtonPrivate *priv = button->priv;
+    HildonButtonPrivate *priv = HILDON_BUTTON_GET_PRIVATE (button);
     GtkWidget *child;
     gint image_spacing;
     const gchar *title, *value;
index a905042..7019ff7 100644 (file)
@@ -48,8 +48,6 @@ typedef struct                                  _HildonButton HildonButton;
 
 typedef struct                                  _HildonButtonClass HildonButtonClass;
 
-typedef struct                                  _HildonButtonPrivate HildonButtonPrivate;
-
 struct                                          _HildonButtonClass
 {
     GtkButtonClass parent_class;
@@ -58,9 +56,6 @@ struct                                          _HildonButtonClass
 struct                                          _HildonButton
 {
     GtkButton parent;
-
-    /* private */
-    HildonButtonPrivate *priv;
 };
 
 typedef enum {
index 30393f0..d39ce3b 100644 (file)
@@ -61,6 +61,8 @@ G_DEFINE_TYPE                                   (HildonTextView, hildon_text_vie
                                                 (G_TYPE_INSTANCE_GET_PRIVATE ((obj), \
                                                 HILDON_TYPE_TEXT_VIEW, HildonTextViewPrivate));
 
+typedef struct                                  _HildonTextViewPrivate HildonTextViewPrivate;
+
 struct                                          _HildonTextViewPrivate
 {
     GtkTextBuffer *main_buffer;                   /* Used to show the "real" contents */
@@ -74,7 +76,7 @@ static const gchar *placeholder_widget_name     = "hildon-text-view-placeholder"
 static void
 hildon_text_view_refresh_contents               (GtkWidget *text_view)
 {
-    HildonTextViewPrivate *priv = HILDON_TEXT_VIEW (text_view)->priv;
+    HildonTextViewPrivate *priv = HILDON_TEXT_VIEW_GET_PRIVATE (text_view);
     gint bufsize = gtk_text_buffer_get_char_count (priv->main_buffer);
 
     if ((bufsize > 0) || GTK_WIDGET_HAS_FOCUS (text_view)) {
@@ -111,7 +113,7 @@ hildon_text_view_set_buffer                     (HildonTextView *text_view,
     g_return_if_fail (HILDON_IS_TEXT_VIEW (text_view));
     g_return_if_fail (GTK_IS_TEXT_BUFFER (buffer));
 
-    priv = text_view->priv;
+    priv = HILDON_TEXT_VIEW_GET_PRIVATE (text_view);
 
     /* If this is the same buffer, don't do anything */
     if (buffer == priv->main_buffer)
@@ -152,11 +154,15 @@ hildon_text_view_set_buffer                     (HildonTextView *text_view,
 GtkTextBuffer *
 hildon_text_view_get_buffer                     (HildonTextView *text_view)
 {
+    HildonTextViewPrivate *priv;
+
     g_return_val_if_fail (HILDON_IS_TEXT_VIEW (text_view), NULL);
 
+    priv = HILDON_TEXT_VIEW_GET_PRIVATE (text_view);
+
     /* Always return priv->main_buffer even if the placeholder is
      * being displayed */
-    return text_view->priv->main_buffer;
+    return priv->main_buffer;
 }
 
 /**
@@ -170,9 +176,13 @@ void
 hildon_text_view_set_placeholder                (HildonTextView *text_view,
                                                  const gchar    *text)
 {
+    HildonTextViewPrivate *priv;
+
     g_return_if_fail (HILDON_IS_TEXT_VIEW (text_view) && text != NULL);
 
-    gtk_text_buffer_set_text (text_view->priv->placeholder_buffer, text, -1);
+    priv = HILDON_TEXT_VIEW_GET_PRIVATE (text_view);
+
+    gtk_text_buffer_set_text (priv->placeholder_buffer, text, -1);
 }
 
 /**
@@ -219,7 +229,7 @@ hildon_text_view_focus_out_event                (GtkWidget     *widget,
 static void
 hildon_text_view_finalize                       (GObject *object)
 {
-    HildonTextViewPrivate *priv = HILDON_TEXT_VIEW (object)->priv;
+    HildonTextViewPrivate *priv = HILDON_TEXT_VIEW_GET_PRIVATE (object);
 
     g_signal_handler_disconnect (priv->main_buffer, priv->changed_id);
     g_object_unref (priv->main_buffer);
@@ -247,8 +257,6 @@ hildon_text_view_init                           (HildonTextView *self)
 {
     HildonTextViewPrivate *priv = HILDON_TEXT_VIEW_GET_PRIVATE (self);
 
-    self->priv = priv;
-
     priv->main_buffer = gtk_text_buffer_new (NULL);
     priv->placeholder_buffer = gtk_text_buffer_new (NULL);
 
index 775a321..87b78a6 100644 (file)
@@ -46,8 +46,6 @@ typedef struct                                  _HildonTextView HildonTextView;
 
 typedef struct                                  _HildonTextViewClass HildonTextViewClass;
 
-typedef struct                                  _HildonTextViewPrivate HildonTextViewPrivate;
-
 struct                                          _HildonTextViewClass
 {
     GtkTextViewClass parent_class;
@@ -56,9 +54,6 @@ struct                                          _HildonTextViewClass
 struct                                          _HildonTextView
 {
     GtkTextView parent;
-
-    /* private */
-    HildonTextViewPrivate *priv;
 };