From ef2153739cb75348ee5e8c744701baac9361cb41 Mon Sep 17 00:00:00 2001 From: Benjamin Berg Date: Thu, 21 Feb 2019 13:30:38 +0100 Subject: [PATCH] idle-monitor: Fix duplicate return value of ResetIdletime call The signal handler must return TRUE as the invocation is already handled by returning an error. Also update the error message a bit to clarify that the API exists only for testing purposes. https://gitlab.gnome.org/GNOME/mutter/merge_requests/457 --- src/backends/meta-idle-monitor-dbus.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backends/meta-idle-monitor-dbus.c b/src/backends/meta-idle-monitor-dbus.c index e55275be7..2065f0177 100644 --- a/src/backends/meta-idle-monitor-dbus.c +++ b/src/backends/meta-idle-monitor-dbus.c @@ -55,8 +55,8 @@ handle_reset_idletime (MetaDBusIdleMonitor *skeleton, g_dbus_method_invocation_return_error_literal (invocation, G_DBUS_ERROR, G_DBUS_ERROR_UNKNOWN_METHOD, - "No such method"); - return FALSE; + "This method is for testing purposes only. MUTTER_DEBUG_RESET_IDLETIME must be set to use it"); + return TRUE; } meta_idle_monitor_reset_idletime (meta_idle_monitor_get_core ());