[glib] gdbusconnection: Don't g_printerr() when exiting
- From: Colin Walters <walters src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [glib] gdbusconnection: Don't g_printerr() when exiting
- Date: Thu, 6 Aug 2015 12:51:30 +0000 (UTC)
commit 66bc9660c44b71c8bff47b4f7e16a801169a9f23
Author: Colin Walters <walters verbum org>
Date: Mon Jan 5 09:40:37 2015 -0500
gdbusconnection: Don't g_printerr() when exiting
exit-on-close for a DBus connection is a completely normal thing. On
a regular GNOME login, gdm retains the X server, but terminates the
session login bus and associated helpers like gnome-settings-dameon,
the a11y tools, etc.
I've seen several downstream reports of confusion as to what these
apparent error messages mean in the system log. It doesn't help
that they're so obtuse.
We're also printing them to stderr, when this is not an error.
The reason this was introduced is presumably some people were confused
as to why their process exited when the system bus did. But the
solution for that I believe is documentation, not printing stuff to
everyone's system log in normal operation.
https://bugzilla.gnome.org/show_bug.cgi?id=742386
gio/gdbusconnection.c | 11 -----------
1 files changed, 0 insertions(+), 11 deletions(-)
---
diff --git a/gio/gdbusconnection.c b/gio/gdbusconnection.c
index 7b71bd0..9824ad9 100644
--- a/gio/gdbusconnection.c
+++ b/gio/gdbusconnection.c
@@ -804,17 +804,6 @@ g_dbus_connection_real_closed (GDBusConnection *connection,
(flags & FLAG_INITIALIZED) != 0 &&
connection->initialization_error == NULL)
{
- if (error != NULL)
- {
- g_printerr ("%s: Remote peer vanished with error: %s (%s, %d). Exiting.\n",
- G_STRFUNC,
- error->message,
- g_quark_to_string (error->domain), error->code);
- }
- else
- {
- g_printerr ("%s: Remote peer vanished. Exiting.\n", G_STRFUNC);
- }
raise (SIGTERM);
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]