From c3503fcc84eec0bcf857cc744580aa9a4d5dc7eb Mon Sep 17 00:00:00 2001 From: Matthias Clasen Date: Tue, 13 Apr 2021 14:10:27 -0400 Subject: [PATCH] x11: Be quiet on exit by default The condition we check for to catch X servers going away may not be accurate anymore, and the warning shows up in logs, causing customers to be concerned. So, be quiet by default, unless the user explicitly asked for a message. --- gdk/x11/gdkmain-x11.c | 23 ++++++----------------- 1 file changed, 6 insertions(+), 17 deletions(-) diff --git a/gdk/x11/gdkmain-x11.c b/gdk/x11/gdkmain-x11.c index 64c7cb4302..cd877ce3e4 100644 --- a/gdk/x11/gdkmain-x11.c +++ b/gdk/x11/gdkmain-x11.c @@ -240,24 +240,13 @@ gdk_x_io_error (Display *display) /* This is basically modelled after the code in XLib. We need * an explicit error handler here, so we can disable our atexit() * which would otherwise cause a nice segfault. - * We fprintf(stderr, instead of g_warning() because g_warning() - * could possibly be redirected to a dialog + * We g_debug() instead of g_warning(), because g_warning() + * could possibly be redirected to the log */ - if (errno == EPIPE) - { - g_message ("The application '%s' lost its connection to the display %s;\n" - "most likely the X server was shut down or you killed/destroyed\n" - "the application.\n", - g_get_prgname (), - display ? DisplayString (display) : gdk_get_display_arg_name ()); - } - else - { - g_message ("%s: Fatal IO error %d (%s) on X server %s.\n", - g_get_prgname (), - errno, g_strerror (errno), - display ? DisplayString (display) : gdk_get_display_arg_name ()); - } + g_debug ("%s: Fatal IO error %d (%s) on X server %s.\n", + g_get_prgname (), + errno, g_strerror (errno), + display ? DisplayString (display) : gdk_get_display_arg_name ()); _exit (1); } -- GitLab