8 files changed, 18 insertions, 21 deletions
diff --git a/configure.ac b/configure.ac
index f492ffa..e6b1b4f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -46,7 +46,7 @@ GLIB_GSETTINGS
 # Dependencies
 
 DBUS_REQUIRED_VERSION=0.30
-GLIB_REQUIRED_VERSION=2.50.0
+GLIB_REQUIRED_VERSION=2.58.0
 GTK_REQUIRED_VERSION=3.22.0
 X11_REQUIRED_VERSION=1.0
 LIBMATE_MENU_REQUIRED_VERSION=1.21.0
diff --git a/src/copy-theme-dialog.c b/src/copy-theme-dialog.c
index 816d789..a4aaff5 100644
--- a/src/copy-theme-dialog.c
+++ b/src/copy-theme-dialog.c
@@ -434,10 +434,8 @@ copy_theme_dialog_finalize (GObject *obj)
 CopyThemeDialog *dlg = COPY_THEME_DIALOG (obj);
 
 g_object_unref (dlg->priv->theme_dir);
- g_slist_foreach (dlg->priv->all_files, (GFunc) (g_object_unref), NULL);
- g_slist_free (dlg->priv->all_files);
- g_slist_foreach (dlg->priv->all_basenames, (GFunc) (g_free), NULL);
- g_slist_free (dlg->priv->all_basenames);
+ g_slist_free_full (dlg->priv->all_files, g_object_unref);
+ g_slist_free_full (dlg->priv->all_basenames, g_free);
 g_object_unref (dlg->priv->cancellable);
 
 if (parent_class->finalize)
diff --git a/src/gs-auth-pam.c b/src/gs-auth-pam.c
index 3ca3e42..ae9cea0 100644
--- a/src/gs-auth-pam.c
+++ b/src/gs-auth-pam.c
@@ -509,9 +509,10 @@ set_pam_error (GError **error,
 
 }
 
-static int
-gs_auth_thread_func (int auth_operation_fd)
+static gpointer
+gs_auth_thread_func (gpointer data)
 {
+ int auth_operation_fd;
 static const int flags = 0;
 int status;
 int status2;
@@ -519,6 +520,8 @@ gs_auth_thread_func (int auth_operation_fd)
 sigset_t set;
 const void *p;
 
+ auth_operation_fd = GPOINTER_TO_INT (data);
+
 timeout.tv_sec = 0;
 timeout.tv_nsec = 1;
 
@@ -602,7 +605,7 @@ done:
 */
 close (auth_operation_fd);
 
- return status;
+ return GINT_TO_POINTER (status);
 }
 
 static gboolean
@@ -664,7 +667,7 @@ gs_auth_pam_verify_user (pam_handle_t *handle,
 (GIOFunc) gs_auth_loop_quit, &thread_done);
 
 auth_thread = g_thread_new ("auththread",
- (GThreadFunc) gs_auth_thread_func,
+ gs_auth_thread_func,
 GINT_TO_POINTER (auth_operation_fds[1]));
 
 if (auth_thread == NULL)
diff --git a/src/gs-lock-plug.c b/src/gs-lock-plug.c
index fddce2a..cb27964 100644
--- a/src/gs-lock-plug.c
+++ b/src/gs-lock-plug.c
@@ -2141,7 +2141,7 @@ gs_lock_plug_init (GSLockPlug *plug)
 date_time_update (plug);
 gtk_widget_show_all (plug->priv->vbox);
 }
- plug->priv->datetime_timeout_id = g_timeout_add_seconds (1, (GSourceFunc) date_time_update, plug);
+ plug->priv->datetime_timeout_id = g_timeout_add_seconds (1, G_SOURCE_FUNC (date_time_update), plug);
 
 if (plug->priv->note_text_view != NULL)
 {
diff --git a/src/gs-manager.c b/src/gs-manager.c
index 00ec982..cca90d8 100644
--- a/src/gs-manager.c
+++ b/src/gs-manager.c
@@ -387,8 +387,7 @@ free_themes (GSManager *manager)
 {
 if (manager->priv->themes)
 {
- g_slist_foreach (manager->priv->themes, (GFunc)g_free, NULL);
- g_slist_free (manager->priv->themes);
+ g_slist_free_full (manager->priv->themes, g_free);
 }
 }
 
diff --git a/src/gs-prefs.c b/src/gs-prefs.c
index 5a0746b..3c3c074 100644
--- a/src/gs-prefs.c
+++ b/src/gs-prefs.c
@@ -202,8 +202,7 @@ _gs_prefs_set_themes (GSPrefs *prefs,
 guint i;
 if (prefs->themes)
 {
- g_slist_foreach (prefs->themes, (GFunc)g_free, NULL);
- g_slist_free (prefs->themes);
+ g_slist_free_full (prefs->themes, g_free);
 }
 
 /* take ownership of the list */
@@ -600,8 +599,7 @@ gs_prefs_finalize (GObject *object)
 
 if (prefs->themes)
 {
- g_slist_foreach (prefs->themes, (GFunc)g_free, NULL);
- g_slist_free (prefs->themes);
+ g_slist_free_full (prefs->themes, g_free);
 }
 
 g_free (prefs->logout_command);
diff --git a/src/mate-screensaver-dialog.c b/src/mate-screensaver-dialog.c
index 62daa98..2074a13 100644
--- a/src/mate-screensaver-dialog.c
+++ b/src/mate-screensaver-dialog.c
@@ -345,7 +345,7 @@ static gboolean auth_check_idle(GSLockPlug* plug)
 if (res)
 {
 again = FALSE;
- g_idle_add((GSourceFunc) quit_response_ok, NULL);
+ g_idle_add (G_SOURCE_FUNC (quit_response_ok), NULL);
 }
 else
 {
@@ -363,7 +363,7 @@ static gboolean auth_check_idle(GSLockPlug* plug)
 /* Don't quit immediately, but rather request that mate-screensaver
 * terminates us after it has finished the dialog shake. Time out
 * after 5 seconds and quit anyway if this doesn't happen though */
- g_idle_add((GSourceFunc) response_request_quit, NULL);
+ g_idle_add (G_SOURCE_FUNC (response_request_quit), NULL);
 g_timeout_add(5000, (GSourceFunc) quit_timeout_cb, NULL);
 }
 }
@@ -595,7 +595,7 @@ int main(int argc, char** argv)
 
 gs_debug_init(verbose, FALSE);
 
- g_idle_add((GSourceFunc) popup_dialog_idle, NULL);
+ g_idle_add (G_SOURCE_FUNC (popup_dialog_idle), NULL);
 
 gtk_main();
 
diff --git a/src/mate-screensaver-preferences.c b/src/mate-screensaver-preferences.c
index 844be50..4ead5ed 100644
--- a/src/mate-screensaver-preferences.c
+++ b/src/mate-screensaver-preferences.c
@@ -807,8 +807,7 @@ theme_installer_run (GtkWidget *prefs_dialog, GList *files)
 GtkWidget *copy_dialog;
 
 copy_dialog = copy_theme_dialog_new (files);
- g_list_foreach (files, (GFunc) (g_object_unref), NULL);
- g_list_free (files);
+ g_list_free_full (files, g_object_unref);
 
 gtk_window_set_transient_for (GTK_WINDOW (copy_dialog),
 GTK_WINDOW (prefs_dialog));