xdg-desktop-portal-gnome/xdg-desktop-portal-gnome-c89.patch

82 lines
2.9 KiB
Diff

commit d0428d7b1aaf2c73293bdc003facc4cabe862e5d
Author: Georges Basile Stavracas Neto <georges.stavracas@gmail.com>
Date: Wed Dec 20 19:07:16 2023 -0300
inputcapture: Store dialog as a GtkWindow
This allows calling g_clear_pointer() without a cast.
Closes:
https://gitlab.gnome.org/GNOME/xdg-desktop-portal-gnome/-/issues/113
diff --git a/src/inputcapture.c b/src/inputcapture.c
index ffa91a1dacbbeee2..7dfefe1b2995f246 100644
--- a/src/inputcapture.c
+++ b/src/inputcapture.c
@@ -70,7 +70,7 @@ typedef struct _InputCaptureDialogHandle
char *session_handle;
GDBusMethodInvocation *create_session_invocation;
- GtkWidget *dialog;
+ GtkWindow *dialog;
ExternalWindow *external_parent;
unsigned int capabilities;
@@ -89,7 +89,7 @@ G_DEFINE_TYPE (InputCaptureSession, input_capture_session, session_get_type ())
static void
input_capture_dialog_handle_free (InputCaptureDialogHandle *dialog_handle)
{
- g_clear_pointer ((GtkWindow**)&dialog_handle->dialog, gtk_window_destroy);
+ g_clear_pointer (&dialog_handle->dialog, gtk_window_destroy);
g_clear_object (&dialog_handle->external_parent);
g_object_unref (dialog_handle->request);
g_free (dialog_handle->session_handle);
@@ -150,8 +150,7 @@ on_request_handle_close_cb (XdpImplRequest *object,
GDBusMethodInvocation *invocation,
InputCaptureDialogHandle *dialog_handle)
{
- gtk_window_close (GTK_WINDOW (dialog_handle->dialog));
-
+ gtk_window_close (dialog_handle->dialog);
return FALSE;
}
@@ -387,7 +386,7 @@ create_input_capture_dialog (GDBusMethodInvocation *invocation,
ExternalWindow *external_parent;
GdkSurface *surface;
GtkWidget *fake_parent;
- GtkWidget *dialog;
+ GtkWindow *dialog;
if (parent_window)
{
@@ -404,9 +403,9 @@ create_input_capture_dialog (GDBusMethodInvocation *invocation,
fake_parent = g_object_new (GTK_TYPE_WINDOW, NULL);
g_object_ref_sink (fake_parent);
- dialog = GTK_WIDGET (input_capture_dialog_new (request->app_id));
- gtk_window_set_transient_for (GTK_WINDOW (dialog), GTK_WINDOW (fake_parent));
- gtk_window_set_modal (GTK_WINDOW (dialog), TRUE);
+ dialog = GTK_WINDOW (input_capture_dialog_new (request->app_id));
+ gtk_window_set_transient_for (dialog, GTK_WINDOW (fake_parent));
+ gtk_window_set_modal (dialog, TRUE);
window_group = gtk_window_group_new ();
gtk_window_group_add_window (window_group, dialog);
@@ -424,13 +423,13 @@ create_input_capture_dialog (GDBusMethodInvocation *invocation,
g_signal_connect (dialog, "done",
G_CALLBACK (on_input_capture_dialog_done_cb), dialog_handle);
- gtk_widget_realize (dialog);
+ gtk_widget_realize (GTK_WIDGET (dialog));
surface = gtk_native_get_surface (GTK_NATIVE (dialog));
if (external_parent)
external_window_set_parent_of (external_parent, surface);
- gtk_widget_show (dialog);
+ gtk_window_present (dialog);
}
static gboolean