xwayland: use g_autoptr for GError in xserver_died
authorRay Strode <rstrode@redhat.com>
Thu, 12 Apr 2018 18:03:03 +0000 (13:03 -0500)
committerMarco Trevisan (Treviño) <marco@ubuntu.com>
Mon, 23 Apr 2018 15:46:57 +0000 (16:46 +0100)
Right now we explicitly g_clear_error any error we find, but that
makes it tricky to return early from the function, which a
subsequent commit will want to to do.

This commit switches GError to use g_autoptr so the error clearing
happens automatically.

Origin: https://gitlab.gnome.org/GNOME/mutter/commit/bb65854

Gbp-Pq: Name xwayland-use-g_autoptr-for-GError-in-xserver_died.patch

src/wayland/meta-xwayland.c

index 4a7c8fffdb9372db7d6dfe7ff0f4b4626f350afc..7749976115038f1e52620ea6f66591375238d867 100644 (file)
@@ -393,13 +393,12 @@ xserver_died (GObject      *source,
               gpointer      user_data)
 {
   GSubprocess *proc = G_SUBPROCESS (source);
-  GError *error = NULL;
+  g_autoptr (GError) error = NULL;
 
   if (!g_subprocess_wait_finish (proc, result, &error))
     {
       if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
         g_error ("Failed to finish waiting for Xwayland: %s", error->message);
-      g_clear_error (&error);
     }
   else if (!g_subprocess_get_successful (proc))
     g_error ("X Wayland crashed; aborting");