From: Zbigniew Jędrzejewski-Szmek Date: Tue, 2 Jun 2020 15:46:58 +0000 (+0200) Subject: resolvectl,systemctl: reduce scope of variables X-Git-Tag: v246-rc1~211^2~3 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c9d243cdef279907cf51715ac4d32a007a03fbff;p=thirdparty%2Fsystemd.git resolvectl,systemctl: reduce scope of variables --- diff --git a/src/resolve/resolvectl.c b/src/resolve/resolvectl.c index fd8c834b8b7..f0673dab5dd 100644 --- a/src/resolve/resolvectl.c +++ b/src/resolve/resolvectl.c @@ -1753,7 +1753,6 @@ static int status_global(sd_bus *bus, StatusMode mode, bool *empty_line) { static int status_all(sd_bus *bus, StatusMode mode) { _cleanup_(sd_netlink_message_unrefp) sd_netlink_message *req = NULL, *reply = NULL; _cleanup_(sd_netlink_unrefp) sd_netlink *rtnl = NULL; - sd_netlink_message *i; bool empty_line = false; int r; @@ -1780,7 +1779,7 @@ static int status_all(sd_bus *bus, StatusMode mode) { return log_error_errno(r, "Failed to enumerate links: %m"); r = 0; - for (i = reply; i; i = sd_netlink_message_next(i)) { + for (sd_netlink_message *i = reply; i; i = sd_netlink_message_next(i)) { const char *name; int ifindex, q; uint16_t type; @@ -3033,7 +3032,7 @@ static int native_main(int argc, char *argv[], sd_bus *bus) { static int translate(const char *verb, const char *single_arg, size_t num_args, char **args, sd_bus *bus) { char **fake, **p; - size_t num, i; + size_t num; assert(verb); assert(num_args == 0 || args); @@ -3044,7 +3043,7 @@ static int translate(const char *verb, const char *single_arg, size_t num_args, *p++ = (char *) verb; if (single_arg) *p++ = (char *) single_arg; - for (i = 0; i < num_args; i++) + for (size_t i = 0; i < num_args; i++) *p++ = args[i]; optind = 0; diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index e085ff99e9f..4d59b5d1f59 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c @@ -397,8 +397,6 @@ static int output_table(Table *table) { static int output_units_list(const UnitInfo *unit_infos, unsigned c) { _cleanup_(table_unrefp) Table *table = NULL; - const UnitInfo *u; - int job_count = 0; int r; table = table_new("", "unit", "load", "active", "sub", "job", "description"); @@ -417,7 +415,8 @@ static int output_units_list(const UnitInfo *unit_infos, unsigned c) { (void) table_set_empty_string(table, "-"); - for (u = unit_infos; unit_infos && u < unit_infos + c; u++) { + int job_count = 0; + for (const UnitInfo *u = unit_infos; unit_infos && u < unit_infos + c; u++) { _cleanup_free_ char *j = NULL; const char *on_underline = "", *on_loaded = "", *on_active = ""; const char *on_circle = "", *id;