From 63af2cd04130783a4e37ffe1599ee1c5ecef77a9 Mon Sep 17 00:00:00 2001 From: Michael Natterer Date: Wed, 30 Jan 2008 15:06:06 +0000 Subject: [PATCH] use the right type for "subparser_data" and remove the (gpointer*) cast. 2008-01-30 Michael Natterer * gtk/gtkbuilderparser.c (parse_custom): use the right type for "subparser_data" and remove the (gpointer*) cast. Fixes bogus aliasing warning. * gtk/updateiconcache.c (add_string): cast const gchar* to gpointer when inserting in a GHashTable. * tests/testcalendar.c (calendar_detail_cb): remove const from return value since it's a newly allocated string. (calendar_update_details): free the detail. svn path=/trunk/; revision=19431 --- ChangeLog | 14 ++++++++++++++ gtk/gtkbuilderparser.c | 4 ++-- gtk/updateiconcache.c | 2 +- tests/testcalendar.c | 6 ++++-- 4 files changed, 21 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 9c1a96b791..04464d574c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,17 @@ +2008-01-30 Michael Natterer + + * gtk/gtkbuilderparser.c (parse_custom): use the right type for + "subparser_data" and remove the (gpointer*) cast. Fixes bogus + aliasing warning. + + * gtk/updateiconcache.c (add_string): cast const gchar* to + gpointer when inserting in a GHashTable. + + * tests/testcalendar.c (calendar_detail_cb): remove const from + return value since it's a newly allocated string. + + (calendar_update_details): free the detail. + 2008-01-29 Johan Dahlin * demos/gtk-demo/builder.c: (quit_activate), (about_activate), diff --git a/gtk/gtkbuilderparser.c b/gtk/gtkbuilderparser.c index ce4b7dbc91..db37467233 100644 --- a/gtk/gtkbuilderparser.c +++ b/gtk/gtkbuilderparser.c @@ -636,7 +636,7 @@ parse_custom (GMarkupParseContext *context, { CommonInfo* parent_info; GMarkupParser parser; - gpointer *subparser_data; + gpointer subparser_data; GObject *object; GObject *child; @@ -671,7 +671,7 @@ parse_custom (GMarkupParseContext *context, child, element_name, &parser, - (gpointer*)&subparser_data)) + &subparser_data)) return FALSE; data->subparser = create_subparser (object, child, element_name, diff --git a/gtk/updateiconcache.c b/gtk/updateiconcache.c index 45d0cf00ef..14462fb6fd 100644 --- a/gtk/updateiconcache.c +++ b/gtk/updateiconcache.c @@ -744,7 +744,7 @@ find_string (const gchar *n) static void add_string (const gchar *n, int offset) { - g_hash_table_insert (string_pool, n, GINT_TO_POINTER (offset)); + g_hash_table_insert (string_pool, (gpointer) n, GINT_TO_POINTER (offset)); } static gboolean diff --git a/tests/testcalendar.c b/tests/testcalendar.c index 4e1bb84947..62423b1c97 100644 --- a/tests/testcalendar.c +++ b/tests/testcalendar.c @@ -105,7 +105,7 @@ static void calendar_update_details (CalendarData *data) { guint year, month, day; - const gchar *detail; + gchar *detail; gtk_calendar_get_date (GTK_CALENDAR (data->calendar_widget), &year, &month, &day); detail = calendar_get_detail (data, year, month, day); @@ -113,6 +113,8 @@ calendar_update_details (CalendarData *data) g_signal_handler_block (data->details_buffer, data->details_changed); gtk_text_buffer_set_text (data->details_buffer, detail ? detail : "", -1); g_signal_handler_unblock (data->details_buffer, data->details_changed); + + g_free (detail); } static void @@ -257,7 +259,7 @@ void calendar_select_font (GtkWidget *button, } } -static G_CONST_RETURN gchar* +static gchar* calendar_detail_cb (GtkCalendar *calendar, guint year, guint month, -- 2.22.0