From: Matthias Clasen <mclasen@redhat.com>
Date: Mon, 31 Oct 2022 19:33:48 +0000 (-0400)
Subject: Be consistent in preconditions for async API
X-Git-Tag: archive/raspbian/4.12.3+ds-1+rpi1~1^2^2^2~22^2~9^2~125^2
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=030e8c0bf9ce05d078af59557b78da937a4c7ff1;p=gtk4.git

Be consistent in preconditions for async API

Do the same checks in all our finish functions.
---

diff --git a/gtk/gtkalertdialog.c b/gtk/gtkalertdialog.c
index a3dabb5845..74b7bbd997 100644
--- a/gtk/gtkalertdialog.c
+++ b/gtk/gtkalertdialog.c
@@ -733,11 +733,11 @@ int
 gtk_alert_dialog_choose_finish (GtkAlertDialog  *self,
                                 GAsyncResult   *result)
 {
-  GTask *task = G_TASK (result);
-
-  g_return_val_if_fail (g_task_get_source_tag (task) == gtk_alert_dialog_choose, -1);
+  g_return_val_if_fail (GTK_IS_ALERT_DIALOG (self), -1);
+  g_return_val_if_fail (g_task_is_valid (result, self), -1);
+  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_alert_dialog_choose, -1);
 
-  return (int) g_task_propagate_int (task, NULL);
+  return (int) g_task_propagate_int (G_TASK (result), NULL);
 }
 
 /**
diff --git a/gtk/gtkcolordialog.c b/gtk/gtkcolordialog.c
index f5d008b896..8c09570df9 100644
--- a/gtk/gtkcolordialog.c
+++ b/gtk/gtkcolordialog.c
@@ -487,6 +487,7 @@ gtk_color_dialog_choose_rgba_finish (GtkColorDialog  *self,
                                      GError         **error)
 {
   g_return_val_if_fail (GTK_IS_COLOR_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
   g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_color_dialog_choose_rgba, NULL);
 
   return g_task_propagate_pointer (G_TASK (result), error);
diff --git a/gtk/gtkfiledialog.c b/gtk/gtkfiledialog.c
index e32f330442..26c662f65e 100644
--- a/gtk/gtkfiledialog.c
+++ b/gtk/gtkfiledialog.c
@@ -801,11 +801,11 @@ gtk_file_dialog_open_finish (GtkFileDialog   *self,
                              GAsyncResult    *result,
                              GError         **error)
 {
-  GTask *task = G_TASK (result);
-
-  g_return_val_if_fail (g_task_get_source_tag (task) == gtk_file_dialog_open, NULL);
+  g_return_val_if_fail (GTK_IS_FILE_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
+  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_file_dialog_open, NULL);
 
-  return finish_file_op (self, task, error);
+  return finish_file_op (self, G_TASK (result), error);
 }
 
 /**
@@ -877,11 +877,11 @@ gtk_file_dialog_select_folder_finish (GtkFileDialog  *self,
                                       GAsyncResult   *result,
                                       GError        **error)
 {
-  GTask *task = G_TASK (result);
-
-  g_return_val_if_fail (g_task_get_source_tag (task) == gtk_file_dialog_select_folder, NULL);
+  g_return_val_if_fail (GTK_IS_FILE_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
+  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_file_dialog_select_folder, NULL);
 
-  return finish_file_op (self, task, error);
+  return finish_file_op (self, G_TASK (result), error);
 }
 
 /**
@@ -958,11 +958,11 @@ gtk_file_dialog_save_finish (GtkFileDialog   *self,
                              GAsyncResult    *result,
                              GError         **error)
 {
-  GTask *task = G_TASK (result);
-
-  g_return_val_if_fail (g_task_get_source_tag (task) == gtk_file_dialog_save, NULL);
+  g_return_val_if_fail (GTK_IS_FILE_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
+  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_file_dialog_save, NULL);
 
-  return finish_file_op (self, task, error);
+  return finish_file_op (self, G_TASK (result), error);
 }
 
 /**
@@ -1032,11 +1032,11 @@ gtk_file_dialog_open_multiple_finish (GtkFileDialog   *self,
                                       GAsyncResult    *result,
                                       GError         **error)
 {
-  GTask *task = G_TASK (result);
-
-  g_return_val_if_fail (g_task_get_source_tag (task) == gtk_file_dialog_open_multiple, NULL);
+  g_return_val_if_fail (GTK_IS_FILE_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
+  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_file_dialog_open_multiple, NULL);
 
-  return finish_multiple_files_op (self, task, error);
+  return finish_multiple_files_op (self, G_TASK (result), error);
 }
 
 /**
@@ -1106,11 +1106,11 @@ gtk_file_dialog_select_multiple_folders_finish (GtkFileDialog   *self,
                                                 GAsyncResult    *result,
                                                 GError         **error)
 {
-  GTask *task = G_TASK (result);
-
-  g_return_val_if_fail (g_task_get_source_tag (task) == gtk_file_dialog_select_multiple_folders, NULL);
+  g_return_val_if_fail (GTK_IS_FILE_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
+  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_file_dialog_select_multiple_folders, NULL);
 
-  return finish_multiple_files_op (self, task, error);
+  return finish_multiple_files_op (self, G_TASK (result), error);
 }
 
 /* }}} */
diff --git a/gtk/gtkfontdialog.c b/gtk/gtkfontdialog.c
index f51633715c..169b4a6029 100644
--- a/gtk/gtkfontdialog.c
+++ b/gtk/gtkfontdialog.c
@@ -691,6 +691,7 @@ gtk_font_dialog_choose_family_finish (GtkFontDialog  *self,
                                       GError        **error)
 {
   g_return_val_if_fail (GTK_IS_FONT_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
   g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_font_dialog_choose_family, NULL);
 
   return g_task_propagate_pointer (G_TASK (result), error);
@@ -769,6 +770,7 @@ gtk_font_dialog_choose_face_finish (GtkFontDialog  *self,
                                     GError        **error)
 {
   g_return_val_if_fail (GTK_IS_FONT_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
   g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_font_dialog_choose_face, NULL);
 
   return g_task_propagate_pointer (G_TASK (result), NULL);
@@ -845,6 +847,7 @@ gtk_font_dialog_choose_font_finish (GtkFontDialog  *self,
                                     GError        **error)
 {
   g_return_val_if_fail (GTK_IS_FONT_DIALOG (self), NULL);
+  g_return_val_if_fail (g_task_is_valid (result, self), NULL);
   g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_font_dialog_choose_font, NULL);
 
   return g_task_propagate_pointer (G_TASK (result), NULL);
@@ -932,6 +935,7 @@ gtk_font_dialog_choose_font_and_features_finish (GtkFontDialog         *self,
   FontResult *font_result;
 
   g_return_val_if_fail (GTK_IS_FONT_DIALOG (self), FALSE);
+  g_return_val_if_fail (g_task_is_valid (result, self), FALSE);
   g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_font_dialog_choose_font_and_features, FALSE);
 
   font_result = g_task_propagate_pointer (G_TASK (result), error);
diff --git a/gtk/gtkshow.c b/gtk/gtkshow.c
index 35a9ebf4ea..15c4ee1b11 100644
--- a/gtk/gtkshow.c
+++ b/gtk/gtkshow.c
@@ -125,7 +125,7 @@ gtk_show_uri_full (GtkWindow           *parent,
   data->context = G_APP_LAUNCH_CONTEXT (context);
   data->uri = g_strdup (uri);
   data->task = g_task_new (parent, cancellable, callback, user_data);
-  g_task_set_source_tag (data->task, gtk_show_uri);
+  g_task_set_source_tag (data->task, gtk_show_uri_full);
 
   if (!parent || !gtk_window_export_handle (parent, window_handle_exported, data))
     window_handle_exported (parent, NULL, data);
@@ -150,7 +150,7 @@ gtk_show_uri_full_finish (GtkWindow     *parent,
 {
   g_return_val_if_fail (parent == NULL || GTK_IS_WINDOW (parent), FALSE);
   g_return_val_if_fail (g_task_is_valid (result, parent), FALSE);
-  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_show_uri, FALSE);
+  g_return_val_if_fail (g_task_get_source_tag (G_TASK (result)) == gtk_show_uri_full, FALSE);
 
   return g_task_propagate_boolean (G_TASK (result), error);
 }