wayland: Don't abort when preparing the source if connection is lost
authorDebarshi Ray <debarshir@gnome.org>
Wed, 24 May 2017 14:08:07 +0000 (16:08 +0200)
committerDebarshi Ray <debarshir@gnome.org>
Thu, 25 May 2017 09:53:15 +0000 (11:53 +0200)
Aborting the application makes it look like an application bug, when
it is the expected thing to do when the Wayland display server goes
way. eg., when the user logs out. The log level is also demoted to
avoid a storm of warnings in the log from all applications whenever
this happens.

This is also what the X11 backend does (see gdk_x_io_error).

https://bugzilla.gnome.org/show_bug.cgi?id=783047

gdk/wayland/gdkeventsource.c

index 142091733b9248f26d869e12bfbf21be6e327653..bdb4c80e1eeec29f1c4078d22d12a6fd6bbe51f9 100644 (file)
@@ -63,7 +63,10 @@ gdk_event_source_prepare (GSource *base,
   source->reading = TRUE;
 
   if (wl_display_flush (display->wl_display) < 0)
-    g_error ("Error flushing display: %s", g_strerror (errno));
+    {
+      g_message ("Error flushing display: %s", g_strerror (errno));
+      _exit (1);
+    }
 
   return FALSE;
 }