diff options
Diffstat (limited to 'widgets/table')
-rw-r--r-- | widgets/table/ChangeLog | 11 | ||||
-rw-r--r-- | widgets/table/e-table-field-chooser-item.c | 4 | ||||
-rw-r--r-- | widgets/table/e-table-header-item.c | 21 | ||||
-rw-r--r-- | widgets/table/e-table-header-item.h | 6 | ||||
-rw-r--r-- | widgets/table/e-table-header-utils.c | 6 | ||||
-rw-r--r-- | widgets/table/e-table.c | 4 |
6 files changed, 29 insertions, 23 deletions
diff --git a/widgets/table/ChangeLog b/widgets/table/ChangeLog index 2ee5268d62..c25ecf1a0d 100644 --- a/widgets/table/ChangeLog +++ b/widgets/table/ChangeLog @@ -1,3 +1,14 @@ +2007-10-09 Matthew Barnes <mbarnes@redhat.com> + + ** Fixes part of bug #437579 + + * e-table-field-chooser-item.c: + * e-table-header-item.c: + * e-table-header-item.h: + * e-table-header-utils.c: + * e-table.c: + Fix various compiler warnings. Patch from Milan Crha. + 2007-10-02 Matthew Barnes <mbarnes@redhat.com> ** Fixes part of bug #469657 diff --git a/widgets/table/e-table-field-chooser-item.c b/widgets/table/e-table-field-chooser-item.c index b1a9d37a54..4ab694f3d6 100644 --- a/widgets/table/e-table-field-chooser-item.c +++ b/widgets/table/e-table-field-chooser-item.c @@ -206,9 +206,7 @@ etfci_update (GnomeCanvasItem *item, double *affine, ArtSVP *clip_path, int flag item->y1 = c1.y; item->x2 = c2.x; item->y2 = c2.y; -#ifndef NO_WARNINGS -#warning Group Child bounds !? -#endif +/* FIXME: Group Child bounds !? */ #if 0 gnome_canvas_group_child_bounds (GNOME_CANVAS_GROUP (item->parent), item); #endif diff --git a/widgets/table/e-table-header-item.c b/widgets/table/e-table-header-item.c index 8c76a4ac10..51f3301f8f 100644 --- a/widgets/table/e-table-header-item.c +++ b/widgets/table/e-table-header-item.c @@ -142,10 +142,8 @@ ethi_dispose (GObject *object){ g_object_unref (ethi->full_header); ethi->full_header = NULL; - if (ethi->etfcd) - g_object_remove_weak_pointer (G_OBJECT (ethi->etfcd), (gpointer *) (void *) ði->etfcd); - + g_object_remove_weak_pointer (G_OBJECT (ethi->etfcd), ði->etfcd_pointer); if (ethi->config) g_object_unref (ethi->config); @@ -220,9 +218,7 @@ ethi_update (GnomeCanvasItem *item, double *affine, ArtSVP *clip_path, int flags item->y1 = c1.y; item->x2 = c2.x; item->y2 = c2.y; -#ifndef NO_WARNINGS -#warning FOO BAA -#endif +/* FIXME: Group Child bounds !? (FOO BAA) */ #if 0 gnome_canvas_group_child_bounds (GNOME_CANVAS_GROUP (item->parent), item); #endif @@ -956,7 +952,7 @@ ethi_draw (GnomeCanvasItem *item, GdkDrawable *drawable, int x, int y, int width for (i = 0; i < length; i++) { ETableSortColumn column = e_table_sort_info_grouping_get_nth(ethi->sort_info, i); g_hash_table_insert (arrows, - GINT_TO_POINTER (column.column), + GINT_TO_POINTER ((gint) column.column), GINT_TO_POINTER (column.ascending ? E_TABLE_COL_ARROW_DOWN : E_TABLE_COL_ARROW_UP)); @@ -965,7 +961,7 @@ ethi_draw (GnomeCanvasItem *item, GdkDrawable *drawable, int x, int y, int width for (i = 0; i < length; i++) { ETableSortColumn column = e_table_sort_info_sorting_get_nth(ethi->sort_info, i); g_hash_table_insert (arrows, - GINT_TO_POINTER (column.column), + GINT_TO_POINTER ((gint) column.column), GINT_TO_POINTER (column.ascending ? E_TABLE_COL_ARROW_DOWN : E_TABLE_COL_ARROW_UP)); @@ -1154,7 +1150,7 @@ ethi_start_drag (ETableHeaderItem *ethi, GdkEvent *event) group_indent ++; g_hash_table_insert ( arrows, - GINT_TO_POINTER (column.column), + GINT_TO_POINTER ((gint) column.column), GINT_TO_POINTER (column.ascending ? E_TABLE_COL_ARROW_DOWN : E_TABLE_COL_ARROW_UP)); @@ -1167,7 +1163,7 @@ ethi_start_drag (ETableHeaderItem *ethi, GdkEvent *event) g_hash_table_insert ( arrows, - GINT_TO_POINTER (column.column), + GINT_TO_POINTER ((gint) column.column), GINT_TO_POINTER (column.ascending ? E_TABLE_COL_ARROW_DOWN : E_TABLE_COL_ARROW_UP)); @@ -1363,7 +1359,7 @@ ethi_popup_remove_column(GtkWidget *widget, EthiHeaderInfo *info) static void ethi_popup_field_chooser(GtkWidget *widget, EthiHeaderInfo *info) { - gpointer etfcd = (gpointer)info->ethi->etfcd; + GtkWidget *etfcd = info->ethi->etfcd; if (etfcd) { gtk_window_present (GTK_WINDOW (etfcd)); @@ -1372,9 +1368,8 @@ ethi_popup_field_chooser(GtkWidget *widget, EthiHeaderInfo *info) } info->ethi->etfcd = e_table_field_chooser_dialog_new (); - etfcd = (gpointer)info->ethi->etfcd; - g_object_add_weak_pointer (G_OBJECT (etfcd), (gpointer*) (void *) &info->ethi->etfcd); + g_object_add_weak_pointer (G_OBJECT (etfcd), &info->ethi->etfcd_pointer); g_object_set (info->ethi->etfcd, "full_header", info->ethi->full_header, diff --git a/widgets/table/e-table-header-item.h b/widgets/table/e-table-header-item.h index ebb4708720..50b5e3239e 100644 --- a/widgets/table/e-table-header-item.h +++ b/widgets/table/e-table-header-item.h @@ -95,9 +95,13 @@ typedef struct { ETableHeader *full_header; ETable *table; ETree *tree; - GtkWidget *etfcd; void *config; + union { + GtkWidget *etfcd; + gpointer etfcd_pointer; + }; + /* For keyboard navigation*/ int selected_col; diff --git a/widgets/table/e-table-header-utils.c b/widgets/table/e-table-header-utils.c index 782543ed42..c36e8d11b3 100644 --- a/widgets/table/e-table-header-utils.c +++ b/widgets/table/e-table-header-utils.c @@ -333,7 +333,7 @@ e_table_header_draw_button (GdkDrawable *drawable, ETableCol *ecol, int inner_width, inner_height; GdkGC *gc; PangoLayout *layout; - static GtkWidget *g_label = NULL; + static gpointer g_label = NULL; g_return_if_fail (drawable != NULL); g_return_if_fail (ecol != NULL); @@ -350,11 +350,11 @@ e_table_header_draw_button (GdkDrawable *drawable, ETableCol *ecol, gtk_widget_ensure_style (window); gtk_widget_ensure_style (button); g_label = GTK_BIN(button)->child; - g_object_add_weak_pointer (G_OBJECT (g_label), (gpointer *) &g_label); + g_object_add_weak_pointer (G_OBJECT (g_label), &g_label); gtk_widget_ensure_style (g_label); } - gc = g_label->style->fg_gc[state]; + gc = GTK_WIDGET (g_label)->style->fg_gc[state]; gdk_gc_set_clip_rectangle (gc, NULL); diff --git a/widgets/table/e-table.c b/widgets/table/e-table.c index dfb15bf073..0925e399f9 100644 --- a/widgets/table/e-table.c +++ b/widgets/table/e-table.c @@ -2926,9 +2926,7 @@ context_destroyed (gpointer data) { ETable *et = data; /* if (!GTK_OBJECT_DESTROYED (et)) */ -#ifndef NO_WARNINGS -#warning FIXME -#endif +/* FIXME: */ { et->last_drop_x = 0; et->last_drop_y = 0; |