From b3a9d980f303ab49614eee336fb9c54e98c9e530 Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Mon, 25 Dec 2023 01:41:11 +0900 Subject: [PATCH] tree-wide: drop space between variable and an increment/decrement --- src/basic/env-util.c | 2 +- src/basic/path-util.h | 2 +- src/basic/socket-util.c | 2 +- src/boot/bootctl.c | 2 +- src/core/dbus-job.c | 2 +- src/core/namespace.c | 8 ++++---- src/core/path.c | 2 +- src/core/service.c | 2 +- src/core/unit.c | 2 +- src/cryptenroll/cryptenroll-tpm2.c | 2 +- src/cryptenroll/cryptenroll.c | 2 +- src/delta/delta.c | 4 ++-- src/firstboot/firstboot.c | 2 +- src/home/user-record-sign.c | 4 ++-- src/import/qcow2-util.c | 2 +- src/journal/journald-context.c | 2 +- src/libsystemd-network/dhcp-option.c | 4 ++-- src/libsystemd/sd-bus/bus-creds.c | 2 +- src/libsystemd/sd-daemon/sd-daemon.c | 2 +- src/libsystemd/sd-device/sd-device.c | 2 +- src/libsystemd/sd-event/sd-event.c | 6 +++--- src/machine/machinectl.c | 2 +- src/network/networkd-dhcp-server.c | 2 +- src/nspawn/nspawn-bind-user.c | 2 +- src/nspawn/nspawn-oci.c | 2 +- src/nspawn/nspawn.c | 2 +- src/resolve/resolved-dns-transaction.c | 2 +- src/shared/bitmap.c | 4 ++-- src/shared/compare-operator.c | 2 +- src/shared/cryptsetup-fido2.c | 2 +- src/shared/dissect-image.c | 2 +- src/shared/fdset.c | 2 +- src/shared/format-table.c | 2 +- src/shared/fstab-util.c | 4 ++-- src/shared/hwdb-util.c | 2 +- src/shared/install.c | 2 +- src/shared/json.c | 10 +++++----- src/shared/loopback-setup.c | 6 +++--- src/shared/mount-setup.c | 2 +- src/shared/rm-rf.c | 2 +- src/shared/varlink-idl.c | 4 ++-- src/shutdown/detach-dm.c | 2 +- src/shutdown/detach-md.c | 2 +- src/sysext/sysext.c | 2 +- src/sysv-generator/sysv-generator.c | 2 +- src/test/test-bus-util.c | 2 +- src/test/test-capability.c | 2 +- src/test/test-mempool.c | 8 ++++---- 48 files changed, 68 insertions(+), 68 deletions(-) diff --git a/src/basic/env-util.c b/src/basic/env-util.c index 7ac47732ba8..23a128bfd13 100644 --- a/src/basic/env-util.c +++ b/src/basic/env-util.c @@ -683,7 +683,7 @@ int replace_env_full( pu = ret_unset_variables ? &unset_variables : NULL; pb = ret_bad_variables ? &bad_variables : NULL; - for (e = format, i = 0; *e && i < n; e ++, i ++) + for (e = format, i = 0; *e && i < n; e++, i++) switch (state) { case WORD: diff --git a/src/basic/path-util.h b/src/basic/path-util.h index 6d943e967f2..39b6714525e 100644 --- a/src/basic/path-util.h +++ b/src/basic/path-util.h @@ -156,7 +156,7 @@ int fsck_exists_for_fstype(const char *fstype); char *_p, *_n; \ size_t _l; \ while (_path[0] == '/' && _path[1] == '/') \ - _path ++; \ + _path++; \ if (isempty(_root)) \ _ret = _path; \ else { \ diff --git a/src/basic/socket-util.c b/src/basic/socket-util.c index 4f28d16b5e4..86472c88474 100644 --- a/src/basic/socket-util.c +++ b/src/basic/socket-util.c @@ -547,7 +547,7 @@ int sockaddr_pretty( } else { if (path[path_len - 1] == '\0') /* We expect a terminating NUL and don't print it */ - path_len --; + path_len--; p = cescape_length(path, path_len); } diff --git a/src/boot/bootctl.c b/src/boot/bootctl.c index 4614ca17801..f608e8cc8e0 100644 --- a/src/boot/bootctl.c +++ b/src/boot/bootctl.c @@ -324,7 +324,7 @@ static int parse_argv(int argc, char *argv[]) { break; case 'R': - arg_print_root_device ++; + arg_print_root_device++; break; case ARG_NO_VARIABLES: diff --git a/src/core/dbus-job.c b/src/core/dbus-job.c index c88d8c2dd5f..56ad88570f1 100644 --- a/src/core/dbus-job.c +++ b/src/core/dbus-job.c @@ -87,7 +87,7 @@ int bus_job_method_get_waiting_jobs(sd_bus_message *message, void *userdata, sd_ if (r < 0) return r; - for (int i = 0; i < n; i ++) { + for (int i = 0; i < n; i++) { _cleanup_free_ char *unit_path = NULL, *job_path = NULL; job_path = job_dbus_path(list[i]); diff --git a/src/core/namespace.c b/src/core/namespace.c index 50d7b05aa09..df6d0b4485c 100644 --- a/src/core/namespace.c +++ b/src/core/namespace.c @@ -1469,7 +1469,7 @@ static int follow_symlink( mount_entry_consume_prefix(m, TAKE_PTR(target)); - m->n_followed ++; + m->n_followed++; return 0; } @@ -2625,7 +2625,7 @@ int bind_mount_add(BindMount **b, size_t *n, const BindMount *item) { *b = c; - c[(*n) ++] = (BindMount) { + c[(*n)++] = (BindMount) { .source = TAKE_PTR(s), .destination = TAKE_PTR(d), .read_only = item->read_only, @@ -2694,7 +2694,7 @@ int mount_image_add(MountImage **m, size_t *n, const MountImage *item) { *m = c; - c[(*n) ++] = (MountImage) { + c[(*n)++] = (MountImage) { .source = TAKE_PTR(s), .destination = TAKE_PTR(d), .mount_options = TAKE_PTR(options), @@ -2745,7 +2745,7 @@ int temporary_filesystem_add( *t = c; - c[(*n) ++] = (TemporaryFileSystem) { + c[(*n)++] = (TemporaryFileSystem) { .path = TAKE_PTR(p), .options = TAKE_PTR(o), }; diff --git a/src/core/path.c b/src/core/path.c index 471d159d81d..f6b5683467a 100644 --- a/src/core/path.c +++ b/src/core/path.c @@ -90,7 +90,7 @@ int path_spec_watch(PathSpec *s, sd_event_io_handler_t handler) { /* If this is a symlink watch both the symlink inode and where it points to. If the inode is * not a symlink both calls will install the same watch, which is redundant and doesn't * hurt. */ - for (int follow_symlink = 0; follow_symlink < 2; follow_symlink ++) { + for (int follow_symlink = 0; follow_symlink < 2; follow_symlink++) { uint32_t f = flags; SET_FLAG(f, IN_DONT_FOLLOW, !follow_symlink); diff --git a/src/core/service.c b/src/core/service.c index 7bd4e99b11f..c6251079c61 100644 --- a/src/core/service.c +++ b/src/core/service.c @@ -2529,7 +2529,7 @@ static void service_enter_restart(Service *s) { /* Count the jobs we enqueue for restarting. This counter is maintained as long as the unit isn't * fully stopped, i.e. as long as it remains up or remains in auto-start states. The user can reset * the counter explicitly however via the usual "systemctl reset-failure" logic. */ - s->n_restarts ++; + s->n_restarts++; s->flush_n_restarts = false; s->notify_access_override = _NOTIFY_ACCESS_INVALID; diff --git a/src/core/unit.c b/src/core/unit.c index 0772a67001c..c5d84837912 100644 --- a/src/core/unit.c +++ b/src/core/unit.c @@ -2314,7 +2314,7 @@ void unit_start_on_failure( log_unit_warning_errno( u, r, "Failed to enqueue %s job, ignoring: %s", dependency_name, bus_error_message(&error, r)); - n_jobs ++; + n_jobs++; } if (n_jobs >= 0) diff --git a/src/cryptenroll/cryptenroll-tpm2.c b/src/cryptenroll/cryptenroll-tpm2.c index 653ad4452a1..0268d8f7721 100644 --- a/src/cryptenroll/cryptenroll-tpm2.c +++ b/src/cryptenroll/cryptenroll-tpm2.c @@ -25,7 +25,7 @@ static int search_policy_hash( if (hash_size == 0) return 0; - for (int token = 0; token < sym_crypt_token_max(CRYPT_LUKS2); token ++) { + for (int token = 0; token < sym_crypt_token_max(CRYPT_LUKS2); token++) { _cleanup_(json_variant_unrefp) JsonVariant *v = NULL; _cleanup_free_ void *thash = NULL; size_t thash_size = 0; diff --git a/src/cryptenroll/cryptenroll.c b/src/cryptenroll/cryptenroll.c index 5a7f7c3bff7..687b908f060 100644 --- a/src/cryptenroll/cryptenroll.c +++ b/src/cryptenroll/cryptenroll.c @@ -571,7 +571,7 @@ static int check_for_homed(struct crypt_device *cd) { /* Politely refuse operating on homed volumes. The enrolled tokens for the user record and the LUKS2 * volume should not get out of sync. */ - for (int token = 0; token < crypt_token_max(CRYPT_LUKS2); token ++) { + for (int token = 0; token < crypt_token_max(CRYPT_LUKS2); token++) { r = cryptsetup_get_token_as_json(cd, token, "systemd-homed", NULL); if (IN_SET(r, -ENOENT, -EINVAL, -EMEDIUMTYPE)) continue; diff --git a/src/delta/delta.c b/src/delta/delta.c index 3337b7f081a..3433250549b 100644 --- a/src/delta/delta.c +++ b/src/delta/delta.c @@ -314,7 +314,7 @@ static int enumerate_dir( dirs[n_dirs] = strdup(de->d_name); if (!dirs[n_dirs]) return -ENOMEM; - n_dirs ++; + n_dirs++; } if (!dirent_is_file(de)) @@ -326,7 +326,7 @@ static int enumerate_dir( files[n_files] = strdup(de->d_name); if (!files[n_files]) return -ENOMEM; - n_files ++; + n_files++; } strv_sort(dirs); diff --git a/src/firstboot/firstboot.c b/src/firstboot/firstboot.c index 5dd7bdb1e7a..0e92c56e802 100644 --- a/src/firstboot/firstboot.c +++ b/src/firstboot/firstboot.c @@ -166,7 +166,7 @@ static int show_menu(char **x, unsigned n_columns, unsigned width, unsigned perc for (i = 0; i < per_column; i++) { - for (j = 0; j < n_columns; j ++) { + for (j = 0; j < n_columns; j++) { _cleanup_free_ char *e = NULL; if (j * per_column + i >= n) diff --git a/src/home/user-record-sign.c b/src/home/user-record-sign.c index dd099a0a6f7..25618d0343a 100644 --- a/src/home/user-record-sign.c +++ b/src/home/user-record-sign.c @@ -136,11 +136,11 @@ int user_record_verify(UserRecord *ur, EVP_PKEY *public_key) { return -EIO; if (EVP_DigestVerify(md_ctx, signature, signature_size, (uint8_t*) text, strlen(text)) <= 0) { - n_bad ++; + n_bad++; continue; } - n_good ++; + n_good++; } return n_good > 0 ? (n_bad == 0 ? USER_RECORD_SIGNED_EXCLUSIVE : USER_RECORD_SIGNED) : diff --git a/src/import/qcow2-util.c b/src/import/qcow2-util.c index c70656bc0e0..fc34f306031 100644 --- a/src/import/qcow2-util.c +++ b/src/import/qcow2-util.c @@ -273,7 +273,7 @@ int qcow2_convert(int qcow2_fd, int raw_fd) { if ((uint64_t) l != sz) return -EIO; - for (i = 0; i < HEADER_L1_SIZE(&header); i ++) { + for (i = 0; i < HEADER_L1_SIZE(&header); i++) { uint64_t l2_begin, j; r = normalize_offset(&header, l1_table[i], &l2_begin, NULL, NULL); diff --git a/src/journal/journald-context.c b/src/journal/journald-context.c index 78401f9857c..b44f70006b5 100644 --- a/src/journal/journald-context.c +++ b/src/journal/journald-context.c @@ -612,7 +612,7 @@ static void client_context_try_shrink_to(Server *s, size_t limit) { if (pid_is_unwaited(c->pid) == 0) client_context_free(s, c); else - idx ++; + idx++; } s->last_cache_pid_flush = t; diff --git a/src/libsystemd-network/dhcp-option.c b/src/libsystemd-network/dhcp-option.c index 5e216c5139c..4025dd36325 100644 --- a/src/libsystemd-network/dhcp-option.c +++ b/src/libsystemd-network/dhcp-option.c @@ -283,7 +283,7 @@ static int parse_options(const uint8_t options[], size_t buflen, uint8_t *overlo int r; while (offset < buflen) { - code = options[offset ++]; + code = options[offset++]; switch (code) { case SD_DHCP_OPTION_PAD: @@ -296,7 +296,7 @@ static int parse_options(const uint8_t options[], size_t buflen, uint8_t *overlo if (buflen < offset + 1) return -ENOBUFS; - len = options[offset ++]; + len = options[offset++]; if (buflen < offset + len) return -EINVAL; diff --git a/src/libsystemd/sd-bus/bus-creds.c b/src/libsystemd/sd-bus/bus-creds.c index c6d8caaf6d7..33ae2a00f68 100644 --- a/src/libsystemd/sd-bus/bus-creds.c +++ b/src/libsystemd/sd-bus/bus-creds.c @@ -731,7 +731,7 @@ static int parse_caps(sd_bus_creds *c, unsigned offset, const char *p) { return -ENOMEM; } - for (i = 0; i < sz; i ++) { + for (i = 0; i < sz; i++) { uint32_t v = 0; for (j = 0; j < 8; ++j) { diff --git a/src/libsystemd/sd-daemon/sd-daemon.c b/src/libsystemd/sd-daemon/sd-daemon.c index 6a60cde4bbc..0eb0797e08f 100644 --- a/src/libsystemd/sd-daemon/sd-daemon.c +++ b/src/libsystemd/sd-daemon/sd-daemon.c @@ -76,7 +76,7 @@ _public_ int sd_listen_fds(int unset_environment) { goto finish; } - for (int fd = SD_LISTEN_FDS_START; fd < SD_LISTEN_FDS_START + n; fd ++) { + for (int fd = SD_LISTEN_FDS_START; fd < SD_LISTEN_FDS_START + n; fd++) { r = fd_cloexec(fd, true); if (r < 0) goto finish; diff --git a/src/libsystemd/sd-device/sd-device.c b/src/libsystemd/sd-device/sd-device.c index 1847ca4de05..071e72707ea 100644 --- a/src/libsystemd/sd-device/sd-device.c +++ b/src/libsystemd/sd-device/sd-device.c @@ -2488,7 +2488,7 @@ _public_ int sd_device_set_sysattr_value(sd_device *device, const char *sysattr, /* drop trailing newlines */ while (len > 0 && strchr(NEWLINE, _value[len - 1])) - len --; + len--; /* value length is limited to 4k */ if (len > 4096) diff --git a/src/libsystemd/sd-event/sd-event.c b/src/libsystemd/sd-event/sd-event.c index 288798a0dcc..b556fdb5348 100644 --- a/src/libsystemd/sd-event/sd-event.c +++ b/src/libsystemd/sd-event/sd-event.c @@ -1165,10 +1165,10 @@ static int source_set_pending(sd_event_source *s, bool b) { assert(s->inotify.inode_data->inotify_data); if (b) - s->inotify.inode_data->inotify_data->n_pending ++; + s->inotify.inode_data->inotify_data->n_pending++; else { assert(s->inotify.inode_data->inotify_data->n_pending > 0); - s->inotify.inode_data->inotify_data->n_pending --; + s->inotify.inode_data->inotify_data->n_pending--; } } @@ -4000,7 +4000,7 @@ static int process_inotify(sd_event *e) { if (r < 0) return r; if (r > 0) - done ++; + done++; } return done; diff --git a/src/machine/machinectl.c b/src/machine/machinectl.c index 3eadb5f4e77..c762e76fc63 100644 --- a/src/machine/machinectl.c +++ b/src/machine/machinectl.c @@ -2740,7 +2740,7 @@ static int parse_argv(int argc, char *argv[]) { /* If we already found the "shell" verb on the command line, and now found the next * non-option argument, then this is the machine name and we should stop processing * further arguments. */ - optind --; /* don't process this argument, go one step back */ + optind--; /* don't process this argument, go one step back */ goto done; } if (streq(optarg, "shell")) diff --git a/src/network/networkd-dhcp-server.c b/src/network/networkd-dhcp-server.c index 607fe0053c3..a486fd89275 100644 --- a/src/network/networkd-dhcp-server.c +++ b/src/network/networkd-dhcp-server.c @@ -423,7 +423,7 @@ static int dhcp4_server_configure(Link *link) { return log_link_warning_errno(link, r, "Failed to %s Rapid Commit support for DHCPv4 server instance: %m", enable_disable(link->network->dhcp_server_rapid_commit)); - for (sd_dhcp_lease_server_type_t type = 0; type < _SD_DHCP_LEASE_SERVER_TYPE_MAX; type ++) { + for (sd_dhcp_lease_server_type_t type = 0; type < _SD_DHCP_LEASE_SERVER_TYPE_MAX; type++) { if (!link->network->dhcp_server_emit[type].emit) continue; diff --git a/src/nspawn/nspawn-bind-user.c b/src/nspawn/nspawn-bind-user.c index 0a8653033d0..810ddbb45af 100644 --- a/src/nspawn/nspawn-bind-user.c +++ b/src/nspawn/nspawn-bind-user.c @@ -153,7 +153,7 @@ static int find_free_uid(const char *directory, uid_t max_uid, uid_t *current_ui assert(directory); assert(current_uid); - for (;; (*current_uid) ++) { + for (;; (*current_uid)++) { if (*current_uid > MAP_UID_MAX || *current_uid > max_uid) return log_error_errno( SYNTHETIC_ERRNO(EBUSY), diff --git a/src/nspawn/nspawn-oci.c b/src/nspawn/nspawn-oci.c index 8f1ac7ccadd..e3b6f895d17 100644 --- a/src/nspawn/nspawn-oci.c +++ b/src/nspawn/nspawn-oci.c @@ -2082,7 +2082,7 @@ static int oci_hooks_array(const char *name, JsonVariant *v, JsonDispatchFlags f return r; } - (*n_array) ++; + (*n_array)++; } return 0; diff --git a/src/nspawn/nspawn.c b/src/nspawn/nspawn.c index fb9df3d053f..4d3783e00a9 100644 --- a/src/nspawn/nspawn.c +++ b/src/nspawn/nspawn.c @@ -4455,7 +4455,7 @@ static int merge_settings(Settings *settings, const char *path) { #endif } - for (rl = 0; rl < _RLIMIT_MAX; rl ++) { + for (rl = 0; rl < _RLIMIT_MAX; rl++) { if ((arg_settings_mask & (SETTING_RLIMIT_FIRST << rl))) continue; diff --git a/src/resolve/resolved-dns-transaction.c b/src/resolve/resolved-dns-transaction.c index fe88e502e7c..c1d0603a187 100644 --- a/src/resolve/resolved-dns-transaction.c +++ b/src/resolve/resolved-dns-transaction.c @@ -489,7 +489,7 @@ static int dns_transaction_pick_server(DnsTransaction *t) { dns_server_unref(t->server); t->server = dns_server_ref(server); - t->n_picked_servers ++; + t->n_picked_servers++; log_debug("Using DNS server %s for transaction %u.", strna(dns_server_string_full(t->server)), t->id); diff --git a/src/shared/bitmap.c b/src/shared/bitmap.c index 6cf08b898f0..9aa766139bb 100644 --- a/src/shared/bitmap.c +++ b/src/shared/bitmap.c @@ -163,9 +163,9 @@ bool bitmap_iterate(const Bitmap *b, Iterator *i, unsigned *n) { rem = BITMAP_NUM_TO_REM(i->idx); bitmask = UINT64_C(1) << rem; - for (; offset < b->n_bitmaps; offset ++) { + for (; offset < b->n_bitmaps; offset++) { if (b->bitmaps[offset]) { - for (; bitmask; bitmask <<= 1, rem ++) { + for (; bitmask; bitmask <<= 1, rem++) { if (b->bitmaps[offset] & bitmask) { *n = BITMAP_OFFSET_TO_NUM(offset, rem); i->idx = *n + 1; diff --git a/src/shared/compare-operator.c b/src/shared/compare-operator.c index 0da28fc5c68..233e9c6b006 100644 --- a/src/shared/compare-operator.c +++ b/src/shared/compare-operator.c @@ -40,7 +40,7 @@ CompareOperator parse_compare_operator(const char **s, CompareOperatorParseFlags * parse_compare_operator() are use on the same string? */ return _COMPARE_OPERATOR_INVALID; - for (size_t i = 0; i < ELEMENTSOF(table); i ++) { + for (size_t i = 0; i < ELEMENTSOF(table); i++) { const char *e; if (table[i].need_mask != 0 && !FLAGS_SET(flags, table[i].need_mask)) diff --git a/src/shared/cryptsetup-fido2.c b/src/shared/cryptsetup-fido2.c index 285b82a7e2b..6e559e17f4a 100644 --- a/src/shared/cryptsetup-fido2.c +++ b/src/shared/cryptsetup-fido2.c @@ -150,7 +150,7 @@ int acquire_fido2_key_auto( /* Loads FIDO2 metadata from LUKS2 JSON token headers. */ - for (int token = 0; token < sym_crypt_token_max(CRYPT_LUKS2); token ++) { + for (int token = 0; token < sym_crypt_token_max(CRYPT_LUKS2); token++) { _cleanup_(json_variant_unrefp) JsonVariant *v = NULL; JsonVariant *w; _cleanup_free_ void *salt = NULL; diff --git a/src/shared/dissect-image.c b/src/shared/dissect-image.c index 98690367a15..6b0e5fe2484 100644 --- a/src/shared/dissect-image.c +++ b/src/shared/dissect-image.c @@ -3396,7 +3396,7 @@ int dissected_image_acquire_metadata(DissectedImage *m, DissectImageFlags extra_ assert(m); - for (; n_meta_initialized < _META_MAX; n_meta_initialized ++) { + for (; n_meta_initialized < _META_MAX; n_meta_initialized++) { assert(paths[n_meta_initialized]); if (pipe2(fds + 2*n_meta_initialized, O_CLOEXEC) < 0) { diff --git a/src/shared/fdset.c b/src/shared/fdset.c index e5b8e92e800..b408a38c371 100644 --- a/src/shared/fdset.c +++ b/src/shared/fdset.c @@ -246,7 +246,7 @@ int fdset_new_listen_fds(FDSet **ret, bool unset) { return -ENOMEM; n = sd_listen_fds(unset); - for (fd = SD_LISTEN_FDS_START; fd < SD_LISTEN_FDS_START + n; fd ++) { + for (fd = SD_LISTEN_FDS_START; fd < SD_LISTEN_FDS_START + n; fd++) { r = fdset_put(s, fd); if (r < 0) return r; diff --git a/src/shared/format-table.c b/src/shared/format-table.c index 41471dade03..66704f4660e 100644 --- a/src/shared/format-table.c +++ b/src/shared/format-table.c @@ -2632,7 +2632,7 @@ int table_print(Table *t, FILE *f) { } fputc('\n', f); - n_subline ++; + n_subline++; } while (more_sublines); } diff --git a/src/shared/fstab-util.c b/src/shared/fstab-util.c index 55e76b6e16b..ef3dae5b624 100644 --- a/src/shared/fstab-util.c +++ b/src/shared/fstab-util.c @@ -226,9 +226,9 @@ int fstab_filter_options( if (IN_SET(*end, ',', '\0')) break; assert(*end == '\\'); - end ++; /* Skip the backslash */ + end++; /* Skip the backslash */ if (*end != '\0') - end ++; /* Skip the escaped char, but watch out for a trailing comma */ + end++; /* Skip the escaped char, but watch out for a trailing comma */ } NULSTR_FOREACH(name, names) { diff --git a/src/shared/hwdb-util.c b/src/shared/hwdb-util.c index 648bfc50265..d96902c7f26 100644 --- a/src/shared/hwdb-util.c +++ b/src/shared/hwdb-util.c @@ -484,7 +484,7 @@ static int import_file(struct trie *trie, const char *filename, uint16_t file_pr if (r == 0) break; - line_number ++; + line_number++; /* comment line */ if (line[0] == '#') diff --git a/src/shared/install.c b/src/shared/install.c index 265407f080e..ad30e9b49c1 100644 --- a/src/shared/install.c +++ b/src/shared/install.c @@ -1810,7 +1810,7 @@ int unit_file_verify_alias( _cleanup_free_ char *dir = NULL; char *p; - path_alias ++; /* skip over slash */ + path_alias++; /* skip over slash */ r = path_extract_directory(dst, &dir); if (r < 0) diff --git a/src/shared/json.c b/src/shared/json.c index 06c9e850ea3..af2fc9aa70c 100644 --- a/src/shared/json.c +++ b/src/shared/json.c @@ -559,7 +559,7 @@ static int _json_variant_array_put_element(JsonVariant *array, JsonVariant *elem return -ELNRNG; if (d >= array->depth) array->depth = d + 1; - array->n_elements ++; + array->n_elements++; *w = (JsonVariant) { .is_embedded = true, @@ -2310,7 +2310,7 @@ static int json_variant_copy(JsonVariant **nv, JsonVariant *v) { source = json_variant_string(v); k = strnlen(source, INLINE_STRING_MAX + 1); if (k <= INLINE_STRING_MAX) { - k ++; + k++; break; } @@ -2594,7 +2594,7 @@ static int json_parse_string(const char **p, char **ret) { return -ENOMEM; s[n++] = ch; - c ++; + c++; continue; } @@ -4574,7 +4574,7 @@ int json_dispatch_full( } } - done ++; + done++; } else { /* Didn't find a matching entry! ☹️ */ @@ -4589,7 +4589,7 @@ int json_dispatch_full( return r; } else - done ++; + done++; } else { json_log(value, flags, 0, "Unexpected object field '%s'.", json_variant_string(key)); diff --git a/src/shared/loopback-setup.c b/src/shared/loopback-setup.c index a02baf8399d..84d723487a3 100644 --- a/src/shared/loopback-setup.c +++ b/src/shared/loopback-setup.c @@ -60,7 +60,7 @@ static int start_loopback(sd_netlink *rtnl, struct state *s) { if (r < 0) return r; - s->n_messages ++; + s->n_messages++; return 0; } @@ -95,7 +95,7 @@ static int add_ipv4_address(sd_netlink *rtnl, struct state *s) { if (r < 0) return r; - s->n_messages ++; + s->n_messages++; return 0; } @@ -136,7 +136,7 @@ static int add_ipv6_address(sd_netlink *rtnl, struct state *s) { if (r < 0) return r; - s->n_messages ++; + s->n_messages++; return 0; } diff --git a/src/shared/mount-setup.c b/src/shared/mount-setup.c index 1226ca121e1..d9de2789a58 100644 --- a/src/shared/mount-setup.c +++ b/src/shared/mount-setup.c @@ -135,7 +135,7 @@ bool mount_point_is_api(const char *path) { /* Checks if this mount point is considered "API", and hence * should be ignored */ - for (size_t i = 0; i < ELEMENTSOF(mount_table); i ++) + for (size_t i = 0; i < ELEMENTSOF(mount_table); i++) if (path_equal(path, mount_table[i].where)) return true; diff --git a/src/shared/rm-rf.c b/src/shared/rm-rf.c index 4664215e906..767db48cc0d 100644 --- a/src/shared/rm-rf.c +++ b/src/shared/rm-rf.c @@ -326,7 +326,7 @@ static int rm_rf_children_impl( dirname = mfree(dirname); /* And now let's back out one level up */ - n_todo --; + n_todo--; d = TAKE_PTR(todos[n_todo].dir); dirname = TAKE_PTR(todos[n_todo].dirname); old_mode = todos[n_todo].old_mode; diff --git a/src/shared/varlink-idl.c b/src/shared/varlink-idl.c index 655324c6c37..858b493d2f6 100644 --- a/src/shared/varlink-idl.c +++ b/src/shared/varlink-idl.c @@ -401,7 +401,7 @@ static int varlink_interface_realloc(VarlinkInterface **interface, size_t n_symb assert(interface); - n_symbols ++; /* Space for trailing NULL end marker symbol */ + n_symbols++; /* Space for trailing NULL end marker symbol */ /* Overflow check */ if (n_symbols > (SIZE_MAX - offsetof(VarlinkInterface, symbols)) / sizeof(VarlinkSymbol*)) @@ -420,7 +420,7 @@ static int varlink_symbol_realloc(VarlinkSymbol **symbol, size_t n_fields) { assert(symbol); - n_fields ++; /* Space for trailing end marker field */ + n_fields++; /* Space for trailing end marker field */ /* Overflow check */ if (n_fields > (SIZE_MAX - offsetof(VarlinkSymbol, fields)) / sizeof(VarlinkField)) diff --git a/src/shutdown/detach-dm.c b/src/shutdown/detach-dm.c index 8b8f72d678b..f6f672c75ad 100644 --- a/src/shutdown/detach-dm.c +++ b/src/shutdown/detach-dm.c @@ -132,7 +132,7 @@ static int dm_points_list_detach(DeviceMapper **head, bool *changed, bool last_t if ((major(rootdev) != 0 && rootdev == m->devnum) || (major(usrdev) != 0 && usrdev == m->devnum)) { log_debug("Not detaching DM %s that backs the OS itself, skipping.", m->path); - n_failed ++; + n_failed++; continue; } diff --git a/src/shutdown/detach-md.c b/src/shutdown/detach-md.c index cf3130d4a70..ac46670f04f 100644 --- a/src/shutdown/detach-md.c +++ b/src/shutdown/detach-md.c @@ -153,7 +153,7 @@ static int md_points_list_detach(RaidDevice **head, bool *changed, bool last_try if ((major(rootdev) != 0 && rootdev == m->devnum) || (major(usrdev) != 0 && usrdev == m->devnum)) { log_debug("Not detaching MD %s that backs the OS itself, skipping.", m->path); - n_failed ++; + n_failed++; continue; } diff --git a/src/sysext/sysext.c b/src/sysext/sysext.c index 33cb8c19635..fe11fe0f07c 100644 --- a/src/sysext/sysext.c +++ b/src/sysext/sysext.c @@ -854,7 +854,7 @@ static int merge_subprocess( if (r < 0) return log_oom(); - n_extensions ++; + n_extensions++; } /* Nothing left? Then shortcut things */ diff --git a/src/sysv-generator/sysv-generator.c b/src/sysv-generator/sysv-generator.c index 4485e2e3680..5689a60d785 100644 --- a/src/sysv-generator/sysv-generator.c +++ b/src/sysv-generator/sysv-generator.c @@ -807,7 +807,7 @@ static int set_dependencies_from_rcnd(const LookupPaths *lp, Hashmap *all_servic return r; STRV_FOREACH(p, sysvrcnd_path) - for (unsigned i = 0; i < ELEMENTSOF(rcnd_table); i ++) { + for (unsigned i = 0; i < ELEMENTSOF(rcnd_table); i++) { _cleanup_closedir_ DIR *d = NULL; _cleanup_free_ char *path = NULL; diff --git a/src/test/test-bus-util.c b/src/test/test-bus-util.c index 2f52bca59ae..2cf44d6e5ab 100644 --- a/src/test/test-bus-util.c +++ b/src/test/test-bus-util.c @@ -11,7 +11,7 @@ static int callback(sd_bus_message *m, void *userdata, sd_bus_error *ret_error) static void destroy_callback(void *userdata) { int *n_called = userdata; - (*n_called) ++; + (*n_called)++; } TEST(destroy_callback) { diff --git a/src/test/test-capability.c b/src/test/test-capability.c index e8a05697375..9df5f5ba070 100644 --- a/src/test/test-capability.c +++ b/src/test/test-capability.c @@ -54,7 +54,7 @@ static void test_last_cap_probe(void) { unsigned long p = (unsigned long)CAP_LAST_CAP; if (prctl(PR_CAPBSET_READ, p) < 0) { - for (p--; p > 0; p --) + for (p--; p > 0; p--) if (prctl(PR_CAPBSET_READ, p) >= 0) break; } else { diff --git a/src/test/test-mempool.c b/src/test/test-mempool.c index d3bc173fd45..1f071dba05a 100644 --- a/src/test/test-mempool.c +++ b/src/test/test-mempool.c @@ -35,7 +35,7 @@ TEST(mempool_trim) { assert_se(!a[x] || a[x]->value == x); if (a[x]) - n_freed ++; + n_freed++; a[x] = mempool_free_tile(&test_mempool, a[x]); } @@ -46,7 +46,7 @@ TEST(mempool_trim) { for (size_t i = 2; i < NN; i += 3) { assert_se(!a[i] || a[i]->value == i); if (a[i]) - n_freed ++; + n_freed++; a[i] = mempool_free_tile(&test_mempool, a[i]); } @@ -66,7 +66,7 @@ TEST(mempool_trim) { for (size_t i = 0; i < NN; i += 1) { assert_se(!a[i] || a[i]->value == i); if (a[i]) - n_freed ++; + n_freed++; a[i] = mempool_free_tile(&test_mempool, a[i]); } @@ -77,7 +77,7 @@ TEST(mempool_trim) { for (size_t i = 0; i < NN; i += 1) { assert_se(!b[i] || b[i]->value == ~(uint64_t) i); if (b[i]) - n_freed ++; + n_freed++; b[i] = mempool_free_tile(&test_mempool, b[i]); } -- 2.39.5