From 71f34e137dc675d30a56556cad2ff59b91cfaeda Mon Sep 17 00:00:00 2001 From: Philip Chimento Date: Sat, 31 Aug 2013 15:28:27 -0700 Subject: [PATCH] Remove support for older GLib and GTK versions The iLiad Reader was holding us back to GTK 2.6, but that product is discontinued and the company is bankrupt. --- libchimara/abort.c | 8 +------- libchimara/chimara-glk.c | 13 +++++-------- 2 files changed, 6 insertions(+), 15 deletions(-) diff --git a/libchimara/abort.c b/libchimara/abort.c index 3e85536..9ee66fc 100644 --- a/libchimara/abort.c +++ b/libchimara/abort.c @@ -143,13 +143,7 @@ shutdown_glk_post(void) /* Empty the event queue */ g_mutex_lock(&glk_data->event_lock); g_queue_foreach(glk_data->event_queue, (GFunc)g_free, NULL); - - /* COMPAT: g_queue_clear could be used here, but only appeared in 2.14 */ - // g_queue_clear(glk_data->event_queue); - g_list_free(glk_data->event_queue->head); - glk_data->event_queue->head = glk_data->event_queue->tail = NULL; - glk_data->event_queue->length = 0; - + g_queue_clear(glk_data->event_queue); g_mutex_unlock(&glk_data->event_lock); /* Reset the abort signaling mechanism */ diff --git a/libchimara/chimara-glk.c b/libchimara/chimara-glk.c index 540639a..90f9e9c 100644 --- a/libchimara/chimara-glk.c +++ b/libchimara/chimara-glk.c @@ -154,9 +154,8 @@ chimara_glk_init(ChimaraGlk *self) priv->glk_styles = g_new0(StyleSet,1); priv->final_message = g_strdup("[ The game has finished ]"); priv->event_queue = g_queue_new(); - priv->char_input_queue = g_async_queue_new(); - priv->line_input_queue = g_async_queue_new(); - /* FIXME Should be g_async_queue_new_full(g_free); but only in GTK >= 2.16 */ + priv->char_input_queue = g_async_queue_new_full(g_free); + priv->line_input_queue = g_async_queue_new_full(g_free); g_mutex_init(&priv->event_lock); g_mutex_init(&priv->abort_lock); @@ -1112,7 +1111,7 @@ free_startup_data(struct StartupData *startup) while(i < startup->args.argc) g_free(startup->args.argv[i++]); g_free(startup->args.argv); - g_free(startup); + g_slice_free(struct StartupData, startup); } /* glk_enter() is the actual function called in the new thread in which @@ -1142,7 +1141,6 @@ glk_enter(struct StartupData *startup) /* Run main function */ glk_main_t glk_main = startup->glk_main; - /* COMPAT: avoid usage of slices */ g_signal_emit_by_name(startup->glk_data->self, "started"); glk_main(); free_startup_data(startup); @@ -1185,9 +1183,8 @@ chimara_glk_run(ChimaraGlk *glk, const gchar *plugin, int argc, char *argv[], GE ChimaraGlkPrivate *priv = CHIMARA_GLK_PRIVATE(glk); - /* COMPAT: avoid usage of slices */ - struct StartupData *startup = g_new0(struct StartupData,1); - + struct StartupData *startup = g_slice_new0(struct StartupData); + g_assert( g_module_supported() ); /* If there is already a module loaded, free it first -- you see, we want to * keep modules loaded as long as possible to avoid crashes in stack unwinding */ -- 2.30.2