analyze: use sd_bus_get_propery_trivial and indentation

This commit is contained in:
Thomas Hindoe Paaboel Andersen 2013-10-31 23:08:16 +01:00
parent 0068027e86
commit a936124f7a

View file

@ -118,30 +118,23 @@ static void pager_open_if_enabled(void) {
} }
static int bus_get_uint64_property(sd_bus *bus, const char *path, const char *interface, const char *property, uint64_t *val) { static int bus_get_uint64_property(sd_bus *bus, const char *path, const char *interface, const char *property, uint64_t *val) {
_cleanup_bus_message_unref_ sd_bus_message *reply = NULL;
_cleanup_bus_error_free_ sd_bus_error error = SD_BUS_ERROR_NULL; _cleanup_bus_error_free_ sd_bus_error error = SD_BUS_ERROR_NULL;
int r; int r;
r = sd_bus_get_property(bus, r = sd_bus_get_property_trivial(
"org.freedesktop.systemd1", bus,
path, "org.freedesktop.systemd1",
interface, path,
property, interface,
&error, property,
&reply, &error,
"t"); 't', val);
if (r < 0) { if (r < 0) {
log_error("Failed to parse reply: %s", bus_error_message(&error, -r)); log_error("Failed to parse reply: %s", bus_error_message(&error, -r));
return r; return r;
} }
r = sd_bus_message_read(reply, "t", val);
if (r < 0) {
log_error("Failed to parse reply.");
return r;
}
return 0; return 0;
} }
@ -210,14 +203,15 @@ static int acquire_time_data(sd_bus *bus, struct unit_times **out) {
struct unit_times *unit_times = NULL; struct unit_times *unit_times = NULL;
struct unit_info u; struct unit_info u;
r = sd_bus_call_method(bus, r = sd_bus_call_method(
"org.freedesktop.systemd1", bus,
"/org/freedesktop/systemd1", "org.freedesktop.systemd1",
"org.freedesktop.systemd1.Manager", "/org/freedesktop/systemd1",
"ListUnits", "org.freedesktop.systemd1.Manager",
&error, "ListUnits",
&reply, &error,
""); &reply,
"");
if (r < 0) { if (r < 0) {
log_error("Failed to parse reply: %s", bus_error_message(&error, -r)); log_error("Failed to parse reply: %s", bus_error_message(&error, -r));
goto fail; goto fail;
@ -821,14 +815,15 @@ static int list_dependencies(sd_bus *bus, const char *name) {
if (path == NULL) if (path == NULL)
return -EINVAL; return -EINVAL;
r = sd_bus_get_property(bus, r = sd_bus_get_property(
"org.freedesktop.systemd1", bus,
path, "org.freedesktop.systemd1",
"org.freedesktop.systemd1.Unit", path,
"Id", "org.freedesktop.systemd1.Unit",
&error, "Id",
&reply, &error,
"s"); &reply,
"s");
if (r < 0) { if (r < 0) {
log_error("Failed to parse reply: %s", bus_error_message(&error, -r)); log_error("Failed to parse reply: %s", bus_error_message(&error, -r));
return r; return r;
@ -1060,14 +1055,15 @@ static int dot(sd_bus *bus, char* patterns[]) {
int r; int r;
struct unit_info u; struct unit_info u;
r = sd_bus_call_method(bus, r = sd_bus_call_method(
"org.freedesktop.systemd1", bus,
"/org/freedesktop/systemd1", "org.freedesktop.systemd1",
"org.freedesktop.systemd1.Manager", "/org/freedesktop/systemd1",
"ListUnits", "org.freedesktop.systemd1.Manager",
&error, "ListUnits",
&reply, &error,
""); &reply,
"");
if (r < 0) { if (r < 0) {
log_error("Failed to parse reply: %s", bus_error_message(&error, -r)); log_error("Failed to parse reply: %s", bus_error_message(&error, -r));
return r; return r;
@ -1112,14 +1108,15 @@ static int dump(sd_bus *bus, char **args) {
pager_open_if_enabled(); pager_open_if_enabled();
r = sd_bus_call_method(bus, r = sd_bus_call_method(
"org.freedesktop.systemd1", bus,
"/org/freedesktop/systemd1", "org.freedesktop.systemd1",
"org.freedesktop.systemd1.Manager", "/org/freedesktop/systemd1",
"Dump", "org.freedesktop.systemd1.Manager",
&error, "Dump",
&reply, &error,
""); &reply,
"");
if (r < 0) { if (r < 0) {
log_error("Failed to parse reply: %s", bus_error_message(&error, -r)); log_error("Failed to parse reply: %s", bus_error_message(&error, -r));
return r; return r;
@ -1150,14 +1147,15 @@ static int set_log_level(sd_bus *bus, char **args) {
value = args[0]; value = args[0];
r = sd_bus_set_property(bus, r = sd_bus_set_property(
"org.freedesktop.systemd1", bus,
"/org/freedesktop/systemd1", "org.freedesktop.systemd1",
"org.freedesktop.systemd1.Manager", "/org/freedesktop/systemd1",
"LogLevel", "org.freedesktop.systemd1.Manager",
&error, "LogLevel",
"s", &error,
value); "s",
value);
if (r < 0) { if (r < 0) {
log_error("Failed to issue method call: %s", bus_error_message(&error, -r)); log_error("Failed to issue method call: %s", bus_error_message(&error, -r));
return -EIO; return -EIO;