diff options
| author | Umut Tezduyar Lindskog <umuttl@axis.com> | 2015-07-08 14:35:32 +0200 |
|---|---|---|
| committer | Umut Tezduyar Lindskog <umuttl@axis.com> | 2015-07-08 14:37:25 +0200 |
| commit | 0f8261015380be71f8a63c8aa37926c2a32d1870 (patch) | |
| tree | 8a3b0f0c48b9ea17c51d47c96031fa3ef3013bd6 /src/network/networkd-network-bus.c | |
| parent | 9a50ce20ef60263a6c88c29470ce761fcc424f2d (diff) | |
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.
Diffstat (limited to 'src/network/networkd-network-bus.c')
| -rw-r--r-- | src/network/networkd-network-bus.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/network/networkd-network-bus.c b/src/network/networkd-network-bus.c index b5f8f5cfb2..5717a15327 100644 --- a/src/network/networkd-network-bus.c +++ b/src/network/networkd-network-bus.c @@ -53,11 +53,7 @@ static int property_get_ether_addrs( return r; } - r = sd_bus_message_close_container(reply); - if (r < 0) - return r; - - return 1; + return sd_bus_message_close_container(reply); } const sd_bus_vtable network_vtable[] = { |