From 5e891cbb5cfb0de54588988f1c9969244a9317a3 Mon Sep 17 00:00:00 2001 From: David Tardon Date: Tue, 11 Apr 2023 17:25:42 +0200 Subject: tree-wide: drop unneeded output params Neither of the callers of bus_deserialize_and_dump_unit_file_changes() touches the changes array, so let's simplify things and keep it internal to the function. --- src/portable/portablectl.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'src/portable') diff --git a/src/portable/portablectl.c b/src/portable/portablectl.c index bb9182ad42..eec9c63d0b 100644 --- a/src/portable/portablectl.c +++ b/src/portable/portablectl.c @@ -556,13 +556,9 @@ static int maybe_enable_disable(sd_bus *bus, const char *path, bool enable) { _cleanup_(sd_bus_message_unrefp) sd_bus_message *m = NULL, *reply = NULL; _cleanup_(sd_bus_error_free) sd_bus_error error = SD_BUS_ERROR_NULL; _cleanup_strv_free_ char **names = NULL; - InstallChange *changes = NULL; const uint64_t flags = UNIT_FILE_PORTABLE | (arg_runtime ? UNIT_FILE_RUNTIME : 0); - size_t n_changes = 0; int r; - CLEANUP_ARRAY(changes, n_changes, install_changes_free); - if (!arg_enable) return 0; @@ -599,7 +595,7 @@ static int maybe_enable_disable(sd_bus *bus, const char *path, bool enable) { return bus_log_parse_error(r); } - (void) bus_deserialize_and_dump_unit_file_changes(reply, arg_quiet, &changes, &n_changes); + (void) bus_deserialize_and_dump_unit_file_changes(reply, arg_quiet); return 0; } -- cgit v1.2.1