From 8286423f0eaf8b4cd8229612bdcbeef381cf1a09 Mon Sep 17 00:00:00 2001 From: Umut Tezduyar Lindskog Date: Wed, 8 Jul 2015 14:35:32 +0200 Subject: [PATCH] property callback returns are consistent It is no different to return 0 over 1 in the property callback. It is confusing to return 1 which made me think 1 has a special purpose. This way code is consistent with the rest of the tree. --- src/login/logind-user-dbus.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/login/logind-user-dbus.c b/src/login/logind-user-dbus.c index 0f72d70b1..36c0e8626 100644 --- a/src/login/logind-user-dbus.c +++ b/src/login/logind-user-dbus.c @@ -103,11 +103,7 @@ static int property_get_sessions( } - r = sd_bus_message_close_container(reply); - if (r < 0) - return r; - - return 1; + return sd_bus_message_close_container(reply); } static int property_get_idle_hint( -- 2.30.2