diff options
| author | iphydf <iphydf@users.noreply.github.com> | 2018-08-02 00:25:20 +0000 |
|---|---|---|
| committer | iphydf <iphydf@users.noreply.github.com> | 2018-08-04 09:04:42 +0000 |
| commit | 7f8b29c3465bae5b34999ad417508c2d0f6e6946 (patch) | |
| tree | cf15e14621c0f1ef63d586063ce636a5efc90009 /toxcore/DHT.c | |
| parent | aa63c1330c49ee1c4b675037e2e54516950203a2 (diff) | |
Avoid multiple for-next expressions.
All for-loops in toxcore are of the form
for (<for-init>; <for-cond>; <for-next>) { <body> }
`for-init` can be a variable declaration (like `int i = 0`), an
assignment (like `i = 0`), or empty.
`for-cond` can be any expression.
`for-next` can be an assignment or a single increment/decrement
expression (like `++i` or `--i`).
No other forms are allowed, so e.g. comma expressions in any of these are
not allowed (so no `for (i = 0, j = n; ...; ++i, --j)`).
Diffstat (limited to 'toxcore/DHT.c')
| -rw-r--r-- | toxcore/DHT.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/toxcore/DHT.c b/toxcore/DHT.c index 01fcfd66..543ee191 100644 --- a/toxcore/DHT.c +++ b/toxcore/DHT.c | |||
| @@ -1704,12 +1704,10 @@ static uint8_t do_ping_and_sendnode_requests(DHT *dht, uint64_t *lastgetnode, co | |||
| 1704 | /* If node is not dead. */ | 1704 | /* If node is not dead. */ |
| 1705 | Client_data *client = &list[i]; | 1705 | Client_data *client = &list[i]; |
| 1706 | 1706 | ||
| 1707 | IPPTsPng *const assocs[] = { &client->assoc6, &client->assoc4, nullptr }; | 1707 | IPPTsPng *const assocs[] = { &client->assoc6, &client->assoc4 }; |
| 1708 | |||
| 1709 | uint32_t j = 0; | ||
| 1710 | 1708 | ||
| 1711 | for (IPPTsPng * const *it = assocs; *it; ++it, ++j) { | 1709 | for (uint32_t j = 0; j < sizeof(assocs) / sizeof(assocs[0]); ++j) { |
| 1712 | IPPTsPng *const assoc = *it; | 1710 | IPPTsPng *const assoc = assocs[j]; |
| 1713 | 1711 | ||
| 1714 | if (!is_timeout(assoc->timestamp, KILL_NODE_TIMEOUT)) { | 1712 | if (!is_timeout(assoc->timestamp, KILL_NODE_TIMEOUT)) { |
| 1715 | sort = 0; | 1713 | sort = 0; |