diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2014-04-21 17:04:24 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-05-13 21:45:20 +0200 |
commit | 3d5908ed0faf400a5c7d9c3e4312597ee1b7786a (patch) | |
tree | 7013e475a41c90370ddd639c7bca4e8d4fc5abda /src/sbus | |
parent | 997d4bcd8e994932ac15c42524bb9f00055ee322 (diff) | |
download | sssd-3d5908ed0faf400a5c7d9c3e4312597ee1b7786a.tar.gz sssd-3d5908ed0faf400a5c7d9c3e4312597ee1b7786a.tar.xz sssd-3d5908ed0faf400a5c7d9c3e4312597ee1b7786a.zip |
SBUS: several trivial style fixes
In SSSD we tend to use {} brackets around single-line blocks, too to
make sure we don't forget to add them should the block become larger.
We also don't add a space between function name and the opening "(".
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Reviewed-by: Stef Walter <stefw@redhat.com>
Diffstat (limited to 'src/sbus')
-rw-r--r-- | src/sbus/sssd_dbus_connection.c | 2 | ||||
-rw-r--r-- | src/sbus/sssd_dbus_meta.c | 9 | ||||
-rw-r--r-- | src/sbus/sssd_dbus_request.c | 4 |
3 files changed, 9 insertions, 6 deletions
diff --git a/src/sbus/sssd_dbus_connection.c b/src/sbus/sssd_dbus_connection.c index 3cbc72437..f27474745 100644 --- a/src/sbus/sssd_dbus_connection.c +++ b/src/sbus/sssd_dbus_connection.c @@ -302,7 +302,7 @@ DBusConnection *sbus_get_connection(struct sbus_connection *conn) return conn->dbus.conn; } -void sbus_disconnect (struct sbus_connection *conn) +void sbus_disconnect(struct sbus_connection *conn) { if (conn == NULL) { return; diff --git a/src/sbus/sssd_dbus_meta.c b/src/sbus/sssd_dbus_meta.c index 57123b743..76e192f71 100644 --- a/src/sbus/sssd_dbus_meta.c +++ b/src/sbus/sssd_dbus_meta.c @@ -28,8 +28,9 @@ sbus_meta_find_method(const struct sbus_interface_meta *interface, const struct sbus_method_meta *method; for (method = interface->methods; method && method->name; method++) { - if (strcmp(method_name, method->name) == 0) + if (strcmp(method_name, method->name) == 0) { return method; + } } return NULL; @@ -42,8 +43,9 @@ sbus_meta_find_signal(const struct sbus_interface_meta *interface, const struct sbus_signal_meta *signal; for (signal = interface->signals; signal && signal->name; signal++) { - if (strcmp(signal_name, signal->name) == 0) + if (strcmp(signal_name, signal->name) == 0) { return signal; + } } return NULL; @@ -56,8 +58,9 @@ sbus_meta_find_property(const struct sbus_interface_meta *interface, const struct sbus_property_meta *property; for (property = interface->properties; property && property->name; property++) { - if (strcmp(property_name, property->name) == 0) + if (strcmp(property_name, property->name) == 0) { return property; + } } return NULL; diff --git a/src/sbus/sssd_dbus_request.c b/src/sbus/sssd_dbus_request.c index 387691dea..342d8dfd2 100644 --- a/src/sbus/sssd_dbus_request.c +++ b/src/sbus/sssd_dbus_request.c @@ -185,7 +185,7 @@ parent_dbus_string_arrays(struct sbus_request *request, int first_arg_type, arg_type == DBUS_TYPE_SIGNATURE) { array_arg = talloc_zero(request, struct array_arg); - if(array_arg == NULL) { + if (array_arg == NULL) { /* no kidding ... */ DEBUG(SSSDBG_CRIT_FAILURE, "Out of memory while trying not to leak memory\n"); return false; @@ -222,7 +222,7 @@ sbus_request_parse_or_finish(struct sbus_request *request, if (dbus_message_get_args_valist(request->message, &error, first_arg_type, va)) { - ret = parent_dbus_string_arrays (request, first_arg_type, va2); + ret = parent_dbus_string_arrays(request, first_arg_type, va2); } else { /* Trying to send the error back to the caller in this case is a joke */ |